actions-on-google / assistant-conversation-nodejs

A developer friendly way to fulfill Actions SDK handlers for the Google Assistant
https://actions-on-google.github.io/assistant-conversation-nodejs
Apache License 2.0
104 stars 27 forks source link

Security vulnerability due to outdated google-auth-library dependency #10

Closed jankratochvilcz closed 3 years ago

jankratochvilcz commented 4 years ago

Running npm run audit results in a high-severity vulnerability

High            Prototype Pollution in node-forge                             

  Package         node-forge                                                    

  Patched in      >= 0.10.0                                                     

  Dependency of   @assistant/conversation                                       

  Path            @assistant/conversation > google-auth-library > gtoken >      
                  google-p12-pem > node-forge                                   

  More info       https://npmjs.com/advisories/1561     

The core issue appears to be the usage of an obsoleted version of the "google-auth-library": "^5.10.1" dependency which prevents us from updating the node-forge upstream dependency.

Fleker commented 4 years ago

Thanks for identifying this.

jankratochvilcz commented 4 years ago

Please let me know how can I help here and at which point you could take a look into this further. Our CI is strict about high-severity vulnerabilities, so I could take a swing at migrating to the new library version as well. There are some non-specified breaking changes awaiting in v6 ref, but maybe they won't be a problem.

Let me know how you'd like to proceed here.

Fleker commented 4 years ago

My plan for this library is to try updating the auth-library to v6 and run tests/sample. I don't believe there'd be any breakages, or if so they'd be minimal.

jankratochvilcz commented 4 years ago

Thanks @Fleker, appreciate it! If you could give me a rough estimate of when you could take a look it'd help us plan on our side :-)

Fleker commented 4 years ago

I hope to have an update before the week's end.

jankratochvilcz commented 3 years ago

Appreciate the fast turnaround here, thanks!