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.

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

VALID does not precede J/CWT processing rules #81

Merged
merged 2 commits into from
Nov 16, 2023

Conversation

panva
Copy link
Member

@panva panva commented Nov 8, 2023

This feedback comes from applying the Status List to an implementation of a general OAuth 2.x server issuing JWT Access Tokens.

Such implementation will typically not keep record of any issued JWT Access Tokens (their claims, expiry, etc) until they need to be revoked due any reason, e.g. getting leaked or just being explicitly revoked by the client.

In such cases the issuer considers the token as VALID when issuing the Status List but this state will not necessarily reflect the token's validity as per the JWT validation rules for claims like nbf or exp. This is important to point out so that the status list is not mistaken for an alternative to actually validating the referenced token's claim set.

Copy link
Contributor

@paulbastian paulbastian left a comment

Choose a reason for hiding this comment

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

Looks good to me. It might be better positioned in a dedicated Verification Processing section once we have that but its good for now

@paulbastian
Copy link
Contributor

@panva could you merge with main and add a line to the document history please

@panva
Copy link
Member Author

panva commented Nov 16, 2023

Not a member, cannot merge.

@paulbastian
Copy link
Contributor

paulbastian commented Nov 16, 2023

I meant sync, sry. But I can add the history on main also.

@paulbastian
Copy link
Contributor

@tplooker Please review and merge if you approve

@tplooker tplooker merged commit 5e89ba2 into oauth-wg:main Nov 16, 2023
1 check passed
@panva panva deleted the note-on-valid branch November 16, 2023 20:10
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.

4 participants