Skip to content
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

Security Warning for Moment.js #415

Closed
omohayui opened this issue May 24, 2021 · 2 comments
Closed

Security Warning for Moment.js #415

omohayui opened this issue May 24, 2021 · 2 comments

Comments

@omohayui
Copy link

WhiteSource (which detects open source libraries with security or compliance issues) alerts an security vulnerability of moment.js.

The moment module before 2.19.3 for Node.js is prone to a regular expression denial of service via a crafted date string, a different vulnerability than CVE-2016-4055.

I know minify is using it in only tests and benchmarktools but it seems better to update this library.

Thank you in advance.

@tdewolff
Copy link
Owner

Thanks for the issue! The truth is that the JS code is purely used for benchmarking and is never executed (the minifiers don't execute JS). To maintain the benchmark results comparable with the past, it is necessary to maintain the samples as they are. In this case there is no security risk so it shouldn't matter. Thank you in any case for mentioning!

@omohayui
Copy link
Author

Thank you for your confirmation!

To maintain the benchmark results comparable with the past, it is necessary to maintain the samples as they are.

I understood it and agree with you. Let me close this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants