puiterwijk / regcfp

Registration and CFP system
MIT License
13 stars 15 forks source link

Drop OpenID 2.0 and make OpenIDC return url provider-specific #203

Closed puiterwijk closed 7 years ago

puiterwijk commented 7 years ago

Signed-off-by: Patrick Uiterwijk puiterwijk@redhat.com

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 91.751% when pulling cf5a2dd2b781007b2253117dbb5b6374c3238cc1 on noopenid2 into 3ccec2c0eb0452864d1728e9de98b6e4b84ac437 on develop.

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 91.751% when pulling 0519847deeefb662fffc5738fc432c88ddf53f16 on noopenid2 into 3ccec2c0eb0452864d1728e9de98b6e4b84ac437 on develop.

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 91.751% when pulling 8733a5cb964fbdb15b9ad1660483932c87ec5503 on noopenid2 into 3ccec2c0eb0452864d1728e9de98b6e4b84ac437 on develop.

ssssam commented 7 years ago

This would mean we can't use the master branch for http://registration.guadec.org/ any more. The only OpenIDC connect provider we have set up is Google and I am definitely not going to require everyone to have a Google account.

We could make a branch for registration.guadec.org though.

puiterwijk commented 7 years ago

@ssssam Both GNOME and Fedora are OpenID Connect providers as well, so you could use that.

puiterwijk commented 7 years ago

e.g.: https://id.fedoraproject.org/openidc/.well-known/openid-configuration

ssssam commented 7 years ago

Good point! I forgot that id.gnome.org had been upgraded. OK, fine by me then.

puiterwijk commented 7 years ago

Thanks. I'll make sure to set up OpenIDC for GNOME and Fedora before we push this to master.