Skip to content
This repository has been archived by the owner on Jul 6, 2021. It is now read-only.

fix(deps): update dependency express-jwt to v6 [security] #347

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Jun 30, 2020

WhiteSource Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
express-jwt 5.3.3 -> 6.0.0 age adoption passing confidence

GitHub Vulnerability Alerts

CVE-2020-15084

Overview

Versions before and including 5.3.3, we are not enforcing the algorithms entry to be specified in the configuration.
When algorithms is not specified in the configuration, with the combination of jwks-rsa, it may lead to authorization bypass.

Am I affected?

You are affected by this vulnerability if all of the following conditions apply:

You are using express-jwt
AND
You do not have algorithms configured in your express-jwt configuration.
AND
You are using libraries such as jwks-rsa as the secret.

How to fix that?

Specify algorithms in the express-jwt configuration. The following is an example of a proper configuration

const checkJwt = jwt({
  secret: jwksRsa.expressJwtSecret({
    rateLimit: true,
    jwksRequestsPerMinute: 5,
    jwksUri: `https://${DOMAIN}/.well-known/jwks.json`
  }),
  // Validate the audience and the issuer.
  audience: process.env.AUDIENCE,
  issuer: `https://${DOMAIN}/`,
  // restrict allowed algorithms
  algorithms: ['RS256']
}); 

Will this update impact my users?

The fix provided in patch will not affect your users if you specified the algorithms allowed. The patch now makes algorithms a required configuration.

Credit

IST Group


Release Notes

auth0/express-jwt

v6.0.0

Compare Source


Configuration

📅 Schedule: "" (UTC).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻️ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box.

This PR has been generated by WhiteSource Renovate. View repository job log here.

@renovate renovate bot force-pushed the renovate/npm-express-jwt-vulnerability branch from 6316308 to 41bc8e0 Compare August 14, 2020 10:54
@renovate renovate bot force-pushed the renovate/npm-express-jwt-vulnerability branch from 41bc8e0 to 6dd449c Compare February 11, 2021 21:28
@renovate renovate bot force-pushed the renovate/npm-express-jwt-vulnerability branch from 6dd449c to 517adb0 Compare April 2, 2021 12:42
@renovate renovate bot force-pushed the renovate/npm-express-jwt-vulnerability branch from 517adb0 to 36bb1c1 Compare May 5, 2021 23:53
@renovate renovate bot force-pushed the renovate/npm-express-jwt-vulnerability branch from 36bb1c1 to dbc0ee4 Compare June 3, 2021 02:17
@renovate renovate bot force-pushed the renovate/npm-express-jwt-vulnerability branch from dbc0ee4 to 2bb342c Compare July 1, 2021 09:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant