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.

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

Incorrect documentation for OIDC Back-Channel Logout #15198

Closed
aelillie opened this issue Jun 4, 2024 · 0 comments
Closed

Incorrect documentation for OIDC Back-Channel Logout #15198

aelillie opened this issue Jun 4, 2024 · 0 comments
Assignees
Labels
in: docs An issue in Documentation or samples type: bug A general bug
Milestone

Comments

@aelillie
Copy link

aelillie commented Jun 4, 2024

Describe the bug
https://docs.spring.io/spring-security/reference/servlet/oauth2/login/logout.html states that you need a way listen to events published by Spring Security to remove old OidcSessionInformation entries, like so:

@Bean
public HttpSessionEventListener sessionEventListener() {
    return new HttpSessionEventListener();
}

However HttpSessionEventListener.java does not exist. It took me a while to figure out that what you need instead is an HttpSessionEventPublisher like so:

@Bean
public HttpSessionEventPublisher httpSessionEventPublisher() {
    return new HttpSessionEventPublisher();
}

Please update the documentation :) Thank you.

@aelillie aelillie added status: waiting-for-triage An issue we've not yet triaged type: bug A general bug labels Jun 4, 2024
@jzheaux jzheaux added in: docs An issue in Documentation or samples and removed status: waiting-for-triage An issue we've not yet triaged labels Jun 6, 2024
@jzheaux jzheaux self-assigned this Jun 6, 2024
@jzheaux jzheaux added this to the 6.2.5 milestone Jun 6, 2024
@jzheaux jzheaux closed this as completed in 0532659 Jun 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: docs An issue in Documentation or samples type: bug A general bug
Projects
None yet
Development

No branches or pull requests

2 participants