Bernardo Santos
Bernardo Santos
Also identified ([here](https://github.com/elastic/elasticsearch-js#compatibility)) that the new ElasticSearch client requires Node `>8` ... With that said, I just removed a little destructuring that I found... I'll try to run a linter...
The tests that worked on TravisCI fail locally Should I try to include `elasticsearch` and `@elasticsearch` on [this](https://github.com/bernardocs/sessionstore/blob/master/test/sessionStoreTest.js#L65) collection to see if things run smoothly on Travis? :thinking:
ElasticSearch new package (`@elastic/elasticsearch`) was not in the `package.json` (duh) and, even so, the client has changed a little the return body... that's why the tests went red... I'll try...
I noticed that the Elasticsearch store implementation tries to use TTL as a mean to invalidate sessions (correct me if I'm wrong)... but there's a problem with that: TTL feature...
Will this ever be merged?
This featured (with @totakoko fix) proved very useful... Thx :))