[openstreetmap/openstreetmap-website] Fix naming in abstract adapter initializer (PR #6476)

rkoeze notifications at github.com
Fri Nov 28 17:36:47 UTC 2025


rkoeze left a comment (openstreetmap/openstreetmap-website#6476)

@pablobm My instinct would be to do what you're describing: let timeouts be configuration details when possible and then fill in any gaps with more targeted/manual solutions where needed.

I think the issue with Timeout is that it can inject an exception into a thread at essentially any point. That kind of behavior is hard to reason about and makes it very difficult to ship a single fix. We’ll always be playing whackamole as exceptions surface in different parts of the stack.

If there isn’t already, I’ll create an issue to centralize this discussion.
 

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6476#issuecomment-3590012581
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/6476/c3590012581 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20251128/bb616e51/attachment.htm>


More information about the rails-dev mailing list