Skip to content

Comments

feat: invalid id token field instead embedded into access token#652

Merged
NathaelB merged 2 commits intomainfrom
491-invalid-id-token-field-instead-embedded-into-access-token
Jan 6, 2026
Merged

feat: invalid id token field instead embedded into access token#652
NathaelB merged 2 commits intomainfrom
491-invalid-id-token-field-instead-embedded-into-access-token

Conversation

@NathaelB
Copy link
Member

@NathaelB NathaelB commented Jan 6, 2026

No description provided.

Add a new `IdTokenClaims` struct and a `TokenClaims` trait to abstract
over different token claim types. Modify `create_jwt` to optionally
generate an ID token if the `openid` scope is present. Update `JwtToken`
to accommodate an optional ID token.
@NathaelB NathaelB requested a review from LeadcodeDev January 6, 2026 08:53
@NathaelB NathaelB self-assigned this Jan 6, 2026
@NathaelB NathaelB linked an issue Jan 6, 2026 that may be closed by this pull request
@NathaelB NathaelB merged commit 43103dc into main Jan 6, 2026
8 checks passed
@NathaelB NathaelB deleted the 491-invalid-id-token-field-instead-embedded-into-access-token branch January 6, 2026 08:58
NathaelB added a commit that referenced this pull request Jan 16, 2026
* feat: refactor token generation and support id_token

Add a new `IdTokenClaims` struct and a `TokenClaims` trait to abstract
over different token claim types. Modify `create_jwt` to optionally
generate an ID token if the `openid` scope is present. Update `JwtToken`
to accommodate an optional ID token.

* fix: refactor token creation to include ID token
NathaelB added a commit that referenced this pull request Jan 16, 2026
* feat: add ThemeProvider and user theme menu (#651)

* feat: invalid id token field instead embedded into access token (#652)

* feat: refactor token generation and support id_token

Add a new `IdTokenClaims` struct and a `TokenClaims` trait to abstract
over different token claim types. Modify `create_jwt` to optionally
generate an ID token if the `openid` scope is present. Update `JwtToken`
to accommodate an optional ID token.

* fix: refactor token creation to include ID token

* fix: return service account not found exception when service account not found (#611) (#631)

* fix: only validate client secret for non-public clients (#657)

* Fix: only validate client secret for non-public clients

* Fix: Simplify client secret validation logic

---------

Co-authored-by: Luis Daniel RUBIERA <[email protected]>
Co-authored-by: Joris VILARDELL <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Invalid ID token field (instead embedded into access token)

2 participants