-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Closed
Labels
dependenciesIssue about dependencies of the packageIssue about dependencies of the packagehelp wantedExtra attention is neededExtra attention is neededpackage: typescript-estreeIssues related to @typescript-eslint/typescript-estreeIssues related to @typescript-eslint/typescript-estree
Description
Lodash has an open security vuln and shows signs of being borderline unmaintained.
Repro
- Install typescript-eslint/eslint-plugin
- Check your Snyk report (e.g. https://snyk.io/test/github/danielnixon/eslint-plugin-total-functions?targetFile=package.json)
- Or run yarn audit / npm audit
Expected Result
No security vuln reported
Actual Result
Lodash security vuln reported
Additional Info
It looks like typescript-estree only uses lodash once, for unescape
. unescape
happens to be tiny and unlikely to evolve over time: https://github.com/lodash/lodash/blob/4.17.11/lodash.js#L15145
I'd be happy to raise a PR to inline unescape
(or maybe replace it with https://www.npmjs.com/package/he or something) and remove the lodash dependency.
Versions
Latest
fbaba-nibtravel, armano2 and fannheyward
Metadata
Metadata
Assignees
Labels
dependenciesIssue about dependencies of the packageIssue about dependencies of the packagehelp wantedExtra attention is neededExtra attention is neededpackage: typescript-estreeIssues related to @typescript-eslint/typescript-estreeIssues related to @typescript-eslint/typescript-estree