Merge pull request #124 from rob-patchett/premature_connect_status

Conclude connection success only once the system starts using the tunnel
passepartoutvpn · Oct 17, 2019 · 5866c37 · 5866c37
2 parents 9619d21 + bdf34f8
commit 5866c37
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/TunnelKit/Sources/Protocols/OpenVPN/AppExtension/OpenVPNTunnelProvider.swift b/TunnelKit/Sources/Protocols/OpenVPN/AppExtension/OpenVPNTunnelProvider.swift
@@ -484,8 +484,6 @@ extension OpenVPNTunnelProvider: OpenVPNSessionDelegate {
 
     /// :nodoc:
     public func sessionDidStart(_ session: OpenVPNSession, remoteAddress: String, options: OpenVPN.Configuration) {
-        reasserting = false
-
         log.info("Session did start")
 
         log.info("Returned ifconfig parameters:")
@@ -522,6 +520,8 @@ extension OpenVPNTunnelProvider: OpenVPNSessionDelegate {
         }
 
         bringNetworkUp(remoteAddress: remoteAddress, localOptions: session.configuration, options: options) { (error) in
+            self.reasserting = false
+
             if let error = error {
                 log.error("Failed to configure tunnel: \(error)")
                 self.pendingStartHandler?(error)