Skip to content

Add Native Auth V2 SMS MFA support, Fixes AB#3754593 - #2578

Open
Silviu Petrescu (spetrescu84) wants to merge 3 commits into
spetrescu/native-auth-v2-signin-email-otpfrom
spetrescu/native-auth-v2-mfa-sms
Open

Silviu Petrescu (spetrescu84) wants to merge 3 commits into
spetrescu/native-auth-v2-signin-email-otpfrom
spetrescu/native-auth-v2-mfa-sms

Conversation

@spetrescu84

@spetrescu84 Silviu Petrescu (spetrescu84) commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • accept both email and SMS methods in MFARequiredStateV2
  • keep the SMS risk-verification hop internal to Common and expose the existing MFA verification state
  • pass through the server-provided SMS code length, channel, and masked destination
  • retain existing V2 email coverage and add separate password-first and email-OTP-first SMS MFA coverage
  • avoid constructing submit-code command parameters for an unknown flow scenario
  • update the Common submodule to 6d286ad98

Testing

  • Native Auth V2 sign-in and interface unit tests
  • local sample application build using the included MSAL checkout

Depends on #2577 and AzureAD/microsoft-authentication-library-common-for-android#3253.

Fixes AB#3754593

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: e495d4e1-5dbf-4c2c-8f13-5d209a136be8
@github-actions

Copy link
Copy Markdown

❌ Work item link check failed. Description does not contain AB#{ID}.

Click here to Learn more.

@github-actions github-actions Bot added the msal label Sep 10, 2026
@spetrescu84 Silviu Petrescu (spetrescu84) changed the title Add Native Auth V2 SMS MFA support Add Native Auth V2 SMS MFA support, Fixes AB#3754593 Sep 14, 2026
…cu/native-auth-v2-mfa-sms

# Conflicts:
#	common
@spetrescu84 Silviu Petrescu (spetrescu84) added the changelog:none No changelog entry or version increment is required. label Sep 14, 2026
…cu/native-auth-v2-mfa-sms

# Conflicts:
#	common
#	msal/src/main/java/com/microsoft/identity/nativeauth/statemachine/states/CodeRequiredStateV2.kt
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog:none No changelog entry or version increment is required. msal

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant