claudioc / jingo

Node.js based Wiki
MIT License
1.02k stars 185 forks source link

express-session memory leak #230

Open roccomuso opened 6 years ago

roccomuso commented 6 years ago

The default "memory" storage for sessions suffer a well known memory leak. As pointed out on the express-session documentation. The usage is discouraged in production.

Consider switching to a leak-proof in-memory store: memorystore

claudioc commented 6 years ago

Hi @roccomuso

yes I know :/

I will definitely take a look a yours to see if I can implement it easily. Thanks for the suggestion!