Temporary Override fails

[kristianwh]

Describe the bug
Trying to set a temporary override.
This results in an error message in the browser:

To Reproduce
Steps to reproduce the behavior:

1. Go to Log treatment
2. Choose Event type "Temporary override"
3. Choose a "Reason"
4. Click Submit form
5. See error

Expected behavior
Expected that the override should be set

Screenshots

Your setup information

• What version of Nightscout (e.g. 14.2.3)
• What type of CGM, and how do you get your data there? (Dexcom G6 and Bridge, but N/A in this case)
• Is your issue specific to a browser (Firefox/Safari/Chrome?) or a device (Android phone, etc.)? No, fails in all I've tried.

Additional context
Also confirmed by others who upgraded to this version.

[bjornoleh]

@sulkaharo and @bewest , it seems Release 14.2.3 broke remote overrides for iOS Loop.

Let me know if you need more info, I may have some time to test right now.

[sulkaharo]

@bewest Wonder if it's 14.2.2...14.2.3#diff-770ec3cf5b644df350f403859877711e4e19266bc83793911a4150872dba7272L19 removing the extended parsing?

[bjornoleh]

@bewest Wonder if it's 14.2.2...14.2.3diff-770ec3cf5b644df350f403859877711e4e19266bc83793911a4150872dba7272L19 removing the extended parsing?

@sulkaharo It wasn´t quite clear to me which diff you were trying to link to above, but I took a chance it might be related to 41fdb8e (respect per route body-parser configuration)

Reverting this one commit did allow me to send remote overrides to iOS Loop, so it seems like this is the culprit.

[sulkaharo]

#7144 should fix this

[bjornoleh]

#7144 should fix this

Thanks @sulkaharo, I just deployed your add_wares_to_v2 branch, and remote overrides is working for me now.

[bjornoleh]

I just deployed dev, which is also working now that the PR was merged.

[bewest]

Thanks to your contributions, we've released 14.2.4 https://github.com/nightscout/cgm-remote-monitor/releases/tag/14.2.4. Closing for now.