diff options
| author | Andrej Mihajlov <and@mullvad.net> | 2022-01-31 13:07:55 +0100 |
|---|---|---|
| committer | Andrej Mihajlov <and@mullvad.net> | 2022-02-01 10:54:15 +0100 |
| commit | 9095135ee78eebb155bb1e71d5304405f35f9b96 (patch) | |
| tree | 95057b44340a80f24c4561e3dfeb1298af2b7fb9 | |
| parent | e3548fe7a570d0531afa76f2ee7622c1bac5be27 (diff) | |
| download | mullvadvpn-9095135ee78eebb155bb1e71d5304405f35f9b96.tar.xz mullvadvpn-9095135ee78eebb155bb1e71d5304405f35f9b96.zip | |
UpdateAddressCacheOperation: add missing finish() call
| -rw-r--r-- | ios/MullvadVPN/AddressCache/UpdateAddressCacheOperation.swift | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/ios/MullvadVPN/AddressCache/UpdateAddressCacheOperation.swift b/ios/MullvadVPN/AddressCache/UpdateAddressCacheOperation.swift index 4ca1d4d3c7..e1214a73fc 100644 --- a/ios/MullvadVPN/AddressCache/UpdateAddressCacheOperation.swift +++ b/ios/MullvadVPN/AddressCache/UpdateAddressCacheOperation.swift @@ -67,7 +67,7 @@ extension AddressCache { private func startUpdate() { guard !isCancelled else { - finish(with: .cancelled) + completeOperation(with: .cancelled) return } @@ -75,7 +75,7 @@ extension AddressCache { let nextUpdate = Date(timeInterval: updateInterval, since: lastUpdate) guard nextUpdate <= Date() else { - finish(with: .throttled(lastUpdate)) + completeOperation(with: .throttled(lastUpdate)) return } @@ -87,27 +87,29 @@ extension AddressCache { self.store.setEndpoints(newEndpoints) { error in self.queue.async { if let error = error { - self.finish(with: .failure(error)) + self.completeOperation(with: .failure(error)) } else { - self.finish(with: .success) + self.completeOperation(with: .success) } } } case .failure(let error): if case URLError.cancelled = error { - self.finish(with: .cancelled) + self.completeOperation(with: .cancelled) } else { - self.finish(with: .failure(error)) + self.completeOperation(with: .failure(error)) } } } } } - private func finish(with result: CacheUpdateResult) { + private func completeOperation(with result: CacheUpdateResult) { completionHandler?(result) completionHandler = nil + + finish() } } } |
