-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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.
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
Provide a way to get session size #23
Comments
Here are the bean attributes that Jetty exposes with its nosql session manager. I believe Spring Session should support a similar set. Jetty's nosql session support uses mongodb and not redis, but these values do not seem mongo-centric. The mongo-centric beans are separate. The values above are provided from http://download.eclipse.org/jetty/stable-9/apidocs/org/eclipse/jetty/server/session/AbstractSessionManager.html |
+1 |
1 similar comment
+1 |
I may be overlooking something obvious here, but what exactly is the "size" of a session? |
Question for the project maintainers. Would there be an objection to creating an implementation of SessionRepository that uses a DropWizard Counter and DropWizard Histogram to provide stats similar to what Jetty does? (see my previous Post) I am curious if there is a philosophical problem with adding the dependency on DropWizard. |
@solidjb I don't know a lot about these portions of DropWizzard so it is hard to say definitively. Practically speaking, we are open to adding new dependencies so long as they are part of Spring IO Platform (as we must be Spring IO compliant) and they are optional. |
I am still working on the testing, but the class itself is pretty simple.
What do you think? |
The above code would only have the following dependencies:
I think that meets your criteria. When I am done with the tests, can I create a pull request? |
@solidjb Thanks for the reply. The dependencies look like it will work. However, the implementation looks like it lacks support for handling an expired session. Can you create a new ticket that we can use to discuss this (since this does not appear to address this specific issue). |
Done - #535 |
.. Using a JMX metric ideally. Very useful when you want to make sure the session size doesn't go out of hand when using frameworks like Vaadin or gwt
The text was updated successfully, but these errors were encountered: