TransforMap / data.transformap.co

Here you find a Node.js daemon to serve the public TransforMap web service.
https://data.transformap.co
GNU Affero General Public License v3.0
7 stars 2 forks source link

425 oauth2 provider #48

Closed jum-s closed 6 years ago

jum-s commented 7 years ago

This PR extends the data.transformap in the following manner:

Questions in the air

so far only one provider is implemented (gitlab), the auth endpoint implement only one strategy. The login button is unique, so shall we chain strategies in something like passport.authenticate(['gitlab', 'discourse', 'blabla'])

almereyda commented 7 years ago

Additionally I have found a commit from a stale branch in this repository, which I wonder about. https://github.com/TransforMap/data.transformap.co/commit/5fb65ad7dbf3214fc75a9fb88fad24af5d031c82 From looking at the diff briefly and comparing it with what we find here, I guess it is from another attempt to experiment with the OAuth interface, which didn't succeed.

jum-s commented 7 years ago

about https://github.com/TransforMap/data.transformap.co/commit/5fb65ad7dbf3214fc75a9fb88fad24af5d031c82 I based my work on what auth branch could bring to 425-oauth2-provider branch. In order to signify this I intentionnaly cherry-pick some already done work. But from your commen Im not sure it brings more clarity than confusion