Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

SessionAuthentication._session is not marked as transient #12309

Closed
janbartel opened this issue Sep 25, 2024 · 0 comments · Fixed by #12310
Closed

SessionAuthentication._session is not marked as transient #12309

janbartel opened this issue Sep 25, 2024 · 0 comments · Fixed by #12310
Assignees
@janbartel
Labels
Bug For general bugs on Jetty side

Comments

@janbartel
Copy link
Contributor

Jetty version(s)
jetty-12.0.x

Jetty Environment
core security

The org.eclipse.jetty.security.authentication.SessionAuthentication class has a HttpSession _session data member which must be marked as transient to prevent it being serialized. This keyword is missing.
@janbartel janbartel added the Bug For general bugs on Jetty side label Sep 25, 2024
@janbartel janbartel self-assigned this Sep 25, 2024
janbartel added a commit that referenced this issue Sep 25, 2024
@janbartel
Issue #12309 replaced transient keyword for SessionAuthentication.ses…
64bf7b8 
…sion
This was referenced Sep 25, 2024
Merged
Merged
janbartel added a commit that referenced this issue Sep 25, 2024
@janbartel
Issue #12309 replaced transient keyword for SessionAuthentication.ses…
04fd45d 
…sion (#12310)
@github-project-automation github-project-automation bot moved this to ✅ Done in Jetty 12.0.14 Sep 25, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@janbartel janbartel

Labels
Bug For general bugs on Jetty side
Projects
No open projects
Jetty 12.0.14
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issue #12309 replaced transient keyword for SessionAuthentication.session jetty/jetty.project
1 participant
@janbartel