1
0
mirror of https://github.com/bitwarden/browser synced 2025-12-10 13:23:34 +00:00

[PM-6125] Handle kp.alg is string (#10285)

* fix: handle cases when `kp.alg` is `string` in `createCredential` for `Fido2`

* feat: simplify and move fix to mappers

* fix null filter

---------

Co-authored-by: Alessio Cosenza <cosenzaalessio17@gmail.com>
Co-authored-by: Merissa Weinstein <merissa.k.weinstein@gmail.com>
Co-authored-by: Todd Martin <tmartin@bitwarden.com>
This commit is contained in:
Andreas Coroiu
2024-08-19 10:17:15 +02:00
committed by GitHub
parent 3e37daf236
commit 71413a7b3b

View File

@@ -36,10 +36,13 @@ export class WebauthnUtils {
extensions: {
credProps: keyOptions.extensions?.credProps,
},
pubKeyCredParams: keyOptions.pubKeyCredParams.map((params) => ({
alg: params.alg,
type: params.type,
})),
pubKeyCredParams: keyOptions.pubKeyCredParams
.map((params) => ({
// Fix for spec-deviation: Sites using KeycloakJS send `kp.alg` as a string
alg: Number(params.alg),
type: params.type,
}))
.filter((params) => !isNaN(params.alg)),
rp: {
id: keyOptions.rp.id,
name: keyOptions.rp.name,