-
Notifications
You must be signed in to change notification settings - Fork 19
Open
Description
Memory leak!
Hello!
Not sure if it is done on purpose, and it is by-design, but I encountered a huge memory leak using your library with express.js.
I created a tiny PoC for you to check it out. https://github.com/Sprit3Dan/mem-leak
The way it could be reproduces is to make the callback for the express path asynchronous by adding the async keyword.
Here are the charts for heap used per ~1k queries.
If you add the library:
If not:
Guess many people could be stuck at this situation, when using you lib.
Hope you find this ticket useful :)
Metadata
Metadata
Assignees
Labels
No labels

