Make ApplicationVerifier params optional in Phone Auth APIs

[NhienLam]

Discussion

• Currently, public phone auth related functions require a reCAPTCHA v2 verifier because that’s the only attestation provider we’re supporting. Since we're adding reCAPTCHA Enterprise support and the reCAPTCHA Enterprise ENFORCE flow doesn't require a reCAPTCHA v2 token, we will make ApplicationVerifier parameter optional.
• If ApplicationVerifier is not available in OFF mode or AUDIT fallback flow, we will throw an auth/argument-error.

Testing

Tested the Phone Auth, SMS MFA Enroll, and SMS MFA Sign-In flows with the demo app when ApplicationVerifier is not set

• Enforce mode: Expect sendVerificationCode to be called with a reCAPTCHA Enterprise token
• Audit mode:
  • First, expect sendVerificationCode to be called with a reCAPTCHA Enterprise token
  • If the reCAPTHCA Enterprise token flow fails, expect a fallback to reCAPTCHA v2 token, resulting in an auth/argument-error
• Off mode: Expect a failure with auth/argument-error

[changeset-bot]

⚠️ No Changeset found

Latest commit: c365447

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[google-oss-bot]

Size Report ¹

Affected Products

• @firebase/auth

  Type	Base (56f904c)	Merge (0bdf294)	Diff
  browser	188 kB	189 kB	+174 B (+0.1%)
  esm5	245 kB	245 kB	+174 B (+0.1%)
  module	188 kB	189 kB	+174 B (+0.1%)
  react-native	207 kB	207 kB	+174 B (+0.1%)

• @firebase/auth/internal

  Type	Base (56f904c)	Merge (0bdf294)	Diff
  browser	199 kB	199 kB	+174 B (+0.1%)
  esm5	258 kB	258 kB	+174 B (+0.1%)
  module	199 kB	199 kB	+174 B (+0.1%)

• bundle

  Type	Base (56f904c)	Merge (0bdf294)	Diff
  auth (Phone)	93.3 kB	93.4 kB	+118 B (+0.1%)

• firebase

  Type	Base (56f904c)	Merge (0bdf294)	Diff
  firebase-auth-compat.js	143 kB	143 kB	+60 B (+0.0%)
  firebase-auth.js	155 kB	155 kB	+60 B (+0.0%)
  firebase-compat.js	789 kB	789 kB	+60 B (+0.0%)

Test Logs

• Base (56f904c): https://github.com/firebase/firebase-js-sdk/actions/runs/9674946662
• Merge (0bdf294): https://github.com/firebase/firebase-js-sdk/actions/runs/9942882949

1. https://storage.googleapis.com/firebase-sdk-metric-reports/cNpPDmGfLm.html

[google-oss-bot]

Size Analysis Report ¹

Affected Products

• @firebase/auth

  • PhoneAuthProvider

    Size

    Type	Base (56f904c)	Merge (0bdf294)	Diff
    size	50.4 kB	50.5 kB	+118 B (+0.2%)
    size-with-ext-deps	71.9 kB	72.0 kB	+118 B (+0.2%)

  • linkWithPhoneNumber

    Size

    Type	Base (56f904c)	Merge (0bdf294)	Diff
    size	51.1 kB	51.2 kB	+118 B (+0.2%)
    size-with-ext-deps	72.7 kB	72.8 kB	+118 B (+0.2%)

  • reauthenticateWithPhoneNumber

    Size

    Type	Base (56f904c)	Merge (0bdf294)	Diff
    size	51.7 kB	51.9 kB	+118 B (+0.2%)
    size-with-ext-deps	73.3 kB	73.4 kB	+118 B (+0.2%)

  • signInWithPhoneNumber

    Size

    Type	Base (56f904c)	Merge (0bdf294)	Diff
    size	51.5 kB	51.6 kB	+118 B (+0.2%)
    size-with-ext-deps	73.1 kB	73.2 kB	+118 B (+0.2%)

Test Logs

• Base (56f904c): https://github.com/firebase/firebase-js-sdk/actions/runs/9674946662
• Merge (0bdf294): https://github.com/firebase/firebase-js-sdk/actions/runs/9942882949

1. https://storage.googleapis.com/firebase-sdk-metric-reports/cezDCcJtfw.html

[Xiaoshouzi-gh]

LGTM