Brightspace / D2L.Security.OAuth2

Brightspace OAuth 2.0 for C#
Apache License 2.0
7 stars 16 forks source link

Remove fallback to jwks when fetching jwk #110

Closed j3parker closed 5 years ago

j3parker commented 5 years ago
j3parker commented 5 years ago

Verified okay for existed auth service clients?

Yep, a bit ago, but I'll revalidate today.

Also, we'll need to make a branch and cherry-pick this over, master isn't shippable (it is theoretically, but not practically)

D'oh, ok. Do you know what commit we should branch at?

omsmith commented 5 years ago

Likely candidates:

j3parker commented 5 years ago

I'll go with a85271c to avoid possible merge conflicts

omsmith commented 5 years ago

Made a branch 7.x with https://github.com/Brightspace/D2L.Security.OAuth2/commit/58d78092d914ac5f76affde9b364825054f263b8 cherry-picked. These apply cleanly on top.

j3parker commented 5 years ago

I can cherry-pick this tomorrow and we'll figure out how exactly we want to do this.