From 8e221904e93a4350df00bb77a1275159e0a804e4 Mon Sep 17 00:00:00 2001 From: Jamil Date: Sun, 5 Jan 2025 15:05:48 -0800 Subject: [PATCH] fix(apple): don't pass completionHandler to IPC if we're not going to call it (#7671) If we pass a `completionHandler` to `sendProviderMessage`, the Network Extension framework will expect that completion handler to be called. Probably doesn't cause any issue as-is, but it would be better to be correct here. --- .../Sources/FirezoneKit/Managers/TunnelManager.swift | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/swift/apple/FirezoneKit/Sources/FirezoneKit/Managers/TunnelManager.swift b/swift/apple/FirezoneKit/Sources/FirezoneKit/Managers/TunnelManager.swift index bf91caa87..0a6c7baab 100644 --- a/swift/apple/FirezoneKit/Sources/FirezoneKit/Managers/TunnelManager.swift +++ b/swift/apple/FirezoneKit/Sources/FirezoneKit/Managers/TunnelManager.swift @@ -293,7 +293,7 @@ public class TunnelManager { func updateInternetResourceState() { guard session()?.status == .connected else { return } - try? session()?.sendProviderMessage(encoder.encode(TunnelMessage.internetResourceEnabled(internetResourceEnabled))) { _ in } + try? session()?.sendProviderMessage(encoder.encode(TunnelMessage.internetResourceEnabled(internetResourceEnabled))) } func toggleInternetResource(enabled: Bool) {