[openstreetmap/openstreetmap-website] Collapse duplicate OAuth authorisations (#1462)

Tom Hughes notifications at github.com
Thu Feb 23 12:14:47 UTC 2017


Only show each application once, even if it has multiple tokens, and revoke all the relevant tokens when asked.

Alternative (though very similar) implementation to #1461 addressing some of the issues raised in #1455.
You can view, comment on, or merge this pull request online at:

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

-- Commit Summary --

  * Collapse duplicate OAuth authorisations

-- File Changes --

    M app/controllers/oauth_controller.rb (21)
    M app/views/oauth_clients/index.html.erb (60)
    M config/locales/en.yml (2)
    M test/controllers/oauth_clients_controller_test.rb (11)
    M test/controllers/oauth_controller_test.rb (54)
    A test/factories/oauth_tokens.rb (6)

-- Patch Links --

https://github.com/openstreetmap/openstreetmap-website/pull/1462.patch
https://github.com/openstreetmap/openstreetmap-website/pull/1462.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/1462
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20170223/b8b3edad/attachment.html>


More information about the rails-dev mailing list