react-notifications icon indicating copy to clipboard operation
react-notifications copied to clipboard

Error: in izomorphic app in server render memory leaks!

Open royroev opened this issue 8 years ago • 3 comments

profiler and headsnapshot rules!)

royroev avatar Aug 26 '16 15:08 royroev

I am not recommending using this lib because it causes memory leaks all the time.

ahromyak avatar Jan 19 '18 08:01 ahromyak

Adding on to this, we've had to completely remove this from production after a week of sporadic downtime due to memleaks originating from this package.

Absolutely do not use this package in an isomorphic / universal application until this has been resolved.

CharlesCarnell avatar Mar 02 '18 10:03 CharlesCarnell

After some review of the code and debugging, this is caused by the event registration componentWillMount lifecycle of the NotificationContainer. This is not the correct place to register events and a few other issues in here are related to this issue. I'm lobbying for a quick resolution since this would effect those using React 15 and 16. Once React 17 is out, this will be broken altogether.

https://react-legacy.netlify.com/docs/react-component.html#componentwillmount

And for the resolution, see:

https://reactjs.org/docs/react-component.html#unsafe_componentwillmount

philn5d-ar avatar Oct 24 '19 15:10 philn5d-ar