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

@google-cloud/pubsub@2.5.0 includes dependencies with security vulnerabilities #1121

Closed
rvillane opened this issue Oct 5, 2020 · 7 comments · Fixed by #1122, googleapis/google-auth-library-nodejs#1078 or #1133
Assignees
Labels
api: pubsub Issues related to the googleapis/nodejs-pubsub API. priority: p2 Moderately-important priority. Fix may not be included in next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@rvillane
Copy link

rvillane commented Oct 5, 2020

node-forge v0.9.1 was recently identified with high severity vulnerability CVE-2020-7720 [1]. The issue has been solved in v0.10.0

└─┬ @google-cloud/pubsub@2.5.0
  └─┬ google-auth-library@6.0.6
    └─┬ gtoken@5.0.3
      └─┬ google-p12-pem@3.0.2
        └── node-forge@0.9.1 

[1] https://www.npmjs.com/advisories/1561

@product-auto-label product-auto-label bot added the api: pubsub Issues related to the googleapis/nodejs-pubsub API. label Oct 5, 2020
@meredithslota meredithslota added priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. labels Oct 6, 2020
@yoshi-automation yoshi-automation added 🚨 This issue needs some love. and removed 🚨 This issue needs some love. labels Oct 11, 2020
@cermakja
Copy link

The json-bigint module dependency also needs to be upgraded to 1.0.0 remediate a security issue.

Prototype Pollution
Vulnerable module: json-bigint
Introduced through: @google-cloud/pubsub@2.5.0
Exploit maturity: Proof of concept
Fixed in: 1.0.0

Affected versions of this package are vulnerable to Prototype Pollution via the parse function.

https://app.snyk.io/vuln/SNYK-JS-JSONBIGINT-608659

@feywind
Copy link
Collaborator

feywind commented Oct 15, 2020

Thanks for the notes, everyone. I'll get that PR looked at and see if I can throw in a quick fix for the json-bigint entry as well.

I'm also having a conversation with others on the team to see if there's a way we can improve the transitive dependency issue on things like the auth library.

@feywind
Copy link
Collaborator

feywind commented Oct 16, 2020

Ahh thanks, GitHub :P I need to reopen this for the second package above.

@feywind
Copy link
Collaborator

feywind commented Oct 16, 2020

I'll need to do another PR for this library once that one is updated.

I was asked to pass along that our general intent with transitive dependencies is basically:

rm -rf node_modules package-lock.json && npm i

(Of course if you really do need to lock versions, you can also manually bump the version of a dependency from your project.)

In other words, because of the depth and complexity of the average Node package's package graph, it's not generally practical for us to maintain minimum versions upstream for every fix, so the recommended thing is just to refresh your versions once in a while to make sure you have compatible fixes.

@feywind feywind added priority: p2 Moderately-important priority. Fix may not be included in next release. and removed 🚨 This issue needs some love. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. labels Oct 19, 2020
@feywind
Copy link
Collaborator

feywind commented Oct 19, 2020

Oh, GitHub. :sigh:

@csombok
Copy link

csombok commented Oct 21, 2020

Thanks for the fix. Do you know the date when the new official version will be available with the vulnerability fixes?

@feywind
Copy link
Collaborator

feywind commented Oct 23, 2020

@csombok It should be up already, actually - 2.6.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: pubsub Issues related to the googleapis/nodejs-pubsub API. priority: p2 Moderately-important priority. Fix may not be included in next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
6 participants