Fix iOS AppHang: dispatch getNetworkInfo off main thread

CHANGELOG.md

@@ -1,3 +1,9 @@
+## 0.0.3
+
+* Fix iOS AppHang: dispatch `SCNetworkReachabilityGetFlags` and `CTTelephonyNetworkInfo()`
+  to a background queue to avoid blocking the main thread during XPC calls.
+* Fix typo: `transport_subtype` for WiFi was `"unknwon"`, now `"unknown"`.
+
 ## 0.0.1
 
 * Initial release supporting the fields:

ios/Classes/FlutterNetworkCapabilitiesPlugin.swift

@@ -12,11 +12,13 @@ public class FlutterNetworkCapabilitiesPlugin: NSObject, FlutterPlugin {
     }
 
     public func handle(_ call: FlutterMethodCall, result: @escaping FlutterResult) {
-        if (call.method != "getNetworkInfo") {
+        if call.method != "getNetworkInfo" {
             result(FlutterMethodNotImplemented)
             return
         }
-        result(getNetworkInfoResult())
+        DispatchQueue.global(qos: .utility).async {
+            result(self.getNetworkInfoResult())
+        }
     }
 
     private func getNetworkInfoResult() -> [String : String] {
@@ -48,7 +50,7 @@ public class FlutterNetworkCapabilitiesPlugin: NSObject, FlutterPlugin {
                 }
             } else {
                 networkInfoResult["transport_type"] = "wifi"
-                networkInfoResult["transport_subtype"] = "unknwon"
+                networkInfoResult["transport_subtype"] = "unknown"
             }
 
             return networkInfoResult

lib/flutter_network_capabilities_method_channel.dart

@@ -12,13 +12,17 @@ class MethodChannelFlutterNetworkCapabilities
 
   @override
   Future<Map<String, String>> getNetworkInfo() async {
-    final networkInfo = await methodChannel
-        .invokeMapMethod<String, String>('getNetworkInfo');
+    try {
+      final networkInfo = await methodChannel
+          .invokeMapMethod<String, String>('getNetworkInfo');
 
-    if (networkInfo == null) {
+      if (networkInfo == null) {
+        return {};
+      }
+
+      return networkInfo;
+    } catch (e) {
       return {};
     }
-
-    return networkInfo;
   }
 }

pubspec.yaml

@@ -1,6 +1,6 @@
 name: flutter_network_capabilities
 description: This package allows monitoring of Network Capabilities from Android and iOS devices.
-version: 0.0.2
+version: 0.0.3
 homepage: https://nubank.com/ 
 repository: https://github.com/nubank/flutter-network-capabilities
 

test/flutter_network_capabilities_method_channel_test.dart

@@ -21,4 +21,18 @@ void main() {
   test('getNetworkInfo', () async {
     expect(await platform.getNetworkInfo(), {"testing": "testing"});
   });
+
+  test('getNetworkInfo returns empty map when native returns null', () async {
+    channel.setMockMethodCallHandler((MethodCall methodCall) async => null);
+
+    expect(await platform.getNetworkInfo(), isEmpty);
+  });
+
+  test('getNetworkInfo returns empty map on PlatformException', () async {
+    channel.setMockMethodCallHandler((MethodCall methodCall) async {
+      throw PlatformException(code: 'UNAVAILABLE');
+    });
+
+    expect(await platform.getNetworkInfo(), isEmpty);
+  });
 }