Skip to content

Commit 4d7edd4

Browse files
committed
Only fetch reCAPTCHA v2 token when FAKE_TOKEN (#8493)
1 parent 30940dd commit 4d7edd4

File tree

1 file changed

+6
-18
lines changed
  • packages/auth/src/platform_browser/strategies

1 file changed

+6
-18
lines changed

packages/auth/src/platform_browser/strategies/phone.ts

Lines changed: 6 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -267,12 +267,8 @@ export async function _verifyPhoneNumber(
267267
authInstance: AuthInternal,
268268
request: StartPhoneMfaEnrollmentRequest
269269
) => {
270-
// If reCAPTCHA Enterprise token is empty or "NO_RECAPTCHA", fetch reCAPTCHA v2 token and inject into request.
271-
if (
272-
!request.phoneEnrollmentInfo.captchaResponse ||
273-
request.phoneEnrollmentInfo.captchaResponse.length === 0 ||
274-
request.phoneEnrollmentInfo.captchaResponse === FAKE_TOKEN
275-
) {
270+
// If reCAPTCHA Enterprise token is FAKE_TOKEN, fetch reCAPTCHA v2 token and inject into request.
271+
if (request.phoneEnrollmentInfo.captchaResponse === FAKE_TOKEN) {
276272
_assert(
277273
verifier?.type === RECAPTCHA_VERIFIER_TYPE,
278274
authInstance,
@@ -329,12 +325,8 @@ export async function _verifyPhoneNumber(
329325
authInstance: AuthInternal,
330326
request: StartPhoneMfaSignInRequest
331327
) => {
332-
// If reCAPTCHA Enterprise token is empty or "NO_RECAPTCHA", fetch reCAPTCHA v2 token and inject into request.
333-
if (
334-
!request.phoneSignInInfo.captchaResponse ||
335-
request.phoneSignInInfo.captchaResponse.length === 0 ||
336-
request.phoneSignInInfo.captchaResponse === FAKE_TOKEN
337-
) {
328+
// If reCAPTCHA Enterprise token is FAKE_TOKEN, fetch reCAPTCHA v2 token and inject into request.
329+
if (request.phoneSignInInfo.captchaResponse === FAKE_TOKEN) {
338330
_assert(
339331
verifier?.type === RECAPTCHA_VERIFIER_TYPE,
340332
authInstance,
@@ -380,12 +372,8 @@ export async function _verifyPhoneNumber(
380372
authInstance: AuthInternal,
381373
request: SendPhoneVerificationCodeRequest
382374
) => {
383-
// If reCAPTCHA Enterprise token is empty or "NO_RECAPTCHA", fetch reCAPTCHA v2 token and inject into request.
384-
if (
385-
!request.captchaResponse ||
386-
request.captchaResponse.length === 0 ||
387-
request.captchaResponse === FAKE_TOKEN
388-
) {
375+
// If reCAPTCHA Enterprise token is FAKE_TOKEN, fetch reCAPTCHA v2 token and inject into request.
376+
if (request.captchaResponse === FAKE_TOKEN) {
389377
_assert(
390378
verifier?.type === RECAPTCHA_VERIFIER_TYPE,
391379
authInstance,

0 commit comments

Comments
 (0)