Skip to content

Commit e2847db

Browse files
authored
Only fetch reCAPTCHA v2 token when FAKE_TOKEN (#8493)
1 parent 41a5f9f commit e2847db

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

+6-18
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)