passepartoutvpn/passepartout-apple
1
0
Fork 0
mirror of https://github.com/passepartoutvpn/passepartout-apple.git synced 2025-01-17 22:19:08 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix similar completionHandler bugs

Browse Source 
See previous commit
This commit is contained in:
Davide De Rosa 2019-03-19 17:27:42 +01:00
parent 9a67918dc0
commit 877dd4d990
1 changed files with 19 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
Passepartout/Sources/VPN/StandardVPNProvider.swift
View File

@ -91,19 +91,23 @@ public class StandardVPNProvider: VPNProvider {
fatalError("Not a NetworkExtensionVPNConfiguration")
}
find(with: bundleIdentifier) {
self.manager = $0
self.manager?.protocolConfiguration = configuration.protocolConfiguration
self.manager?.onDemandRules = configuration.onDemandRules
self.manager?.isOnDemandEnabled = true
self.manager?.isEnabled = true
self.manager?.saveToPreferences { (error) in
guard let manager = $0 else {
completionHandler?(nil)
return
}
self.manager = manager
manager.protocolConfiguration = configuration.protocolConfiguration
manager.onDemandRules = configuration.onDemandRules
manager.isOnDemandEnabled = true
manager.isEnabled = true
manager.saveToPreferences { (error) in
guard error == nil else {
self.manager?.isOnDemandEnabled = false
self.manager?.isEnabled = false
manager.isOnDemandEnabled = false
manager.isEnabled = false
completionHandler?(error)
return
}
self.manager?.loadFromPreferences { (error) in
manager.loadFromPreferences { (error) in
completionHandler?(error)
}
}
@ -153,8 +157,12 @@ public class StandardVPNProvider: VPNProvider {
public func uninstall(completionHandler: (() -> Void)?) {
find(with: bundleIdentifier) { (manager) in
manager?.connection.stopVPNTunnel()
manager?.removeFromPreferences { (error) in
guard let manager = manager else {
completionHandler?()
return
}
manager.connection.stopVPNTunnel()
manager.removeFromPreferences { (error) in
self.manager = nil
completionHandler?()
}