You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Apr 29, 2022. It is now read-only.
PRE doesn’t use a session store yet so Sebastien think that’s the issue. Sessions are done with https://github.com/expressjs/session in. Rapid uses Redis for the session store.
Very true that the default session store will leak memory, and that should be solved.
I doubt that it's the sole reason for the memory accumulation which is quite rapid: sessions are tiny and the number of users is not enough to account for it. I'd also argue that given the scale of the project, for PREreview at least, Redis is overkill. Just a simple file, leveldb or sqlite would be fine, are trivial to implement, cost nothing to run, and use a tiny fraction of the resources.
There is a bug in the code that makes it so that sometimes when users go on it crashes.
The text was updated successfully, but these errors were encountered: