Adds a endpoint to expose only the JWKS (public key) #171

netoisc · 2020-07-02T19:36:46Z

Remove the public_jwks field from the config endpoint (LTI13ConfigHandler) and create another endpoint to expose only the jwks data.
Closes #169

…nly the json config

jgwerner

LGTM

netoisc added 5 commits July 2, 2020 09:32

Add endpoint to expose public jwks and rename the existing to serve o…

6b2f2fb

…nly the json config

minor change to fix a debugging log.

0cc7a11

fix unit tests. Separate tests rleated to jwks handler to into own file

4b31e29

merge with upstream main branch

1b44917

Flake8 formatting

6433659

netoisc requested a review from jgwerner July 2, 2020 19:36

jgwerner assigned netoisc Jul 3, 2020

jgwerner added refactor Move code around to keep things DRY feature New feature or request and removed refactor Move code around to keep things DRY labels Jul 3, 2020

jgwerner approved these changes Jul 3, 2020

View reviewed changes

jgwerner added the lti13 LTI 1.3 authenticator and handlers label Jul 3, 2020

jgwerner merged commit e2dbca7 into IllumiDesk:main Jul 3, 2020

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Adds a endpoint to expose only the JWKS (public key) #171

Adds a endpoint to expose only the JWKS (public key) #171

netoisc commented Jul 2, 2020

jgwerner left a comment

Adds a endpoint to expose only the JWKS (public key) #171

Adds a endpoint to expose only the JWKS (public key) #171

Conversation

netoisc commented Jul 2, 2020

jgwerner left a comment

Choose a reason for hiding this comment