-
-
Notifications
You must be signed in to change notification settings - Fork 17k
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
Upgrade Version of debug package #6204
Comments
This has already been updated in the main branch and released with v5: Line 41 in e4a61bd
However, v4 is still pulling in the older version: Line 42 in 2041584
It seems that v4 just needs a backport with this change to address the security concern. Here are the release notes from when that change was made before. I am sure a PR would be welcome - do you want to open one @rodrigoramos ? If not, I'm happy to. |
It's worth noting that |
Yep @slagiewka is right about updating to debug@4 in express@4 (its a major driver for us revamping the project). And I am not sure if we can even update to the [email protected] branch without breaking compat (I think this was discussed in a pr in the past, search for it and you should find the context). Ideally if we can this would be the kind of thing we would work with the debug maintainers to backport in the currently depended on major line if it impacted I am going to close this unless you can show that |
Hi, there!
IDK if it's the appropriate channel to discuss about that but the package debug in use of this app is on 2.6.9 version. And it has a problem of memory leak as discussed here.
This issue suggests upgrade this package to version 3.7.0 which this problem was already solved.
I can also submit a Pull Request if it would help.
The text was updated successfully, but these errors were encountered: