[openstreetmap/openstreetmap-website] Update to Omniauth 2.x (#3078)

Tom Hughes notifications at github.com
Tue Feb 2 19:48:39 UTC 2021


This requires converting all use of the /auth endpoints to use the POST method as GET is no longer supported.

Unfortunately I had to change all the `follow_redirect!` calls in the tests as in rails 6.0 that always uses GET regardless of the response status - in rails 6.1 that has actually been fixed but we can't move to that yet.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Update to Omniauth 2.x

-- File Changes --

    M .rubocop_todo.yml (2)
    M Gemfile (3)
    M Gemfile.lock (17)
    M app/controllers/users_controller.rb (2)
    M app/helpers/user_helper.rb (1)
    M config/routes.rb (2)
    M test/helpers/user_helper_test.rb (4)
    M test/integration/user_creation_test.rb (36)
    M test/integration/user_login_test.rb (50)

-- Patch Links --

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

-- 
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/3078
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20210202/2b144d33/attachment.htm>


More information about the rails-dev mailing list