-
Notifications
You must be signed in to change notification settings - Fork 424
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
[TEZ-4426][CVE-2018-1000620] Upgrade cryptiles from 2.0.5 to 4.1.2 #227
[TEZ-4426][CVE-2018-1000620] Upgrade cryptiles from 2.0.5 to 4.1.2 #227
Conversation
🎊 +1 overall
This message was automatically generated. |
…needs nodeVersion >= 8.9.0 as denoted in the build stack trace.
The version of node that cryptiles 4.1.2 needs is >= 8.9.0 but we cannot use the current LTS version of node since the build is throwing the following error: (node:9989) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead. If we want to fix this error, we need to fix all the occurences of new Buffer(string) with Buffer.alloc(), which can be a huge change. ### Also the max version of node in which the build works perfectly is 9.11.2 So to summarize if we want to upgrade to cryptiles to 4.1.2 without any code changes related to Buffer, we need to use nodeVersion >=8.9.0 and nodeVersion<=9.11.2. That's why I have used version 8.9.0 for now. |
@guptanikhil007 Can you please approve this change. |
💔 -1 overall
This message was automatically generated. |
with addendum patch nodejs was properly downloaded, module was successfully built, +1 |
[TEZ-4426][CVE-2018-1000620] Upgrade cryptiles from 2.0.5 to 4.1.2 to fix the vulnerability.
Link to JIRA : https://issues.apache.org/jira/browse/TEZ-4426
Link to parent JIRA : https://issues.apache.org/jira/browse/TEZ-4419
RFC documentation : https://github.com/yarnpkg/rfcs/blob/master/implemented/0000-selective-versions-resolutions.md