zerebubuth/openstreetmap-cgimap

Adding OAuth2 support

Closed this issue · 6 comments

@cquest, @frodrigo: I've seen the comments in https://annuel.framapad.org/p/N_IDAQXYLHswlpU2s3oE

The cgimap part is not relevant for your Discourse OAuth2 integration, all you need is in the OSM website issue.

We was asking ourself about the status of the OAuth2 on the last OSM-FR chapter meeting. If the implementation is still going on or has been stoped.
But after looking at openstreetmap/openstreetmap-website@master...tomhughes:doorkeeper I see @tomhughes is still working on it. Don't worry, it's not for pressure, but rather to provide in a way or in an other resources if necessary.

@tomhughes : feature branch https://github.com/zerebubuth/openstreetmap-cgimap/tree/feature/oauth2 now comes with oauth2 support. A cgimap test instance similar to https://github.com/openstreetmap/chef/blob/master/roles/dev.rb#L128-L129 could be set up now.

@tomhughes : I wanted to double check what your plans are to include CGImap in the current oauth2 dev instance test cycle. Moving this code right to production without any testing doesn't seem like a good approach to me.

I've added it to the test instance now.

@cquest, @frodrigo: as a FYI: OAuth2 is in production now, you might want to check, how openstreetmap/openstreetmap-website#1408 (comment) could be useful for your osm-fr discourse instance.