-
-
Notifications
You must be signed in to change notification settings - Fork 8k
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
Vulnerabilities #2831
Comments
Hi @Templore, Thank you for highlighting the security vulnerabilities in xlsx version 0.18.5. You're correct that this version has two significant vulnerabilities:
These security issues have been addressed in newer versions. The recommended solution is to use SheetJS directly from their CDN with version 0.20.3:
For newer Yarn versions that require explicit package naming, use the Note that if you're using Snyk security tooling, you might still see warnings about "Prototype Pollution" (SNYK-JS-XLSX-5457926), but according to SheetJS, these are false positives for versions 0.19.3 and later. Their official recommendation is to suppress these warnings. Thanks for bringing attention to these security issues. This information is also documented in the official SheetJS documentation: https://docs.sheetjs.com/docs/getting-started/installation/nodejs/ Since there's a clear solution available and the maintainers are aware of these issues, I believe this ticket can be closed. 👍 |
If you don't want to give up the convenience of NPM, you can use these packages instead of @e965/xlsx (#2822 (comment)) If you don't want to edit imports throughout the code, you can do it this way (for example): - "xlsx": "0.18.5",
+ "xlsx": "npm:@e965/[email protected]", |
works fine |
Hi, @JuanTorchia, my team upgraded to the latest version (
|
https://www.npmjs.com/package/xlsx
latest version 0.18.5 have two vulnerabilities
The text was updated successfully, but these errors were encountered: