this.ttl not set

[nono88]

Hello;

I got an issue (which is maybe due to a misuse of your module)

I use it this way :

var MS = require('express-mongoose-store')(expressSession, mongoose);
var sessionStore = new MS({ttl: 600000});

but when i the cookie is set in function MongooseStore.prototype.set = function (sid, sess, fn) {
on this line :
var expires = Date.now() + this.ttl;
This.ttl is not set ( = null) then expires is an invalidate Date; and so i do not manage to get a functional cookie.

Did i miss something ?

Thank you
Arnaud

[nono88]

this.ttl has been removed by this commit :
e1e9e65

[battlejj]

@nono88 this should fix the issue (36c04e5), thanks for bringing it to my attention, I missed this when reviewing the PR that introduced the bug. I will be pushing it to npm shortly.

[battlejj]

I've pushed the updated version to npm, going to close this issue out, but if you still have issues please let me know, better tests for the library is on the todo list but obviously isn't quite there yet. Thanks again.

[nono88]

Perfect thank you for your prompt answer :) and thanks for the module ;)