freeCodeCamp / camper-gitter-bot

Finally, robotic beings rule the world.
BSD 3-Clause "New" or "Revised" License
96 stars 59 forks source link

Fix for the ReDOS vulnerability #114

Closed snyk-community closed 7 years ago

snyk-community commented 8 years ago

camperbot is currently affected by the high-severity ReDoS vulnerability.

Vulnerable module: negotiator Introduced through: express

This PR fixes the ReDoS vulnerability by upgrading express to version 4.14.0. This upgrade will also fix the following other vulnerabilities:

Check out the Snyk test report to review other vulnerabilities that affect this repo.

Watch the repo to

Stay secure, The Snyk team

QuincyLarson commented 7 years ago

@raisedadead since you now have CamperBot running locally, can you verify that updating the Express version doesn't seem to break any existing functionality?

raisedadead commented 7 years ago

Before I test and merge this, I want to confirm with @BerkeleyTrue

BerkeleyTrue commented 7 years ago

@raisedadead Confirm what?

raisedadead commented 7 years ago

Opps, sorry incorrect mention. I meant @dhcodes !

Dylan, could also double check?

dhcodes commented 7 years ago

Doesn't seem to break anything on my end. Merging in.

dhcodes commented 7 years ago

Submitted a revert request. This does actually break the install. Based on the error, Express now has you install middlewares separately so we need to determine which we need to add to package.json before we update express. Here's the list: https://github.com/senchalabs/connect#middleware.