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

Http sessions are not removed when expired #4442

Closed
srs opened this issue Feb 20, 2017 · 3 comments
Closed

Http sessions are not removed when expired #4442

srs opened this issue Feb 20, 2017 · 3 comments
Assignees
@srs
Labels
Bug
Milestone
6.9.3

Comments

@srs
Copy link
Contributor

srs commented Feb 20, 2017 •
edited

Http sessions are not removed when expired in Jetty. Implement a thread that will remove unused sessions from the map.

Best solution is to upgrade to Jetty 9.4.x, but this will probably take more time.
@srs srs added the Bug label Feb 20, 2017
@srs srs self-assigned this Feb 20, 2017
@srs
Copy link
Contributor Author

srs commented Feb 21, 2017

Seems that they are removed after all. But the timeout behaves strangely.

srs pushed a commit that referenced this issue Feb 21, 2017
Http sessions are not removed when expired (#4442).
14a2a8a
srs pushed a commit that referenced this issue Feb 21, 2017
Http sessions are not removed when expired (#4442).
dcc60a6
@srs
Copy link
Contributor Author

srs commented Feb 21, 2017

Problem seemed to be two things:

  • Timeout setting for session timeout was wrong (in sec, not ms).
  • Felix session wrapper does strange things and invalidates too soon.

Both fixed.

@srs
Copy link
Contributor Author

srs commented Feb 21, 2017

This can also be related to other problems regarding session timeout.

@srs srs closed this as completed Feb 28, 2017
@jsi jsi added this to the 6.9.3 milestone Feb 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@srs srs

Labels
Bug
Projects
None yet
Milestone
6.9.3
Development

No branches or pull requests
2 participants
@srs @jsi