]> git.ipfire.org Git - thirdparty/wireguard-apple.git/commitdiff
WireGuardKit: Remove handleLogLine from WireGuardAdapter
authorAndrej Mihajlov <and@mullvad.net>
Wed, 2 Dec 2020 12:53:16 +0000 (13:53 +0100)
committerAndrej Mihajlov <and@mullvad.net>
Thu, 3 Dec 2020 12:22:52 +0000 (13:22 +0100)
Signed-off-by: Andrej Mihajlov <and@mullvad.net>
WireGuardKit/Sources/WireGuardKit/WireGuardAdapter.swift

index 12f7d2373ac316058849643720f4893c13bcd5a5..c5a6e2c5dd285ca3ef7538b872a98460af456d9a 100644 (file)
@@ -239,7 +239,7 @@ public class WireGuardAdapter {
             let swiftString = String(cString: message).trimmingCharacters(in: .newlines)
             let tunnelLogLevel = WireGuardLogLevel(rawValue: logLevel) ?? .debug
 
-            unretainedSelf.handleLogLine(level: tunnelLogLevel, message: swiftString)
+            unretainedSelf.logHandler(tunnelLogLevel, swiftString)
         }
     }
 
@@ -316,33 +316,20 @@ public class WireGuardAdapter {
         return resolvedEndpoints
     }
 
-    /// Private helper to pass the logs coming from WireGuard to
-    /// - Parameters:
-    ///   - level: log level
-    ///   - message: message
-    private func handleLogLine(level: WireGuardLogLevel, message: String) {
-        workQueue.async {
-            self.logHandler?(level, message)
-        }
-    }
-
     /// Helper method used by network path monitor.
     /// - Parameter path: new network path
     private func didReceivePathUpdate(path: Network.NWPath) {
+        guard let handle = self.wireguardHandle else { return }
 
-        if let handle = self.wireguardHandle {
-            self.handleLogLine(level: .debug, message: "Network change detected with \(path.status) route and interface order \(path.availableInterfaces)")
+        self.logHandler(.debug, "Network change detected with \(path.status) route and interface order \(path.availableInterfaces)")
 
-            #if os(iOS)
-            if let settingsGenerator = self.settingsGenerator {
-                wgSetConfig(handle, settingsGenerator.endpointUapiConfiguration())
-            }
-
-            // TODO: dynamically turn on or off WireGuard backend when entering airplane mode
-            #endif
-
-            wgBumpSockets(handle)
+        #if os(iOS)
+        if let settingsGenerator = self.settingsGenerator {
+            wgSetConfig(handle, settingsGenerator.endpointUapiConfiguration())
         }
+        #endif
+
+        wgBumpSockets(handle)
     }
 }