pavelvlasov / koa-generic-session-mongo

MongoDB store for koa-session middleware backed by node-mongodb-native.
MIT License
37 stars 16 forks source link

koa-generic-session set error: Cannot read property 'maxAge' of undefined #14

Closed sumaolin closed 7 years ago

sumaolin commented 8 years ago
koa-generic-session set error: Cannot read property 'maxAge' of undefined
    at MongoStore.set$ (f:\Users\dev\bbs\node_modules\koa-generic-session-mongo\dist\store.js:195:33)
    at tryCatch (f:\Users\dev\bbs\node_modules\babel-runtime\regenerator\runtime.js:72:40)
    at GeneratorFunctionPrototype.invoke [as _invoke] (f:\Users\dev\bbs\node_modules\babel-runtime\regenerator\runtime.js:334:22)
    at GeneratorFunctionPrototype.prototype.(anonymous function) [as next] (f:\Users\dev\bbs\node_modules\babel-runtime\regenerator\runtime.js:105:21)
    at onFulfilled (f:\Users\dev\bbs\node_modules\co\index.js:65:19)
    at f:\Users\dev\bbs\node_modules\co\index.js:54:5
    at Object.co (f:\Users\dev\bbs\node_modules\co\index.js:50:10)
    at Object.toPromise (f:\Users\dev\bbs\node_modules\co\index.js:118:63)
    at next (f:\Users\dev\bbs\node_modules\co\index.js:99:29)
    at onFulfilled (f:\Users\dev\bbs\node_modules\co\index.js:69:7)
    at f:\Users\dev\bbs\node_modules\co\index.js:54:5
    at Object.co (f:\Users\dev\bbs\node_modules\co\index.js:50:10)
    at Object.toPromise (f:\Users\dev\bbs\node_modules\co\index.js:118:63)
    at next (f:\Users\dev\bbs\node_modules\co\index.js:99:29)
    at onFulfilled (f:\Users\dev\bbs\node_modules\co\index.js:69:7)
    at f:\Users\dev\bbs\node_modules\co\index.js:54:5

错误如上,相关的代码

app.use(session({
  store: new MongoStore(config.mongodb)
}));

是windows 7 下 node V4.4.3 koa-generic-session-mongo : v:0.0.25

jurgob commented 7 years ago

I have the same error using this store with koa-header-session

attekei commented 7 years ago

Experiencing the same error too, in my case in conjunction with koa-session-minimal.

jurgob commented 7 years ago

I've fixed it with this pull request: https://github.com/pavelvlasov/koa-generic-session-mongo/commit/891d9a2e2ebe061f4ab6cfad2a74f242243ba52b

yuu2lee4 commented 7 years ago

@jurgob maybe is #16

lzztt commented 7 years ago

@jurgob @yuu2lee4 got the same issue reported. I think the store's set interface need to be updated. Otherwise, the database records will have undefined TTLs.

The expected interface should be set(sid, sess, ttl): set session object for sid, with a ttl (in ms), for both koa-session-minimal and koa-generic-session.

pavelvlasov commented 7 years ago

thanks guys, interface is updated in https://github.com/pavelvlasov/koa-generic-session-mongo/commit/6115c41039759e2b0bfc5ea1fb3f7a9007c1e766 (v0.3.0)