tor-browser

commit 285d66fff370e61c3cfac751e85b65871665553b
parent a2bfc14e2d56b588ecf3b8ff79aca32d6be11ee4
Author: Sammy Khamis <skhamis@mozilla.com>
Date:   Wed, 26 Nov 2025 00:40:07 +0000

Bug 2002410 - hasKeysForScope should only check keys and not verified r=markh

Differential Revision: https://phabricator.services.mozilla.com/D274064

Diffstat:
M
services/fxaccounts/FxAccountsKeys.sys.mjs
 | 
3
---
M
services/fxaccounts/tests/xpcshell/test_accounts.js
 | 
5
+++--

2 files changed, 3 insertions(+), 5 deletions(-)
diff --git a/services/fxaccounts/FxAccountsKeys.sys.mjs b/services/fxaccounts/FxAccountsKeys.sys.mjs
@@ -117,9 +117,6 @@ export class FxAccountsKeys {
       if (!userData) {
         return false;
       }
-      if (!userData.verified) {
-        return false;
-      }
       return !!(
         userData.scopedKeys && userData.scopedKeys.hasOwnProperty(scope)
       );
diff --git a/services/fxaccounts/tests/xpcshell/test_accounts.js b/services/fxaccounts/tests/xpcshell/test_accounts.js
@@ -452,8 +452,9 @@ add_task(async function test_hasKeysForScope_not_verified() {
     ...MOCK_ACCOUNT_KEYS,
   };
   const fxa = await MakeFxAccounts({ credentials });
-  // Should return false when user is not verified
-  Assert.ok(!(await fxa.keys.hasKeysForScope(SCOPE_APP_SYNC)));
+  // Should return true even keys exist locally
+  // (Keys are provided during OAuth before the account is marked verified)
+  Assert.ok(await fxa.keys.hasKeysForScope(SCOPE_APP_SYNC));
 });
 
 add_task(async function test_hasKeysForScope_no_keys() {