-
Notifications
You must be signed in to change notification settings - Fork 35
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SSR Memory leak #20
Comments
Has it been solved? |
This is still an issue. Switching to a WeakMap would likely help. I opened a PR with a basic implementation. |
It seems like the problem has not been solved yet. Is there any plan? 😭 |
I'm integrating part of this library in an experiment and ran into this issue. another potential solution is to add an indicator to the "child" that it was generated by or instead of returning the string |
I recently added this package in our project and I was told it's causing memory leak in our prod env.
After reading the code, it seems like it's the
sanitized
cache that's causing the problem.Every generated html string(
s
) would be stored in this cache, and it seems to have caused our server run out of memory and keeps restarting.The text was updated successfully, but these errors were encountered: