Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

JWKS support #312

Merged
merged 27 commits into from
Sep 30, 2021
Merged

JWKS support #312

merged 27 commits into from
Sep 30, 2021

Conversation

afflerbach
Copy link
Contributor

No description provided.

@afflerbach afflerbach marked this pull request as draft September 9, 2021 12:20
@afflerbach afflerbach changed the title WIP: JWKS support JWKS support Sep 9, 2021
@afflerbach afflerbach force-pushed the jwks_uri branch 3 times, most recently from c35b21a to 0780b49 Compare September 16, 2021 07:55
@afflerbach afflerbach marked this pull request as ready for review September 22, 2021 06:53
@malud malud requested a review from johakoch September 23, 2021 07:29
johakoch
johakoch reviewed Sep 23, 2021
View reviewed changes
accesscontrol/jwks.go Show resolved Hide resolved
config/ac_jwt.go Show resolved Hide resolved
@afflerbach afflerbach removed the request for review from malud September 24, 2021 07:02
johakoch
johakoch reviewed Sep 24, 2021
View reviewed changes
accesscontrol/jwt.go Outdated Show resolved Hide resolved
johakoch
johakoch requested changes Sep 24, 2021
View reviewed changes
accesscontrol/jwk.go Outdated Show resolved Hide resolved
accesscontrol/jwk.go Show resolved Hide resolved
accesscontrol/jwks.go Outdated Show resolved Hide resolved
accesscontrol/jwt.go Outdated Show resolved Hide resolved
accesscontrol/jwt.go Outdated Show resolved Hide resolved
@afflerbach afflerbach self-assigned this Sep 28, 2021
afflerbach and others added 17 commits September 29, 2021 09:33
@afflerbach
New jwt options jwks_uri/jwks_ttl for reading JWK sets
93bce95
@afflerbach
Simple HTTP request
4a4adea
@afflerbach
Add JWT backend for use with JWKS
d7e64f6
@afflerbach
Move tests
8b10d4f
@afflerbach
Check internal error messages properly.
d080985
@afflerbach
Load JWKS only when expired (configured via jwks_ttl)
7728b95
@afflerbach
Configure JWKS requests with inline backend
f227e11
@afflerbach
Wrap simple JWKS reuqest without backend in a backend to allow for …
560b250 
…logging etc.
@afflerbach
Resolve backend reference for JWKS within jwt blocks
b742d7a
@afflerbach
Changelog
5790c2d
@afflerbach
test for JWK with p and e
fb18f34
@afflerbach
jwks_uri renamed jwks_url, JWKsURI renamed JWKsURL
c16dbaa
@afflerbach
fixed changelog entry
cbe6000
@afflerbach
signature_algorithm is no longer required; added documentation on whe…
3bf0bf4 
…n to use jwks_url or signature_algorithm with key/key_file
@afflerbach
set request URL in context as request.URLAttribute; origin in URL mus…
8878659 
…t match backend's origin if given
@afflerbach
backend creation similar to other blocks with backend
880fc42
@afflerbach
changed error message
bae2b0c
@afflerbach afflerbach force-pushed the jwks_uri branch 2 times, most recently from 06baeff to e4b23cf Compare September 29, 2021 08:33
johakoch
johakoch previously approved these changes Sep 29, 2021
View reviewed changes
@johakoch johakoch merged commit f695c38 into master Sep 30, 2021
@afflerbach afflerbach deleted the jwks_uri branch October 20, 2021 06:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@johakoch johakoch johakoch approved these changes

Assignees

@afflerbach afflerbach

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@afflerbach @johakoch