-
Notifications
You must be signed in to change notification settings - Fork 123
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
There seems to be something wrong with CVE-2017-16138 #469
Comments
If it helps, the npm site has the full list of which versions are vulnerable vs not: https://www.npmjs.com/advisories/535/versions |
Hi Doug, Looks like you are correct with regards to 1.4.1 not being vulnerable and I can assume that the fix was backported after the CVE was requested and that's why it's not in the original CVE report. Also, it seems that the CVE was requested through HackerOne but probably with an older process than what we have today which is perhaps why I can't find the entry for that in our database. I will email HackerOne support to see if they can help update this information, but it's not an immediate action and will take some time for them to process it through their support queue and such. Is there anything else we can do to help? |
No, it was never backported. The 2 and 1 lines are maintained in that module. The fix was released for both majors at the same time: broofa/mime#167 (comment)
Maybe if there is just somewhere I can point people to? I am trying to enjoy holiday but getting multiple emails a day, some are not very nice. |
You can point to the npm page as you did in the thread. Hope you get to enjoy the holiday. It will not help you with email but perhaps consider having the security issue on github open so people can see it and the discussion there otherwise they might go ahead and create new issues and you'll keep closing them. |
Support ticket to HackerOne opened: 257136 |
Good point. I just did and updated the title so hopefully it helps vs people posting update mime over and over :) |
Sure thing. |
@dougwilson I have some positive update from the folks at HackerOne on this:
I'll keep you posted. |
The issue is still open in H1 and their support hadn't got back to me after several pings on my part. |
This has been attended to so I'm closing. |
Hi, I'm not sure if this is the right place to report this, but my understanding is that this is the group that manages the Node.js CVEs.
Recently I started getting reports that something is detecting CVE-2017-16138 present in the latest version of Express.js as it is detecting that the dependency mime 1.4.1 is vulnerable to CVE-2017-16138.
Unfortunately this is incorrect, as 1.4.1 was the specific release made to fix that CVE.
No, I don't yet know what software is alerting this, as everyone so far that has reported this just simply ghosted and never followed back up.
I even got a few who posted this as a public issue on GitHub (though the majority of the reports were through private channels): expressjs/express#3841
Is there anything that just needs to be updated in CVE-2017-16138 to correct this? What can we do?
The text was updated successfully, but these errors were encountered: