Open alexandruluca opened 7 years ago
Had to use almost the same approach.
@AleksMeshkov have you opened a pull request? This will be a problem for anyone using CLS with express.
@othiym23 Do we need a test for @AleksMeshkov s fix in a pull request?
Have you tried using ns.bindEmitter(res)
?
Seems likely there is a branch of the execution tree that CLS is not aware of, resulting in the tree resolving early and exiting when some activity is still pending. The most likely cause of that is event emitters as they are not patched automatically.
I'm using this package to create a session for the current user so I can access session inside nested promises, including mongoose plugins and so on. For whatever reason, the namespace exists too quick and the session get's lost.
I debugged your code a little bit and I came up with this solution for my problem. In context.js line 44 in the Namespace.prototype.run method I patched it like this
Basically I simply exit the namespace on an error and always exit on response finish
This way I am sure that my session is active untill I send back my response to the client
This works for me as intended. Are there any cons to this approach? Looking forward to your feedback