Skip to content
This repository was archived by the owner on Feb 11, 2025. It is now read-only.

Add properties to TokenIntrospectionResponse for all optional claims defined in RFC 7662 (OAuth 2.0 Token Introspection) #577

Closed

Conversation

0xced
Copy link
Contributor

@0xced 0xced commented May 29, 2024

As described by RFC 7662 - OAuth 2.0 Token Introspection

Note

This pull request is stacked on top of #576

@0xced 0xced force-pushed the TokenIntrospectionResponseAdditions branch from 3d72d7f to b6315d7 Compare October 2, 2024 18:57
@damianh
Copy link
Contributor

damianh commented Oct 4, 2024

#576 has been merged. Could you kindly rebase this PR? Thx.

@damianh
Copy link
Contributor

damianh commented Oct 4, 2024

@josephdecock any chance you can review this PR please?

We'll look to have it in by early next week (pending a rebase).

@0xced 0xced force-pushed the TokenIntrospectionResponseAdditions branch from b6315d7 to 70b64f9 Compare October 4, 2024 16:37
@0xced
Copy link
Contributor Author

0xced commented Oct 4, 2024

It's rebased. Thanks for merging my other pull requets!

Copy link
Contributor

@josephdecock josephdecock left a comment

Choose a reason for hiding this comment

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

I think it's probably a good idea to model the optional claims from the spec like this, but I think if we're going to do that, we should do all of the claims listed here.

@0xced
Copy link
Contributor Author

0xced commented Oct 9, 2024

Good point. Should I implement the additional properties as part of this pull request or should it be done in a followup pull request?

@0xced 0xced force-pushed the TokenIntrospectionResponseAdditions branch from 70b64f9 to 738ceff Compare October 9, 2024 11:52
@0xced 0xced changed the title Add Expiration, IssuedAt and NotBefore to TokenIntrospectionResponse Add properties to TokenIntrospectionResponse for all optional claims defined in RFC 7662 (OAuth 2.0 Token Introspection) Oct 9, 2024
@0xced
Copy link
Contributor Author

0xced commented Oct 9, 2024

I went ahead and implemented properties for all optional claims defined in RFC 7662.

@0xced 0xced force-pushed the TokenIntrospectionResponseAdditions branch 4 times, most recently from 3822fe2 to 7f53cf1 Compare October 9, 2024 12:00
@damianh
Copy link
Contributor

damianh commented Feb 11, 2025

This PR has been migrated to DuendeSoftware/foss#98

@damianh damianh closed this Feb 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants