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.

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

session_check_interval is in milliseconds instead of seconds #41

Closed
dop251 opened this issue Apr 23, 2012 · 0 comments
Closed

session_check_interval is in milliseconds instead of seconds #41

dop251 opened this issue Apr 23, 2012 · 0 comments

Comments

@dop251
Copy link

dop251 commented Apr 23, 2012

Hi,

I've noticed that the session check interval is taken directly from the settings (where it's said to be in seconds and default value is 15) and supplied to the PeriodicCallback constructor. However the constructor expects milliseconds, rather than seconds. It should be fixed by multiplying check_interval by 1000 in the following line:

self._sessions_cleanup = ioloop.PeriodicCallback(self._sessions.expire,
check_interval,
self.io_loop)

Dmitry

@mrjoes mrjoes closed this as completed in 550653b Apr 23, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants