Putting back 'get' instead of 'setnx'

roidrage / redis-session-store

A simple session store for Rails based on Redis.

http://github.com/roidrage/redis-session-store

MIT License

366 stars 147 forks source link

Closed jjeb closed 10 years ago

jjeb commented 10 years ago

Putting back 'get' instead of 'setnx' since it was not performing as expected, also changed to use 'setnx' instead of 'setex' when setting

robertosanchez commented 10 years ago

That looks good to me :+1: :shipit:

meatballhat commented 10 years ago

Was this rebased against roidrage/master? It has duplicate commits with what's already in upstream, yet the commit IDs are different.

meatballhat commented 10 years ago

I'd like the reversion to using setnx + expire to be within a multi call if possible.

jjeb commented 10 years ago

Done