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.

Sign up for GitHub

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

Jump to bottom

redirect_uri resolver is incorrect #1012

Closed
leshalv opened this issue Dec 18, 2022 · 0 comments
Closed

redirect_uri resolver is incorrect #1012

leshalv opened this issue Dec 18, 2022 · 0 comments
Assignees
@leshalv
Labels
type: bug A general bug
Milestone
0.4.1

Comments

@leshalv
Copy link
Contributor

leshalv commented Dec 18, 2022

In the authorization code mode, PKCE verification fails to throw an exception, because the authorizationRequest object is not passed, when there are multiple redirectUris, the redirectUri is obtained incorrectly, resulting in inconsistency between the redirect address and the client delivery address, resulting in a redirect error.

image

image
@leshalv leshalv added the type: bug A general bug label Dec 18, 2022
leshalv pushed a commit to leshalv/spring-authorization-server that referenced this issue Dec 18, 2022
Fix: spring-projects#1012
0ff0f38
This was referenced Dec 18, 2022
Closed
Closed
@jgrandja jgrandja added this to the 0.4.1 milestone Jan 10, 2023
@jgrandja jgrandja changed the title The redirectUri selection is incorrect The redirectUri resolver is incorrect Jan 10, 2023
leshalv added a commit to leshalv/spring-authorization-server that referenced this issue Jan 12, 2023
@leshalv
Fix: spring-projects#1012
f6c2045
leshalv added a commit to leshalv/spring-authorization-server that referenced this issue Jan 17, 2023
@leshalv
Fix: spring-projects#1012
2c3c93a
@jgrandja jgrandja changed the title The redirectUri resolver is incorrect redirect_uri resolver is incorrect Feb 15, 2023
jgrandja added a commit that referenced this issue Feb 15, 2023
@jgrandja
Merge branch '0.4.x'
db5416c 
Fixes gh-1012
@jgrandja jgrandja mentioned this issue Feb 15, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@leshalv leshalv

Labels
type: bug A general bug
Projects
None yet
Milestone
0.4.1
Development

No branches or pull requests
2 participants
@jgrandja @leshalv