openid-connect-generic icon indicating copy to clipboard operation
openid-connect-generic copied to clipboard

Use discovery

Open spikyjt opened this issue 5 years ago • 3 comments

Acquire (and cache) the provider's configuration information using discovery https://openid.net/specs/openid-connect-discovery-1_0.html#ProviderConfig

This will keep things working if anything changes at the OP end and means less config for the plugin user.

It could be optional of course, in case some OPs don't support it (they should).

spikyjt avatar Oct 07 '19 16:10 spikyjt

I like this idea a lot, thanks for brining this up.

Would you like to attempt a PR, or would you prefer someone else do it?

daggerhart avatar Oct 24 '19 10:10 daggerhart

@daggerhart I think #88 needs to come first, as there is no point implementing this if you would then rip out the underlying library and replace it. Ideally the requirements for #88 would be that the lib implements discovery. If it doesn't and we need to extend it, then I'd be happy to look at doing a PR for this.

spikyjt avatar Oct 30 '19 12:10 spikyjt

It's kinda similar to #71 which basically is the same idea but not as precisely articulated. I think this would be an important addition as it makes setup a lot easier and faster

Kiina avatar Jun 07 '20 03:06 Kiina