search

roccomuso / memorystore

express-session full featured MemoryStore layer without leaks!
MIT License
120 stars 20 forks source link

fix: Handling touch session that is previously removed #14

Closed shayantabatabaee closed 4 years ago

shayantabatabaee commented 4 years ago

Hi,

Due to this issue #13 I made these changes and commit to the repository, please kindly review the code and accept the pull request If you find it useful.

thanks you, Shayan Tabatabaee

roccomuso commented 4 years ago

not sure this is needed. The error is catched anyway,

shayantabatabaee commented 4 years ago

Yes the error is catched, but the error is thrown from memorystore module is meaningless, unable to parse JSON u at position 0 , and it is hard to trace problem. So why deleted session should be touched ?

shayantabatabaee commented 4 years ago

@roccomuso : Any update ?

roccomuso commented 4 years ago

Released new version v1.6.2