Skip to content

Remove OAuth2AuthorizationRequest when a distributed session is used #7334

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

Closed
wants to merge 1 commit into from
Closed

Remove OAuth2AuthorizationRequest when a distributed session is used #7334

wants to merge 1 commit into from

Conversation

AndreasKl
Copy link
Contributor

Dirties the WebSession by putting the amended AUTHORIZATION_REQUEST map into
the WebSession even it was already in the map. This causes common SessionRepository
implementations like Redis to persist the updated attribute.

Fixes gh-7327

Author: Andreas Kluth mail@andreaskluth.net

Dirties the WebSession by putting the amended AUTHORIZATION_REQUEST map into
the WebSession even it was already in the map. This causes common SessionRepository
implementations like Redis to persist the updated attribute.

Fixes gh-7327

Author: Andreas Kluth <mail@andreaskluth.net>
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Aug 30, 2019
@jgrandja jgrandja added in: oauth2 An issue in OAuth2 modules (oauth2-core, oauth2-client, oauth2-resource-server, oauth2-jose) type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Sep 5, 2019
@jgrandja jgrandja added this to the 5.2.0.RC1 milestone Sep 5, 2019
@jgrandja jgrandja self-assigned this Sep 5, 2019
@jgrandja
Copy link
Contributor

jgrandja commented Sep 5, 2019

@AndreasKl Thanks for the PR @AndreasKl. This is now in master. FYI, I applied a little polish to the test.

@jgrandja jgrandja closed this Sep 5, 2019
@AndreasKl AndreasKl deleted the gh-7327-removed-auth-requests-are-not-propagated-using-distributed-sessions branch September 5, 2019 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: oauth2 An issue in OAuth2 modules (oauth2-core, oauth2-client, oauth2-resource-server, oauth2-jose) type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OAuth2AuthorizationRequest not removed from session
3 participants