Merge pull request #238 from santhoshvai/fix-android-34

fix(android): specify export mode on BroadcastReceivers

Author: zxcpoiu

android/build.gradle

@@ -10,7 +10,7 @@ android {
         namespace "com.zxcpoiu.incallmanager"
     }
 
-    compileSdkVersion safeExtGet('compileSdkVersion', 30)
+    compileSdkVersion safeExtGet('compileSdkVersion', 33)
     buildToolsVersion safeExtGet('buildToolsVersion', "30.0.2")
 
     defaultConfig {

android/src/main/java/com/zxcpoiu/incallmanager/AppRTC/AppRTCBluetoothManager.java

@@ -461,7 +461,11 @@ protected AudioManager getAudioManager(Context context) {
     return (AudioManager) context.getSystemService(Context.AUDIO_SERVICE);
   }
   protected void registerReceiver(BroadcastReceiver receiver, IntentFilter filter) {
-    apprtcContext.registerReceiver(receiver, filter);
+    if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU) {
+      apprtcContext.registerReceiver(receiver, filter, Context.RECEIVER_NOT_EXPORTED);
+    } else {
+      apprtcContext.registerReceiver(receiver, filter);
+    }
   }
   protected void unregisterReceiver(BroadcastReceiver receiver) {
     apprtcContext.unregisterReceiver(receiver);

android/src/main/java/com/zxcpoiu/incallmanager/InCallManagerModule.java

@@ -292,12 +292,7 @@ public void onReceive(Context context, Intent intent) {
                     }
                 }
             };
-            ReactContext reactContext = getReactApplicationContext();
-            if (reactContext != null) {
-                reactContext.registerReceiver(wiredHeadsetReceiver, filter);
-            } else {
-                Log.d(TAG, "startWiredHeadsetEvent() reactContext is null");
-            }
+            this.registerReceiver(wiredHeadsetReceiver, filter);
         }
     }
 
@@ -322,12 +317,7 @@ public void onReceive(Context context, Intent intent) {
                     }
                 }
             };
-            ReactContext reactContext = getReactApplicationContext();
-            if (reactContext != null) {
-                reactContext.registerReceiver(noisyAudioReceiver, filter);
-            } else {
-                Log.d(TAG, "startNoisyAudioEvent() reactContext is null");
-            }
+            this.registerReceiver(noisyAudioReceiver, filter);
         }
     }
 
@@ -389,12 +379,8 @@ public void onReceive(Context context, Intent intent) {
                     }
                 }
             };
-            ReactContext reactContext = getReactApplicationContext();
-            if (reactContext != null) {
-                reactContext.registerReceiver(mediaButtonReceiver, filter);
-            } else {
-                Log.d(TAG, "startMediaButtonEvent() reactContext is null");
-            }
+
+            this.registerReceiver(mediaButtonReceiver, filter);
         }
     }
 
@@ -1669,7 +1655,16 @@ public AudioDevice getSelectedAudioDevice() {
 
     /** Helper method for receiver registration. */
     private void registerReceiver(BroadcastReceiver receiver, IntentFilter filter) {
-        getReactApplicationContext().registerReceiver(receiver, filter);
+        final ReactContext reactContext = getReactApplicationContext();
+        if (reactContext != null) {
+            if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU) {
+                reactContext.registerReceiver(receiver, filter, Context.RECEIVER_NOT_EXPORTED);
+            } else {
+                reactContext.registerReceiver(receiver, filter);
+            }
+        }  else {
+            Log.d(TAG, "registerReceiver() reactContext is null");
+        }
     }
 
     /** Helper method for unregistration of an existing receiver. */