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

Vulnerability in jose subdependency #2494

Closed
m-wagner98 opened this issue Mar 14, 2024 · 3 comments
Closed

Vulnerability in jose subdependency #2494

m-wagner98 opened this issue Mar 14, 2024 · 3 comments
Assignees

Comments

@m-wagner98
Copy link

Environment:

  • Operating System version: _____
  • Firebase SDK version: 12.0.0
  • Firebase Product:Top-level idk (auth, database, storage, etc)
  • Node.js version: n/a
  • NPM version: n/a

[REQUIRED] Step 3: Describe the problem

Steps to reproduce:

  1. Create a npm project and install the firebase-admin package.
  2. Perform an security analysis of the dependencies using OWASP dependency-check plugin
  3. The analysis fails due to a known vulnerability in the jose package.
  4. With npm ls jose we can find out where the dependency comes from:

@computer% npm ls jose
@app/[email protected] /Users/wagnem46/dev/notificationmanager-v2
└─┬ [email protected]
└─┬ [email protected]
└── [email protected]

Vulnerability: GHSA-hhhv-q57g-882q
https://nvd.nist.gov/vuln/detail/CVE-2024-28176

@google-oss-bot
Copy link

I found a few problems with this issue:

  • I couldn't figure out how to label this issue, so I've labeled it for a human to triage. Hang tight.
  • This issue does not seem to follow the issue template. Make sure you provide all the required information.

@lahirumaramba
Copy link
Member

lahirumaramba commented Mar 14, 2024

Thanks for filing this issue. Since the vulnerability is in a dependency of jwks-rsa the fix should be addressed in that package. It also doesn't look like jwks-rsa pins to a specific version of jose so you might be able to upgrade jose to v4.15.5 (which includes the fix) in your environment. See auth0/node-jwks-rsa#403

@m-wagner98
Copy link
Author

Okay we will upgrade on our environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants