Skip to content

Conversation

@SebLevDev
Copy link
Contributor

Fix CVE-2021-46743
Introduced through: firebase/[email protected]

Affected versions of this package are vulnerable to Insecure Encryption due to an algorithm-confusion issue (e.g., RS256 / HS256) that exists via the kid (aka Key ID) header when multiple types of keys are loaded in a key ring. This allows an attacker to forge tokens that validate under the incorrect key.

Introduced through: firebase/[email protected]
https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2021-46743
Affected versions of this package are vulnerable to Insecure Encryption due to an algorithm-confusion issue (e.g., RS256 / HS256) that exists via the kid (aka Key ID) header when multiple types of keys are loaded in a key ring. This allows an attacker to forge tokens that validate under the incorrect key.
@SebLevDev SebLevDev requested a review from a team as a code owner March 31, 2022 06:44
@google-cla
Copy link

google-cla bot commented Mar 31, 2022

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

For more information, open the CLA check for this pull request.

@ahthomsen
Copy link

ahthomsen commented Apr 5, 2022

Sure would love for this one to be merged in soon in order for us to meet our security SLAs

@bshaffer bshaffer merged commit 0735218 into googleapis:main Apr 5, 2022
@bshaffer
Copy link
Contributor

bshaffer commented Apr 5, 2022

@ahthomsen Done and tagged in v2.12.2

@ahthomsen
Copy link

@bshaffer that is awesome - thanks a ton.

Now we just ned "google/auth" to be upgraded to support firebase/[email protected] as well

@bshaffer
Copy link
Contributor

bshaffer commented Apr 6, 2022

I'm working on it!!

googleapis/google-auth-library-php#391

@mfn
Copy link

mfn commented Apr 6, 2022

image

😏

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants