feat(profile-sync): add MFA authentication services by mathieuartu · Pull Request #10265 · MetaMask/core

mathieuartu · 2026-09-16T13:42:33Z

Explanation

Wires the MFA HTTP client into the SRP JWT auth SDK on top of the validation foundation.

mfa/services calls /api/v2/mfa/* (enroll, enroll complete, verify, verify complete, credentials), validates request/response bodies, maps server codes to MfaError subclasses, and handles OTP cooldown / Retry-After.
SRPJwtBearerAuth exposes begin/complete enrollment and verification, credential listing, and assertion-to-token exchange.
Public JwtBearerAuth forwards those methods and rejects non-SRP auth types.
Nock fixtures and unit tests cover happy paths and error mapping.

Clients still go through AuthenticationController in later PRs; this layer is not UI-facing.

References

Depends on #10264. Follow-ups: #10266, #10267. Stack: stack #10268.
Related to: https://consensyssoftware.atlassian.net/browse/MUL-2261

Checklist

I've updated the test suite for new or updated code as appropriate
I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
I've communicated my changes to consumers by updating changelogs for packages I've changed
I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

Note

High Risk
Touches authentication, step-up tokens, and passkey/OTP handling; mistakes could weaken session elevation or mishandle credentials, though coverage is extensive.

Overview
Adds passkey and email OTP MFA to the profile-sync JWT auth SDK: a new HTTP layer calls /api/v2/mfa/* (enroll, complete, verify, credentials), validates payloads, maps server codes to MfaError subclasses, and handles OTP cooldown plus Retry-After.

SRPJwtBearerAuth and public JwtBearerAuth expose enrollment/step-up flows, credential listing, and exchangeMfaAssertion (AAL2 JWT → elevated access token via existing OIDC). MFA is SRP-only; email enrollment uses a separate email option from entropySourceId.

Schemas mark challenges, OTP codes, passkey payloads, and tokens with sensitive(). Tests, nock fixtures, changelog, and SDK exports for MFA types are included.

^{Reviewed by Cursor Bugbot for commit 56b3671. Bugbot is set up for automated code reviews on this repo. Configure here.}

cursor

Stale Bugbot comment from a previous run.

cursor

Stale Bugbot comment from a previous run.

cursor

Stale Bugbot comment from a previous run.

## Explanation Adds the MFA validation layer in `@metamask/profile-sync-controller` that later PRs in this stack consume. - Superstruct schemas and inferred types for auth-service MFA responses, WebAuthn ceremony payloads, and controller-boundary requests (passkey and email OTP only). - `MfaError` family with a stable enumerable `mfaCode`, plus helpers that survive JSON-RPC serialization (`getMfaErrorCode`, `isMfaError`, `getMfaRetryAfterMs`). - Shared `decodeJwtPayload` (no signature verification) and slightly stricter login JWT `exp` handling. - Expanded `HTTP_STATUS_CODES` for later MFA error mapping. This PR does not call MFA HTTP endpoints or change controller behavior. ## References Stacked under [stack #10268](https://github.com/MetaMask/core/pull/10268). Follow-ups: MetaMask#10265, MetaMask#10266, MetaMask#10267. Related to: https://consensyssoftware.atlassian.net/browse/MUL-2260 ## Checklist - [x] I've updated the test suite for new or updated code as appropriate - [x] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate - [x] I've communicated my changes to consumers by [updating changelogs for packages I've changed](https://github.com/MetaMask/core/tree/main/docs/processes/updating-changelogs.md) - [ ] I've introduced [breaking changes](https://github.com/MetaMask/core/tree/main/docs/processes/breaking-changes.md) in this PR and have prepared draft pull requests for clients and consumer packages to resolve them  --- > [!NOTE] > **Medium Risk** > Touches authentication-adjacent validation, JWT claim handling, and structured MFA errors, though runtime sign-in/MFA flows are unchanged until follow-up PRs. > > **Overview** > Introduces the **MFA validation layer** in `@metamask/profile-sync-controller` for later stack PRs—no MFA HTTP calls or controller behavior changes in this diff. > > Adds Superstruct schemas and inferred types under `authentication-jwt-bearer/mfa` for passkey and email OTP flows: auth-service responses, WebAuthn ceremony payloads, enrollment/step-up requests, and **AAL2 elevated-token** claim parsing (including Hydra `ext` nesting). `assertValidMfaRequest` / `assertValidMfaResponse` map validation failures to `MfaError` with path details. > > Expands **`MfaError`** with typed subclasses, stable enumerable **`mfaCode`**, and helpers (`getMfaErrorCode`, `isMfaError`, `getMfaRetryAfterMs`) that work after JSON-RPC serialization. Adds shared **`decodeJwtPayload`** and refactors login JWT expiry checks to use it with stricter `exp` typing. **`HTTP_STATUS_CODES`** gains `401` and `502` for upcoming error mapping. > > Dependencies: **`@metamask/superstruct`** (runtime), **`@metamask/rpc-errors`** (tests). Changelog updated. > > <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit d6d147b. Bugbot is set up for automated code reviews on this repo. Configure [here](https://www.cursor.com/dashboard/bugbot).</sup>  --------- Co-authored-by: Cursor <cursoragent@cursor.com>

cursor

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 32a03ff. Configure here.}

Co-authored-by: Cursor <cursoragent@cursor.com>

- Recognise 401 before requiring a JSON error body - Map code-less 429s to rate_limited instead of otp_resend_cooldown - Stop parsing the unstable error message for a retry delay - Return only the assertion from verify/complete; drop profile mapping - Tolerate unknown credential statuses and missing email.verified - Export domain types only from the SDK entrypoint Co-authored-by: Cursor <cursoragent@cursor.com>

gantunesr

lgtm

lets wait for @ccharly to approve it too before merging

ccharly

LGTM. I left 1 note about a potential missing sensitive (can be done in the next PR).

Also, as discussed internally, we could potentially enforce typing in some of the input type, so the compiler can enforce it, e.g.: using type: 'passkey' and enforce the field passkey_attestation to be required.

But this can come later too. Might be relevant if we add more credential type in the future!

ccharly · 2026-09-18T11:13:00Z

@@ -241,7 +247,7 @@ export const GetElevatedTokenRequestStruct = object({
 });

 export const ElevatedTokenClaimsStruct = type({


Maybe we could put the entire token as sensitive too? Just in case

(can be done in a follow-up too)

mathieuartu requested review from a team as code owners September 16, 2026 13:42

mathieuartu added this pull request to stack #10268 September 16, 2026 13:42

This was referenced Sep 16, 2026

feat(profile-sync): add MFA validation foundation #10264

Merged

feat(profile-sync): add MFA credential enrollment #10266

Open

cursor Bot reviewed Sep 16, 2026

View reviewed changes

Comment thread packages/profile-sync-controller/src/sdk/authentication-jwt-bearer/mfa/services.ts Outdated

mathieuartu force-pushed the mfa/sdk-services branch from 906f800 to 4ff3dc2 Compare September 16, 2026 14:08

mathieuartu requested review from a team as code owners September 16, 2026 14:27

mathieuartu self-assigned this Sep 16, 2026

mathieuartu added the team-accounts label Sep 16, 2026

cursor Bot reviewed Sep 16, 2026

View reviewed changes

Comment thread packages/assets-controller/src/AssetsController.spam-cleanup.test.ts

cursor Bot reviewed Sep 16, 2026

View reviewed changes

Comment thread packages/profile-sync-controller/src/sdk/authentication-jwt-bearer/mfa/services.ts Outdated

mathieuartu force-pushed the mfa/sdk-services branch from 7872ea7 to 47c2854 Compare September 16, 2026 20:13

mathieuartu force-pushed the mfa/sdk-services branch 2 times, most recently from 3b1793f to ccdc7a1 Compare September 17, 2026 06:53

mathieuartu force-pushed the mfa/sdk-services branch from ccdc7a1 to ac84c3f Compare September 17, 2026 06:54

Base automatically changed from mfa/sdk-foundation to main September 17, 2026 13:39

mathieuartu force-pushed the mfa/sdk-services branch 2 times, most recently from 7625f86 to 55ed40f Compare September 17, 2026 13:41

mathieuartu force-pushed the mfa/sdk-services branch from 55ed40f to 32a03ff Compare September 17, 2026 14:16

cursor Bot reviewed Sep 17, 2026

View reviewed changes

Comment thread packages/profile-sync-controller/src/sdk/authentication-jwt-bearer/flow-srp.ts

mathieuartu force-pushed the mfa/sdk-services branch from ef981db to 9b772b3 Compare September 17, 2026 16:32

gantunesr reviewed Sep 18, 2026

View reviewed changes

Comment thread packages/profile-sync-controller/src/sdk/authentication-jwt-bearer/mfa/services.ts Outdated

gantunesr reviewed Sep 18, 2026

View reviewed changes

Comment thread packages/profile-sync-controller/src/sdk/authentication-jwt-bearer/mfa/services.ts Outdated

mathieuartu and others added 7 commits September 18, 2026 09:23

feat(profile-sync): add MFA authentication services

82e3d31

Co-authored-by: Cursor <cursoragent@cursor.com>

fix: lint and cursor feedback

7335708

fix: remove unneeded comment

16ea2f9

fix: change beginMfaEnrollment shape for two optional args

6f660ff

fix: add sensitive fields to schemas + fix lint

3621d32

fix: rename type and return error message fallback

32dce23

mathieuartu force-pushed the mfa/sdk-services branch from 9b772b3 to 32dce23 Compare September 18, 2026 07:23

ccharly reviewed Sep 18, 2026

View reviewed changes

fix: address PR feedbacks

56b3671

gantunesr approved these changes Sep 18, 2026

View reviewed changes

ccharly approved these changes Sep 18, 2026

View reviewed changes

mathieuartu added this pull request to the merge queue Sep 18, 2026

Merged via the queue into main with commit 8762cfa Sep 18, 2026
136 checks passed

mathieuartu deleted the mfa/sdk-services branch September 18, 2026 12:13

Navigation Menu

Uh oh!

feat(profile-sync): add MFA authentication services - #10265

feat(profile-sync): add MFA authentication services#10265
mathieuartu merged 8 commits into
mainfrom
mfa/sdk-services

mathieuartu commented Sep 16, 2026 •

edited by cursor Bot

Loading

Uh oh!

cursor Bot left a comment •

edited

Loading

Uh oh!

Uh oh!

cursor Bot left a comment •

edited

Loading

Uh oh!

Uh oh!

cursor Bot left a comment •

edited

Loading

Uh oh!

Uh oh!

cursor Bot left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

gantunesr left a comment

Uh oh!

ccharly left a comment

Uh oh!

ccharly Sep 18, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

		@@ -241,7 +247,7 @@ export const GetElevatedTokenRequestStruct = object({
		});

		export const ElevatedTokenClaimsStruct = type({

Uh oh!

Conversation

mathieuartu commented Sep 16, 2026 • edited by cursor Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Explanation

References

Checklist

Uh oh!

cursor Bot left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

cursor Bot left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

cursor Bot left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

gantunesr left a comment

Choose a reason for hiding this comment

Uh oh!

ccharly left a comment

Choose a reason for hiding this comment

Uh oh!

ccharly Sep 18, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

mathieuartu commented Sep 16, 2026 •

edited by cursor Bot

Loading

cursor Bot left a comment •

edited

Loading

cursor Bot left a comment •

edited

Loading

cursor Bot left a comment •

edited

Loading