-
Notifications
You must be signed in to change notification settings - Fork 5
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
CVE-2022-37434 (zlib) found on main #50
Comments
Link to discussion on the relevant commit for completeness: madler/zlib@eff308a Needs clarification from maintainer and MITRE. |
@mscdex since you helped us move to the chromium implementation I wonder if you know how/when they would incorporte a fix for this ? |
@mhdawson You're probably better off asking the Chrome/Chromium team since they maintain the actual code. All I did was just extract their fork and create the gyp. |
@mhdawson could you give permissions to the Security-WG to this repo? I need to add the flag Node.js doesn't use the |
@RafaelGSS you should now have access. I changed core collaborators to read/write. Since the security-wg team has relatively open membership and we don't trim the members very often I'd prefer to give access to non core-collaborators on an individual basis as needed. |
@RafaelGSS thanks for checking. Hopefully you now have access to add that comment to the issues and mark with |
To follow this ticket. |
Fixed on nodejs/node#45387 |
The text was updated successfully, but these errors were encountered: