-
Notifications
You must be signed in to change notification settings - Fork 842
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
EJS security vulnerability #323
Comments
Could you provide me with a little more context here? |
https://nvd.nist.gov/vuln/detail/CVE-2017-1000188
|
Wow, wasn't aware there was a CVE for this. @qq645381995 You need to upgrade to the latest version of EJS. @mde Shouldn't we deprecate all the older versions? |
There's not a great way to prevent people from using older versions, unless we un-publish from NPM, which could cause even worse problems (c.f., "left pad"). |
@mde I was referring to |
Oh, nice. I haven't used that. (What version of NPM added that? :)) I'll add a deprecation ASAP. |
@mde I don't remember hearing about it being added, so I don't know if it existed ever since I was in the Node world, or if I just never heard any fanfare. |
The text was updated successfully, but these errors were encountered: