Hotfix: handle 500 errors in app

ruuvi / com.ruuvi.station.webui

Ruuvi webUI

BSD 3-Clause "New" or "Revised" License

4 stars 3 forks source link

Hotfix: handle 500 errors in app #423

Closed markoaamunkajo closed 2 weeks ago

markoaamunkajo commented 3 months ago

Description:

We have found that 500 errors can cause surprising issues in apps, we need to handle those in apps. Check also that Bad internet connection doesn't cause issues.

markoaamunkajo commented 3 weeks ago

@io53

500 error:

When error was pushed to staging, the page did not tell about the 500 error status, this is fine on a browser app imo, no need for changes.
When I then reloaded page, it started to push internal error message in 3 seconds interval. This is confirmed on server logs:

We need to change this retry interval not to ddos ourselves. In Android app we changed this to 1 minute retry, I think it would be sufficient for Web too.

markoaamunkajo commented 3 weeks ago

Re-tested, now no ddos issues, this is ready to deploy.