`npm audit` is failing

[jfly]

Here's what npm audit shows against main:

npm audit

``` $ npm audit ... === npm audit security report === ┌──────────────────────────────────────────────────────────────────────────────┐ │ Manual Review │ │ Some vulnerabilities require your attention to resolve │ │ │ │ Visit https://go.npm.me/audit-guide for additional guidance │ └──────────────────────────────────────────────────────────────────────────────┘ ┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Low │ Prototype Pollution │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ minimist │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Patched in │ >=0.2.1 <1.0.0 || >=1.2.3 │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ gulp-iconfont [dev] │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ gulp-iconfont > gulp-svg2ttf > gulp-util > minimist │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://npmjs.com/advisories/1179 │ └───────────────┴──────────────────────────────────────────────────────────────┘ ┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Low │ Prototype Pollution │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ minimist │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Patched in │ >=0.2.1 <1.0.0 || >=1.2.3 │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ gulp-iconfont [dev] │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ gulp-iconfont > gulp-ttf2eot > gulp-util > minimist │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://npmjs.com/advisories/1179 │ └───────────────┴──────────────────────────────────────────────────────────────┘ ┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Low │ Prototype Pollution │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ minimist │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Patched in │ >=0.2.1 <1.0.0 || >=1.2.3 │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ gulp-iconfont [dev] │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ gulp-iconfont > gulp-ttf2woff > gulp-util > minimist │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://npmjs.com/advisories/1179 │ └───────────────┴──────────────────────────────────────────────────────────────┘ ┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Low │ Prototype Pollution │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ minimist │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Patched in │ >=0.2.1 <1.0.0 || >=1.2.3 │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ gulp-jimp [dev] │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ gulp-jimp > gulp-util > minimist │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://npmjs.com/advisories/1179 │ └───────────────┴──────────────────────────────────────────────────────────────┘ ┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Low │ Prototype Pollution │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ minimist │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Patched in │ >=0.2.1 <1.0.0 || >=1.2.3 │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ gulp-svg2png [dev] │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ gulp-svg2png > gulp-util > minimist │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://npmjs.com/advisories/1179 │ └───────────────┴──────────────────────────────────────────────────────────────┘ ┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Low │ Prototype Pollution │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ minimist │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Patched in │ >=0.2.1 <1.0.0 || >=1.2.3 │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ gulp-jimp [dev] │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ gulp-jimp > jimp > mkdirp > minimist │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://npmjs.com/advisories/1179 │ └───────────────┴──────────────────────────────────────────────────────────────┘ ┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Low │ Prototype Pollution │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ minimist │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Patched in │ >=0.2.1 <1.0.0 || >=1.2.3 │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ gulp-svg2png [dev] │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ gulp-svg2png > svg2png > phantomjs-prebuilt > extract-zip > │ │ │ mkdirp > minimist │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://npmjs.com/advisories/1179 │ └───────────────┴──────────────────────────────────────────────────────────────┘ ┌───────────────┬──────────────────────────────────────────────────────────────┐ │ Low │ Prototype Pollution │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Package │ yargs-parser │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Patched in │ >=13.1.2 <14.0.0 || >=15.0.1 <16.0.0 || >=18.1.2 │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Dependency of │ gulp-svg2png [dev] │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ Path │ gulp-svg2png > svg2png > yargs > yargs-parser │ ├───────────────┼──────────────────────────────────────────────────────────────┤ │ More info │ https://npmjs.com/advisories/1500 │ └───────────────┴──────────────────────────────────────────────────────────────┘ found 8 low severity vulnerabilities in 793 scanned packages 8 vulnerabilities require manual review. See the full report for details. ```

These stragglers all are indirectly related to these top level dependencies that I think may be unmaintained now. The state of things as of 2020-07-16:

• gulp-svg2png: Last commit Jan 26, 2017
• gulp-jimp: Last commit Jan 10, 2018)
• gulp-iconfont: Last commit Jun 13, 2019

It might not be worth going to a lot of effort to fix this:

• gulp may be going out of style, so even if the core project is maintained, plugin developers seem to be fading away. I don't know that it's worth us forking the above projects and trying to get them working. @lgarron suggests it might be worth investigating parcel as a more modern alternative.
• These security vulnerabilities don't actually affect us: we are just generating static content, and this code all runs in a VM secured by Travis.

So, for now, I'm going to just comment out the npm audit step of our build and link back to this issue.

[dmint789]

Is this being worked on? This seems like an urgent issue to fix

[jfly]

It's not actively being worked on, but there's https://github.com/cubing/icons/pull/112, which would make these problems evaporate.

The original post explains why this isn't urgent.