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.

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

MemcachedSessionDataMap needs to set the context classloader before serialization/deseriazliation. #6556

Closed
janbartel opened this issue Jul 28, 2021 · 0 comments · Fixed by #6557
Closed

MemcachedSessionDataMap needs to set the context classloader before serialization/deseriazliation. #6556

janbartel opened this issue Jul 28, 2021 · 0 comments · Fixed by #6557
Assignees
@janbartel
Labels
Bug For general bugs on Jetty side

Comments

@janbartel
Copy link
Contributor

Because the MemcachedSessionDataMap does not use ContextHandler.handle(Runnable), the context classloader is not set correctly when the SessionData is serialized into memcached.
@janbartel janbartel added the Bug For general bugs on Jetty side label Jul 28, 2021
@janbartel janbartel self-assigned this Jul 28, 2021
janbartel added a commit that referenced this issue Jul 28, 2021
@janbartel
Issue #6556 Ensure context classloader set when operating on memcache.
a80a379 
Signed-off-by: Jan Bartel <janb@webtide.com>
@janbartel janbartel mentioned this issue Jul 28, 2021
Merged
janbartel added a commit that referenced this issue Jul 29, 2021
@janbartel
Issue #6556 Fix arg
9679b0c 
Signed-off-by: Jan Bartel <janb@webtide.com>
janbartel added a commit that referenced this issue Jul 29, 2021
@janbartel
Issue #6556 Use FuturePromise<SessionData>
d438a1b 
Signed-off-by: Jan Bartel <janb@webtide.com>
janbartel added a commit that referenced this issue Jul 29, 2021
@janbartel
Issue #6556 Also use FuturePromise<Void>
627a34a 
Signed-off-by: Jan Bartel <janb@webtide.com>
janbartel added a commit that referenced this issue Jul 30, 2021
@janbartel
Issue #6556 change after review
ef40ad2 
Signed-off-by: Jan Bartel <janb@webtide.com>
janbartel added a commit that referenced this issue Aug 2, 2021
@janbartel
Issue #6556 Ensure context classloader set when operating on memcache. (
90a72b0 
#6557)

* Issue #6556 Ensure context classloader set when operating on memcache.

Signed-off-by: Jan Bartel <janb@webtide.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@janbartel janbartel

Labels
Bug For general bugs on Jetty side
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issue #6556 Ensure context classloader set when operating on memcache. jetty/jetty.project
1 participant
@janbartel