Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle OSM API Timeouts #1487

Open
bhousel opened this issue Jul 11, 2024 · 1 comment
Open

Handle OSM API Timeouts #1487

bhousel opened this issue Jul 11, 2024 · 1 comment
Assignees
Labels
chore Improvements to the code or development stack, cleanups

Comments

@bhousel
Copy link
Contributor

bhousel commented Jul 11, 2024

Followup on #1304
We already handle a bunch of different error conditions with the OSM API.
But when things get really bad, we don't even get a response from the capabilities or any of the other calls, just a connection timeout:

Screenshot 2024-07-11 at 9 41 22 AM

We should handle this situation too.

@bhousel bhousel added the chore Improvements to the code or development stack, cleanups label Jul 11, 2024
@bhousel
Copy link
Contributor Author

bhousel commented Jul 11, 2024

This relates to #1451 too where I said:

While we're at it, the AbortSignal can now have a timeout. This is a thing that all modern browsers support now - details. The default fetch timeout in Chrome is 300 seconds, but we should probably drop most requests after like 5-10sec.

The timeout should be quicker.

Some data taken during today's DDoS storm:
Screenshot 2024-07-11 at 9 49 37 AM

2.4 minutes is kinda a long time to wait for a response 😆
After a few seconds we can cancel it and display the red bar of sadness.

@bhousel bhousel self-assigned this Jul 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Improvements to the code or development stack, cleanups
Projects
None yet
Development

No branches or pull requests

1 participant