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

NPM Audit reveals high vulnerability in static expiry #20

Open
SamBroner opened this issue May 27, 2018 · 5 comments
Open

NPM Audit reveals high vulnerability in static expiry #20

SamBroner opened this issue May 27, 2018 · 5 comments

Comments

@SamBroner
Copy link

NPM audit reveals high vulnerability issues in dependency. This is fixable by revving fresh to 0.5.2

┌───────────────┬──────────────────────────────────────────────────────────────┐
│ High │ Regular Expression Denial of Service │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package │ fresh │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Patched in │ >= 0.5.2 │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ static-expiry │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path │ static-expiry > fresh │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ More info │ https://nodesecurity.io/advisories/526
└───────────────┴──────────────────────────────────────────────────────────────┘

@ethanresnick
Copy link
Collaborator

Thanks @SamBroner. I've pushed a commit to fix this. I don't think I have npm publishing permissions, though, so @paulwalker's gonna have to take care of that.

@ethanresnick ethanresnick reopened this May 27, 2018
@Mihailoff
Copy link

@paulwalker

@plavski
Copy link

plavski commented Oct 25, 2018

@paulwalker getting this update published would be very useful, thanks

@apolunin
Copy link

Thank you @ethanresnick for fixing this problem!
@paulwalker it would be awesome if you could find time to publish a new version to npm registry with vulnerability issue fixed.

Thank you in advance

@jorgecolonconsulting
Copy link

You could still install the static-expiry version with the security fix by installing it directly from github using the commit version: npm install paulwalker/connect-static-expiry#2bac82d. Not as flexible as relying on semver, but it's a decent workaround while the commit gets tagged and published to npm.

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

6 participants