fix(firebase_dart_flutter): fixes error for phone auth and mfa

Fixes: "One of RECEIVER_EXPORTED or RECEIVER_NOT_EXPORTED should be specified"

Author: rbellens

packages/firebase_dart_flutter/android/build.gradle

@@ -39,7 +39,7 @@ apply plugin: 'com.android.library'
 android {
     namespace 'be.appsup.firebase_dart_flutter'
 
-    compileSdkVersion 29
+    compileSdkVersion 34
 
     defaultConfig {
         minSdkVersion 16

packages/firebase_dart_flutter/android/src/main/java/be/appsup/firebase_dart_flutter/FirebaseDartFlutterPlugin.java

@@ -9,6 +9,8 @@
 import android.content.pm.Signature;
 import android.net.Uri;
 import android.os.Bundle;
+import android.os.Build;
+
 
 import androidx.annotation.NonNull;
 
@@ -96,22 +98,26 @@ public void onMethodCall(@NonNull MethodCall call, @NonNull final Result result)
             }
             break;
         case "getAuthResult":
-            binding.getApplicationContext().registerReceiver(new BroadcastReceiver() {
+            BroadcastReceiver receiver = new BroadcastReceiver() {
                 @Override
                 public void onReceive(Context context, Intent intent) {
                     result.success(bundleToMap(intent.getExtras()));
                     binding.getApplicationContext().unregisterReceiver(this);
                 }
-            }, new IntentFilter(ACTION_AUTH_RECEIVED));
+            };
+            IntentFilter filter = new IntentFilter(ACTION_AUTH_RECEIVED);
+            binding.getApplicationContext().registerReceiver(receiver, filter, Context.RECEIVER_EXPORTED);
             break;
         case "getVerifyResult":
-            binding.getApplicationContext().registerReceiver(new BroadcastReceiver() {
+            receiver = new BroadcastReceiver() {
                 @Override
                 public void onReceive(Context context, Intent intent) {
                     result.success(bundleToMap(intent.getExtras()));
                     binding.getApplicationContext().unregisterReceiver(this);
                 }
-            }, new IntentFilter(ACTION_RECAPTCHA_RECEIVED));
+            };
+            filter = new IntentFilter(ACTION_RECAPTCHA_RECEIVED);
+            binding.getApplicationContext().registerReceiver(receiver, filter, Context.RECEIVER_EXPORTED);
             break;
         case "isGooglePlayServicesAvailable":
             int v = GoogleApiAvailability.getInstance().isGooglePlayServicesAvailable(binding.getApplicationContext(), 12451000);
@@ -173,8 +179,7 @@ public void onFailure(@NonNull Exception e) {
         }
     });
 
-
-    binding.getApplicationContext().registerReceiver(new BroadcastReceiver() {
+    BroadcastReceiver receiver = new BroadcastReceiver() {
         @Override
         public void onReceive(Context context, Intent intent) {
             Bundle extras = intent.getExtras();
@@ -193,7 +198,9 @@ public void onReceive(Context context, Intent intent) {
             }
             binding.getApplicationContext().unregisterReceiver(this);
         }
-    }, new IntentFilter(SmsRetriever.SMS_RETRIEVED_ACTION));
+    };
+    IntentFilter filter = new IntentFilter(SmsRetriever.SMS_RETRIEVED_ACTION);
+    binding.getApplicationContext().registerReceiver(receiver, filter, Context.RECEIVER_EXPORTED);
   }
 
   private String getAppSignatureHash() throws PackageManager.NameNotFoundException, NoSuchAlgorithmException {

packages/firebase_dart_flutter/lib/src/auth_handlers.dart

@@ -253,9 +253,12 @@ class AndroidSmsRetriever extends SmsRetriever {
   @override
   Future<String?> retrieveSms() {
     if (!kIsWeb && platform_info.Platform.instance.android) {
-      return Future(() async {
+      return Future<String?>(() async {
         var v = (await _channel.invokeMethod<String>('retrieveSms'))!;
         return v;
+      }).catchError((e, tr) {
+        Logger('firebase_dart_flutter').warning('Failed retrieving SMS', e, tr);
+        return null;
       });
     }
     return Future.value();