eduvpn / apple

app for iOS and macOS
Other
58 stars 18 forks source link

Better error messages #511

Closed roop closed 1 year ago

roop commented 1 year ago

Fixes #489.

All errors from accessing the server API (APIv3), or on parsing the data got from the server API (including JSON parsing errors), show the server URL.

@fkooman Some examples:

"/info" errors out:

Screenshot 2023-02-13 at 3 37 37 PM

"/connect" errors out:

Screenshot 2023-02-13 at 3 38 02 PM

Error parsing returned JSON:

Screenshot 2023-02-13 at 3 36 15 PM

Only the most used APIv2 stuff is updated -- the rest of APIv2's error messaging is left unimproved.

ghost commented 1 year ago

Awesome :heart: Thanks!