Skip to content

HazelcastSessionRepository generates invalid session events #1077

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
Closed
vpavic opened this issue May 15, 2018 · 0 comments
Closed

HazelcastSessionRepository generates invalid session events #1077

vpavic opened this issue May 15, 2018 · 0 comments
Assignees
@vpavic
Labels
in: hazelcast type: bug A general bug
Milestone
2.0.4

Comments

@vpavic
Copy link
Contributor

vpavic commented May 15, 2018 •
edited
Loading

Since HazelcastSessionRepository uses IMap#remove and IMap#set while handling change session id, this scenario will trigger EntryRemovedListener#entryRemoved and EntryAddedListener#entryAdded therefore generating invalid session destroyed and session created events.
@vpavic vpavic self-assigned this May 15, 2018
@vpavic vpavic changed the title HazelcastSessionRepository generates invalid session destroyed events HazelcastSessionRepository generates invalid session events May 15, 2018
@vpavic vpavic added this to the 2.0.4 milestone May 31, 2018
@vpavic vpavic closed this as completed in 8dac35c Jun 13, 2018
@vpavic vpavic mentioned this issue Jun 22, 2018
Closed
jtodt added a commit to jtodt/spring-session that referenced this issue Jan 16, 2019
@jtodt
backport spring-projectsgh-1077 to 1.3.x. Fix NullPointerException in…
1cf9db0 
… entryRemoved.

detected with hazelcast 3.11, spring session 1.3.4, spring boot 1.5
@jtodt jtodt mentioned this issue Jan 16, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vpavic vpavic

Labels
in: hazelcast type: bug A general bug
Projects
None yet
Milestone
2.0.4
Development

No branches or pull requests
1 participant
@vpavic