[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