Skip to content

Prevent NPE in AuthenticationManager.backchannelLogout (#23306) - #23307

Merged
ahus1 merged 1 commit into
keycloak:mainfrom
thomasdarimont:issue/GH-23306-Avoid-NPE-in-AuthenticationManager_backchannelLogout
Sep 18, 2023
Merged

Prevent NPE in AuthenticationManager.backchannelLogout (#23306)#23307
ahus1 merged 1 commit into
keycloak:mainfrom
thomasdarimont:issue/GH-23306-Avoid-NPE-in-AuthenticationManager_backchannelLogout

Conversation

@thomasdarimont

Copy link
Copy Markdown
Contributor

Previously, if the user was already removed from the userSession and the log level was set to DEBUG, then an NPE was triggered by the debug log statement during backchannelLogout.

Fixes #23306

Previously, if the user was already removed from the userSession
and the log level was set to DEBUG, then an NPE was triggered by
the debug log statement during backchannelLogout.

Fixes keycloak#23306
@thomasdarimont
thomasdarimont requested a review from a team as a code owner September 17, 2023 20:17

@ahus1 ahus1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you for fixing this!

@ahus1
ahus1 merged commit 04d16ed into keycloak:main Sep 18, 2023
@thomasdarimont

Copy link
Copy Markdown
Contributor Author

@ahus1 thanks for taking a look! I send #23313 for release/22.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

NPE in AuthenticationManager backchannelLogout

2 participants