session icon indicating copy to clipboard operation
session copied to clipboard

default session() to cookieSession() when a store: is not given

Open jonathanong opened this issue 11 years ago • 5 comments

migrating from https://github.com/senchalabs/connect/issues/516

jonathanong avatar Feb 16 '14 01:02 jonathanong

btw we should just always warn something like memory store is currently being used and is not suitable for production usage. either use a db store or use "cookie-session".

side note: this plugin system is pretty silly. we should be able to just do app.use(require('redis-session')()) and have it work.

jonathanong avatar May 29 '14 22:05 jonathanong

side note: this plugin system is pretty silly.

lol, yes

dougwilson avatar May 29 '14 22:05 dougwilson

@jonathanong May I ask why the memory store is bad?

ghost avatar Aug 27 '14 19:08 ghost

@CelestialWalrus restart your app? all sessions are gone. app accidentally crashes? all session are gone. need to scale your app with cluster? no, your users will randomly be logged out all the time.

dougwilson avatar Aug 27 '14 19:08 dougwilson

@dougwilson Ah, thank you.

ghost avatar Aug 27 '14 19:08 ghost