[openstreetmap/openstreetmap-website] Remove redundant cookies_serializer initializer (PR #6475)
rkoeze
notifications at github.com
Sat Oct 25 10:43:01 UTC 2025
### Description
In recent years [Rails has cut down on the number of initializers](https://github.com/rails/rails/pull/42538) it ships with. Some because they are commented out files users may never need. Others because they specify values that are already set by default. This PR, like https://github.com/openstreetmap/openstreetmap-website/pull/6456, simplifies our initializers folder by removing an initializer[ that is already set by default](https://github.com/rails/rails/blob/f73b7f173b3004ec9e920b99dafaaf029887847a/railties/lib/rails/application/configuration.rb#L231) and no longer ships with new Rails projects.
### How has this been tested?
The tests pass and I was able to navigate the app. Running `Rails.application.config.action_dispatch.cookies_serializer` returned `:json` before and after this change.
You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/6475
-- Commit Summary --
* Remove serializer that sets already set default
-- File Changes --
D config/initializers/cookies_serializer.rb (7)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/6475.patch
https://github.com/openstreetmap/openstreetmap-website/pull/6475.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6475
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/6475 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20251025/6e8b5441/attachment-0001.htm>
More information about the rails-dev
mailing list