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

HttpSession lastAccessedTime and isNew not getting updated #12070

Closed
Horcrux7 opened this issue Jul 22, 2024 · 2 comments · Fixed by #12078
Closed

HttpSession lastAccessedTime and isNew not getting updated #12070

Horcrux7 opened this issue Jul 22, 2024 · 2 comments · Fixed by #12078
Assignees
@janbartel
Labels
Bug For general bugs on Jetty side

Comments

@Horcrux7
Copy link

Jetty version(s)
12.0.11

Jetty Environment
ee8

Java version/vendor (use: java -version)
openjdk version "21" 2023-09-19
OpenJDK Runtime Environment (build 21+35-2513)
OpenJDK 64-Bit Server VM (build 21+35-2513, mixed mode, sharing)

OS type/version
Windows 10 and Windows 11

Description
In our environment the value of the HttpSession.getLastAccessedTime() is always identical to HttpSession.getCreationTime(). Also the value of HttpSession.isNew() returns always true.

The getId() and the hashCode() does not change between different HTTP request. This means the session cookie was send and received correctly. It was not a new session.

The requests are HTTP1/1 POST requests.

Attached is JettyServer Dump:
jettydump.txt
@Horcrux7 Horcrux7 added the Bug For general bugs on Jetty side label Jul 22, 2024
@janbartel
Copy link
Contributor

Looking into this.

janbartel added a commit that referenced this issue Jul 23, 2024
@janbartel
Issue #12070 fix session lastAccessedTime and isNew
e20762d
janbartel added a commit that referenced this issue Jul 24, 2024
@janbartel
Issue #12070 fix session lastAccessedTime and isNew (#12078)
5c6e27f
@joakime joakime linked a pull request Jul 24, 2024 that will close this issue
Issue #12070 fix session lastAccessedTime and isNew #12078
Merged
@janbartel
Copy link
Contributor

Fixed via #12078

@joakime joakime changed the title lastAccessedTime and isNew not getting updated HttpSession lastAccessedTime and isNew not getting updated Jul 30, 2024
@joakime joakime mentioned this issue Aug 1, 2024
Closed
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
No open projects
🧊 Jetty 12.0.12 - FROZEN
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issue #12070 fix session lastAccessedTime and isNew jetty/jetty.project
2 participants
@Horcrux7 @janbartel