[openstreetmap/openstreetmap-website] Fix changeset comments feed timeout (PR #5178)

Anton Khorev notifications at github.com
Mon Sep 9 16:29:06 UTC 2024


Was part of #4617 which is outdated now.

Comments feed timeout had a html template despite not being an html page and an atom template despite not being an atom feed.
You can view, comment on, or merge this pull request online at:

  https://github.com/openstreetmap/openstreetmap-website/pull/5178

-- Commit Summary --

  * Timeout instantly if web_timeout setting is negative
  * Respond to timeouts with a server error code
  * Fix changeset comments feed timeout

-- File Changes --

    M app/controllers/application_controller.rb (11)
    D app/views/changeset_comments/feeds/timeout.atom.builder (12)
    D app/views/changeset_comments/feeds/timeout.html.erb (1)
    A app/views/changeset_comments/feeds/timeout.rss.builder (12)
    M test/controllers/changeset_comments/feeds_controller_test.rb (20)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/5178.patch
https://github.com/openstreetmap/openstreetmap-website/pull/5178.diff

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

Message ID: <openstreetmap/openstreetmap-website/pull/5178 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240909/0d7f8791/attachment-0001.htm>


More information about the rails-dev mailing list