WFLY-4602 Web logout does not fully clear security context #7432

Merged
merged 1 commit into from May 8, 2015

Conversation

Projects
None yet
3 participants
@stuartwdouglas
Contributor

stuartwdouglas commented May 8, 2015

No description provided.

@stuartwdouglas stuartwdouglas added the 9.x label May 8, 2015

- public LogoutNotificationReceiver(AuthenticationManager manager) {
+ public LogoutNotificationReceiver(AuthenticationManager manager, String securityDomain) {

This comment has been minimized.

@sguilhen

sguilhen May 8, 2015

Contributor

you might not need this - iirc you can retrieve the security domain from the current security context before clearing it

@sguilhen

sguilhen May 8, 2015

Contributor

you might not need this - iirc you can retrieve the security domain from the current security context before clearing it

This comment has been minimized.

@stuartwdouglas

stuartwdouglas May 8, 2015

Contributor

I don't think it really matters much either way, the end result should be the same.

@stuartwdouglas

stuartwdouglas May 8, 2015

Contributor

I don't think it really matters much either way, the end result should be the same.

bstansberry added a commit that referenced this pull request May 8, 2015

Merge pull request #7432 from stuartwdouglas/WFLY-4602-9.x
WFLY-4602 Web logout does not fully clear security context

@bstansberry bstansberry merged commit ebad0dd into wildfly:9.x May 8, 2015

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