diff options
| author | Andrej Mihajlov <and@mullvad.net> | 2020-07-21 15:50:32 +0300 |
|---|---|---|
| committer | Andrej Mihajlov <and@mullvad.net> | 2020-07-22 18:35:43 +0300 |
| commit | efcd0d08dea67df8917677332362aa54b1e7cf87 (patch) | |
| tree | cfc180a3590b8eef1590413e0af6d4ede631e847 | |
| parent | c5732199cdeaff5faaa41add81b4354fb0a62fd1 (diff) | |
| download | mullvadvpn-efcd0d08dea67df8917677332362aa54b1e7cf87.tar.xz mullvadvpn-efcd0d08dea67df8917677332362aa54b1e7cf87.zip | |
Fix: forgot to call resume() on key rotation task
| -rw-r--r-- | ios/MullvadVPN/AutomaticKeyRotationManager.swift | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/ios/MullvadVPN/AutomaticKeyRotationManager.swift b/ios/MullvadVPN/AutomaticKeyRotationManager.swift index a58940d76f..b9f706d11d 100644 --- a/ios/MullvadVPN/AutomaticKeyRotationManager.swift +++ b/ios/MullvadVPN/AutomaticKeyRotationManager.swift @@ -62,7 +62,7 @@ class AutomaticKeyRotationManager { private var isAutomaticRotationEnabled = false /// A REST request for replacing the key on server - private var request: URLSessionTask? + private var dataTask: URLSessionTask? /// A variable backing the `eventHandler` public property private var _eventHandler: ((KeyRotationResult) -> Void)? @@ -106,8 +106,8 @@ class AutomaticKeyRotationManager { self.isAutomaticRotationEnabled = false - self.request?.cancel() - self.request = nil + self.dataTask?.cancel() + self.dataTask = nil self.timerSource?.cancel() @@ -139,9 +139,11 @@ class AutomaticKeyRotationManager { switch result { case .success(let newTask): - self.request = newTask + self.dataTask = newTask + newTask.resume() + case .failure(let error): - self.request = nil + self.dataTask = nil self.didCompleteKeyRotation(result: .failure(.rest(error))) } } else { |
