[openstreetmap/openstreetmap-website] Add MAPS.ME authentication (#1433)
notifications at github.com
Thu Feb 9 14:05:25 UTC 2017
We, as in maps.me, have published the first beta version of our own authentication service, passport.maps.me (does not have the front page yet). The plan is to use it instead of osm.org oauth to authenticate users, and then register them on osm.org using that account.
It would give us an independent profile storage and, more important, an option to use native SDKs for facebook and google on mobile devices. I could not negotiate that for osm.org (re: #1114 and mailing lists), so we decided to make an alternative service. It might be possible for other mobile apps to use the same service. That would be a benefit for other authors of OSM mobile editors, simplifying their sign-up workflow.
Until it is possible to link multiple social accounts to a profile (see #1274), we plan to use this only for new users, directing existing users to osm.org for all kinds of linked accounts, including facebook and google. Nothing is changed for the osm side, besides a new icon on the login page.
This pull requests adds two auth providers: `mapsme` for a regular web-based authentication via the passport site, and `mapsme_token` for a token-based authentication, which would allow skipping the passport site when logging in on a mobile device. The latter rewrites the provider string to `mapsme`, so there is no difference to the database which one of these is used.
You can view, comment on, or merge this pull request online at:
-- Commit Summary --
* Add MAPS.ME authentication
-- File Changes --
M Gemfile (1)
M Gemfile.lock (5)
A app/assets/images/mapsme.png (0)
M app/controllers/user_controller.rb (2)
M app/views/user/login.html.erb (3)
M config/example.application.yml (4)
M config/initializers/omniauth.rb (4)
M config/locales/en.yml (3)
M lib/auth.rb (1)
M test/controllers/user_controller_test.rb (2)
M test/fixtures/users.yml (14)
M test/integration/user_creation_test.rb (114)
M test/integration/user_login_test.rb (90)
-- Patch Links --
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the rails-dev