Skip to content

Commit d7dc19b

Browse files
varjolintuphoerious
authored andcommitted
Passkeys: Fix default BE and BS flag value (#13122)
1 parent 3743d71 commit d7dc19b

File tree

1 file changed

+12
-8
lines changed

1 file changed

+12
-8
lines changed

src/browser/BrowserService.cpp

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -775,12 +775,16 @@ QJsonObject BrowserService::showPasskeysAuthenticationPrompt(const QJsonObject&
775775
const auto userHandle = selectedEntry->attributes()->value(EntryAttributes::KPEX_PASSKEY_USER_HANDLE);
776776

777777
// Get BE and BS flags if present
778-
const auto beFlag = selectedEntry->attributes()->hasKey(EntryAttributes::KPEX_PASSKEY_FLAG_BE)
779-
? selectedEntry->attributes()->value(EntryAttributes::KPEX_PASSKEY_FLAG_BE) == TRUE_STR
780-
: DEFAULT_BE_FLAG;
781-
const auto bsFlag = selectedEntry->attributes()->hasKey(EntryAttributes::KPEX_PASSKEY_FLAG_BS)
782-
? selectedEntry->attributes()->value(EntryAttributes::KPEX_PASSKEY_FLAG_BS) == TRUE_STR
783-
: DEFAULT_BS_FLAG;
778+
const auto beFlag =
779+
selectedEntry->attributes()->hasKey(EntryAttributes::KPEX_PASSKEY_FLAG_BE)
780+
? selectedEntry->attributes()->value(EntryAttributes::KPEX_PASSKEY_FLAG_BE) == "1"
781+
|| selectedEntry->attributes()->value(EntryAttributes::KPEX_PASSKEY_FLAG_BE) == TRUE_STR
782+
: DEFAULT_BE_FLAG;
783+
const auto bsFlag =
784+
selectedEntry->attributes()->hasKey(EntryAttributes::KPEX_PASSKEY_FLAG_BS)
785+
? selectedEntry->attributes()->value(EntryAttributes::KPEX_PASSKEY_FLAG_BS) == "1"
786+
|| selectedEntry->attributes()->value(EntryAttributes::KPEX_PASSKEY_FLAG_BS) == TRUE_STR
787+
: DEFAULT_BS_FLAG;
784788

785789
auto publicKeyCredential = browserPasskeys()->buildGetPublicKeyCredential(
786790
assertionOptions, credentialId, userHandle, privateKeyPem, beFlag, bsFlag);
@@ -863,8 +867,8 @@ void BrowserService::addPasskeyToEntry(Entry* entry,
863867
entry->attributes()->set(EntryAttributes::KPEX_PASSKEY_PRIVATE_KEY_PEM, privateKey, true);
864868
entry->attributes()->set(EntryAttributes::KPEX_PASSKEY_RELYING_PARTY, rpId);
865869
entry->attributes()->set(EntryAttributes::KPEX_PASSKEY_USER_HANDLE, userHandle, true);
866-
entry->attributes()->set(EntryAttributes::KPEX_PASSKEY_FLAG_BE, TRUE_STR);
867-
entry->attributes()->set(EntryAttributes::KPEX_PASSKEY_FLAG_BS, TRUE_STR);
870+
entry->attributes()->set(EntryAttributes::KPEX_PASSKEY_FLAG_BE, "1");
871+
entry->attributes()->set(EntryAttributes::KPEX_PASSKEY_FLAG_BS, "1");
868872
entry->addTag(tr("Passkey"));
869873

870874
entry->endUpdate();

0 commit comments

Comments
 (0)