fix: nwpath thread safety issue (#24)

Co-authored-by: xiaoweii <[email protected]>
awslabs · Jul 31, 2023 · ceecc65 · ceecc65
1 parent db64731
commit ceecc65
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/Sources/Clickstream/Network/NetworkMonitor.swift b/Sources/Clickstream/Network/NetworkMonitor.swift
@@ -16,17 +16,16 @@ protocol NetworkMonitor: AnyObject {
 }
 
 var currentNetWorkType: String = NetWorkType.UnKnow
+var currentIsOnline: Bool = true
 
 extension NWPathMonitor: NetworkMonitor {
-    var isOnline: Bool {
-        currentPath.status == .satisfied
-    }
-
+    var isOnline: Bool { currentIsOnline }
     var netWorkType: String { currentNetWorkType }
 
     func startMonitoring(using queue: DispatchQueue) {
         start(queue: queue)
         pathUpdateHandler = { path in
+            currentIsOnline = path.status == .satisfied
             if path.usesInterfaceType(.wifi) {
                 currentNetWorkType = NetWorkType.Wifi
             } else if path.usesInterfaceType(.cellular) {

diff --git a/Sources/Clickstream/PackageInfo.swift b/Sources/Clickstream/PackageInfo.swift
@@ -8,5 +8,5 @@
 enum PackageInfo {
     /// the clickstream analytics iOS sdk version
     /// note: update and align the version with new tag version before committing new tag
-    static let version = "0.5.1"
+    static let version = "0.5.2"
 }