Skip to content
This repository was archived by the owner on Aug 26, 2026. It is now read-only.

Parse pushed request URI from last delimiter - #2347

Closed
goutamadwant wants to merge 1 commit into
spring-attic:mainfrom
goutamadwant:GH-2346
Closed

Parse pushed request URI from last delimiter#2347
goutamadwant wants to merge 1 commit into
spring-attic:mainfrom
goutamadwant:GH-2346

Conversation

@goutamadwant

Copy link
Copy Markdown

Fixes gh-2346

This updates pushed authorization request URI parsing so the expiration timestamp is read after the last delimiter. The generated state uses Base64 URL characters, so it can contain underscores and may include the same delimiter sequence used before the timestamp.

Tests:

  • ./gradlew :spring-security-oauth2-authorization-server:test --tests OAuth2PushedAuthorizationRequestUriTests --tests OAuth2AuthorizationCodeRequestAuthenticationProviderTests --tests OAuth2PushedAuthorizationRequestAuthenticationProviderTests

Fixes spring-atticgh-2346

Signed-off-by: goutamadwant <workwithgoutam@gmail.com>
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jun 18, 2026
@jgrandja

Copy link
Copy Markdown
Contributor

Closing as per comment

@jgrandja jgrandja closed this Aug 18, 2026
@jgrandja jgrandja self-assigned this Aug 18, 2026
@jgrandja jgrandja added status: declined A suggestion or change that we don't feel we should currently apply and removed status: waiting-for-triage An issue we've not yet triaged labels Aug 18, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

status: declined A suggestion or change that we don't feel we should currently apply

Development

Successfully merging this pull request may close these issues.

OAuth2PushedAuthorizationRequestUri generates URIs which cannot be parsed due to ___ in the state part

4 participants