JavaScript heap out of memory about zxcvbn HOT 8 CLOSED

I've tested it in multiple environments including entirely new projects. I've also used the node max_old_space_size env with as much as 8GB of ram (It should be noted that it isn't the browser tab that crashes, it's the actual dev process).

The problem isn't some absolute minimum memory footprint - but that like I've mentioned, regardless of the initial max memory constraint, the usage just immediately skyrockets and keeps climbing until hitting the max and crashing the process. This seems to actually stem from a real memory leak that has something to do with webpack-dev-server interop, as running the module in a simple node project (not CRA, just plain old node), causes no problems.

It should also be noted that this hasn't happened to me with the original zxcvbn package, just with zxcvbn-ts.

from zxcvbn.

I don't think it is a direct issue with zxcvbn-ts because it will already crash if you only import the password.js file into a fresh repo. The content of this file is only one variable with a big array.

But it seems like the approach from the original repo is more fit for big data like that. In the original repo those dictionaries are not an array but a string that is splitted on runtime.
I thought it would be a good idea to directly use an array to increase runtime performance but it seems like this comes with a downside for some dev servers 🤔

I will adjust the dictionaries so that they match the original initalization with the split function.
Thanks for the finding 👍

from zxcvbn.

I just release version 1.0.0 with this issue included. In my starter setup for CRA4 it worked now. Can you try it out for youself?

from zxcvbn.

You need to be aware that the dictionaries are huge there are 2MB of plain old json data.

I can say for myself that i never encountered this issue but i am using vue and nuxt instead of react. On the other side im using webpack for vue and nuxt too.
Does this happen for a new project or do you included it into a bigger project?

from zxcvbn.

This is weird, there are no functions that could go into a memory leak but i will check it out 🤔

Wich version are you using?

from zxcvbn.

I'm using the latest version. Another thing to add is that both JS and TS templates of CRA crashed for me, on different computers as well.

from zxcvbn.

Interesting! Thank you.

from zxcvbn.

It seems to work now - thank you very much.

from zxcvbn.