Skip to content
This repository has been archived by the owner on Nov 9, 2022. It is now read-only.

Move away from MemoryStore session store #201

Open
ryanjdew opened this issue May 15, 2015 · 4 comments
Open

Move away from MemoryStore session store #201

ryanjdew opened this issue May 15, 2015 · 4 comments

Comments

@ryanjdew
Copy link
Contributor

MemoryStore isn't meant for production use. We should look into moving to another session store.

@grtjn
Copy link
Contributor

grtjn commented May 15, 2015

@withjam has a good alternative if I am not mistaken.. :)

@withjam
Copy link

withjam commented May 18, 2015

Yes, I made an implementation of a connect session store that uses MarkLogic to store sessions. https://github.com/withjam/connect-marklogic

@ryanjdew
Copy link
Contributor Author

That is great @withjam. Should we be fine using the same ML REST app as DemoCat or would you recommend creating a separate REST app and database for node.js sessions?

@withjam
Copy link

withjam commented May 18, 2015

Session data is very small and it goes into it's own collection, so I don't think it would need a separate REST app unless you have a compelling reason to do so. The only think to keep in mind is that the middleware itself doesn't take care of clearing expired sessions, so it may be something to monitor or clean up at some point.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants