[openstreetmap/openstreetmap-website] Fix timeout messages for former browse pages (PR #4617)

Tom Hughes notifications at github.com
Thu Apr 4 16:59:11 UTC 2024


@tomhughes commented on this pull request.



> @@ -11,7 +11,7 @@ class NotesController < ApplicationController
 
   before_action :lookup_user, :only => [:index]
   before_action :set_locale
-  around_action :web_timeout
+  around_action :web_timeout, :only => :show

Just because there wasn't a template defined doesn't mean the timeout didn't work and protect the site - it just means the user got a bad response when the timeout happened!

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

Message ID: <openstreetmap/openstreetmap-website/pull/4617/review/1980597896 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240404/cdfed34c/attachment.htm>


More information about the rails-dev mailing list