fix: pass autoOTPVerify only to necessary functions

invertase · Nov 19, 2024 · 5a96645 · 5a96645
1 parent 57fb872
commit 5a96645
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 19 deletions.
diff --git a/.../auth/android/src/main/java/io/invertase/firebase/auth/ReactNativeFirebaseAuthModule.java b/.../auth/android/src/main/java/io/invertase/firebase/auth/ReactNativeFirebaseAuthModule.java
@@ -984,8 +984,8 @@ public void signInWithPhoneNumber(
       String appName,
       final String phoneNumber,
       final boolean forceResend,
-      final Promise promise,
-      final boolean autoOTPVerify) {
+      final boolean autoOTPVerify,
+      final Promise promise) {
     Log.d(TAG, "signInWithPhoneNumber");
     FirebaseApp firebaseApp = FirebaseApp.getInstance(appName);
     final FirebaseAuth firebaseAuth = FirebaseAuth.getInstance(firebaseApp);
@@ -1126,8 +1126,9 @@ public void verifyPhoneNumberWithMultiFactorInfo(
       final String appName,
       final String hintUid,
       final String sessionKey,
-      final Promise promise,
-      final boolean autoOTPVerify) {
+      final boolean autoOTPVerify,
+      final Promise promise
+      ) {
     final MultiFactorResolver resolver = mCachedResolvers.get(sessionKey);
     if (resolver == null) {
       // See https://firebase.google.com/docs/reference/node/firebase.auth.multifactorresolver for
@@ -1199,8 +1200,9 @@ public void verifyPhoneNumberForMultiFactor(
       final String appName,
       final String phoneNumber,
       final String sessionKey,
-      final Promise promise,
-      final boolean autoOTPVerify) {
+      final boolean autoOTPVerify,
+      final Promise promise
+      ) {
     final MultiFactorSession multiFactorSession = mMultiFactorSessions.get(sessionKey);
     if (multiFactorSession == null) {
       rejectPromiseWithCodeAndMessage(

diff --git a/packages/auth/lib/PhoneAuthListener.js b/packages/auth/lib/PhoneAuthListener.js
@@ -21,13 +21,12 @@ import NativeFirebaseError from '@react-native-firebase/app/lib/internal/NativeF
 let REQUEST_ID = 0;
 
 export default class PhoneAuthListener {
-  constructor(auth, phoneNumber, timeout, forceResend, autoOTPVerify) {
+  constructor(auth, phoneNumber, timeout, forceResend) {
     this._auth = auth;
     this._reject = null;
     this._resolve = null;
     this._promise = null;
     this._jsStack = new Error().stack;
-    this._autoOTPVerify = autoOTPVerify;
 
     this._timeout = timeout || 20;
     this._phoneAuthRequestId = REQUEST_ID++;
@@ -56,7 +55,6 @@ export default class PhoneAuthListener {
         this._phoneAuthRequestId + '',
         this._timeout,
         this._forceResending,
-        this._autoOTPVerify,
       );
     }
 

diff --git a/packages/auth/lib/index.js b/packages/auth/lib/index.js
@@ -294,30 +294,31 @@ class FirebaseAuthModule extends FirebaseModule {
   verifyPhoneNumber(phoneNumber, autoVerifyTimeoutOrForceResend, forceResend) {
     let _forceResend = forceResend;
     let _autoVerifyTimeout = 60;
-    let _autoOTPVerify = this._settings.autoOTPVerify;
 
     if (isBoolean(autoVerifyTimeoutOrForceResend)) {
       _forceResend = autoVerifyTimeoutOrForceResend;
     } else {
       _autoVerifyTimeout = autoVerifyTimeoutOrForceResend;
     }
 
-    return new PhoneAuthListener(
-      this,
-      phoneNumber,
-      _autoVerifyTimeout,
-      _forceResend,
-      _autoOTPVerify,
-    );
+    return new PhoneAuthListener(this, phoneNumber, _autoVerifyTimeout, _forceResend);
   }
 
   verifyPhoneNumberWithMultiFactorInfo(multiFactorHint, session) {
-    return this.native.verifyPhoneNumberWithMultiFactorInfo(multiFactorHint.uid, session);
+    return this.native.verifyPhoneNumberWithMultiFactorInfo(
+      multiFactorHint.uid,
+      session,
+      this._settings.autoOTPVerify,
+    );
   }
 
   verifyPhoneNumberForMultiFactor(phoneInfoOptions) {
     const { phoneNumber, session } = phoneInfoOptions;
-    return this.native.verifyPhoneNumberForMultiFactor(phoneNumber, session);
+    return this.native.verifyPhoneNumberForMultiFactor(
+      phoneNumber,
+      session,
+      this._settings.autoOTPVerify,
+    );
   }
 
   resolveMultiFactorSignIn(session, verificationId, verificationCode) {