zapier/zapier-platform-cli

Obsolete Mocha depends on vulnerable Growl

Closed this issue · 5 comments

wking commented

This package currently depends on Mocha 3.4.2, which in turn depends on Growl 1.9.2. Growl <1.10.2 are vulnerable to arbitrary command injection, so npm audit run in a Zapier app with the Mocha dev dependency reports that advisory as a critical security issue. Mocha is currently up to 5.1.1, which depends on Growl 1.10.3. It would be nice to bump this package to the current Mocha to avoid the reported security issue.

wking commented

For what it's worth, a brief skim of the Mocha 5.0.0 and 4.0.0 didn't turn up anything too suspicious, and with my app,

$ npm install --only=dev 'mocha@5.1.1'
$ zapier test

seems to work fine. I don't know enough about this package's Mocha consumption to know how reliable that single success is ;).

wking commented

This will be fixed by #320.

closing as duplicate of #319

wking commented

#319 is about the core package. This issue is about this package.

that's fair. Given that all 3 repos will be updated simultaneously, it seems like a single "npm audit returns a lot of issues" is the cleaner way to track. They all use mocha, so it's the same fix for each.