[openstreetmap/openstreetmap-website] Api timeout rollbacks (#1381)

Tom Hughes notifications at github.com
Mon Nov 28 12:10:58 UTC 2016


Do we actually need a separate class? The only requirement to make the exception visible within the block is to pass the class as an argument, so we could just pass the normal Timeout class...

Also, do we need to do it for the web timeout? I'd rather limit the scope if possible as there is the potential for this to go wrong if anything catches the timeout and doesn't rethrow it, which is I presume why the timeout library behaves the way it does in the first place.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/1381#issuecomment-263255815
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20161128/0b34b5c5/attachment.html>


More information about the rails-dev mailing list