Skip to content

Skip unknown kty in JWKSet import per RFC 7517 §5 - #388

Open
rjeffman wants to merge 1 commit into
latchset:mainfrom
rjeffman:gh_issue_385
Open

Skip unknown kty in JWKSet import per RFC 7517 §5#388
rjeffman wants to merge 1 commit into
latchset:mainfrom
rjeffman:gh_issue_385

Conversation

@rjeffman

@rjeffman rjeffman commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator

RFC 7517 §5 states that implementations SHOULD ignore JWKs with unrecognized kty values. Previously, a single unknown kty would abort the entire keyset import, breaking consumers when providers add post-quantum or other novel key types to their JWKS endpoints.

Fixes: #385

Assisted-by: Claude Opus 4.6 (1M context) noreply@anthropic.com

RFC 7517 §5 states that implementations SHOULD ignore JWKs with
unrecognized kty values. Previously, a single unknown kty would
abort the entire keyset import, breaking consumers when providers
add post-quantum or other novel key types to their JWKS endpoints.

Fixes: latchset#385

Assisted-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Rafael Guterres Jeffman <rjeffman@redhat.com>
@rjeffman
rjeffman requested a review from simo5 August 6, 2026 17:43

@simo5 simo5 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@eirnym

eirnym commented Aug 6, 2026

Copy link
Copy Markdown

could you please restart checks?

@rjeffman

rjeffman commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator Author

@eirnym all checks passed. Do you agree on merging the changes?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

JWKSet.import_keyset raises on an unknown kty, failing the whole set (RFC 7517 §5)

3 participants