]> git.ipfire.org Git - thirdparty/wireguard-apple.git/commitdiff
Removed SetTunnelStatus
authorAlessio Nossa <alessio.nossa@gmail.com>
Sat, 1 Apr 2023 14:00:24 +0000 (16:00 +0200)
committerAlessio Nossa <alessio.nossa@gmail.com>
Sat, 1 Apr 2023 14:00:24 +0000 (16:00 +0200)
Signed-off-by: Alessio Nossa <alessio.nossa@gmail.com>
Sources/Shared/Intents.intentdefinition
Sources/WireGuardApp/Tunnel/TunnelsManager.swift
Sources/WireGuardApp/UI/iOS/ViewController/MainViewController.swift
Sources/WireGuardApp/UI/iOS/ViewController/TunnelDetailTableViewController.swift
Sources/WireGuardApp/UI/iOS/ViewController/TunnelsListTableViewController.swift
Sources/WireGuardIntentsExtension/IntentHandler.swift
Sources/WireGuardIntentsExtension/IntentHandling.swift

index 6f2d38c0b4f0a0a98673d54744ce0a83e9966118..74bbf1fc0063b6ac93e5f94daf73a2144615b3a9 100644 (file)
        <key>INIntentDefinitionNamespace</key>
        <string>6NREiY</string>
        <key>INIntentDefinitionSystemVersion</key>
-       <string>21C52</string>
+       <string>21G419</string>
        <key>INIntentDefinitionToolsBuildVersion</key>
-       <string>13C100</string>
+       <string>14C18</string>
        <key>INIntentDefinitionToolsVersion</key>
-       <string>13.2.1</string>
+       <string>14.2</string>
        <key>INIntents</key>
        <array>
                <dict>
@@ -447,271 +447,6 @@ Example
                        <key>INIntentVerb</key>
                        <string>Do</string>
                </dict>
-               <dict>
-                       <key>INIntentCategory</key>
-                       <string>toggle</string>
-                       <key>INIntentConfigurable</key>
-                       <true/>
-                       <key>INIntentDescription</key>
-                       <string>Enables or disables a tunnel.
-
-The action will not return an error if there are problems during the activation, it just launches the procedure.</string>
-                       <key>INIntentDescriptionID</key>
-                       <string>T7NEBw</string>
-                       <key>INIntentEligibleForWidgets</key>
-                       <true/>
-                       <key>INIntentLastParameterTag</key>
-                       <integer>4</integer>
-                       <key>INIntentManagedParameterCombinations</key>
-                       <dict>
-                               <key>state,tunnel,operation</key>
-                               <dict>
-                                       <key>INIntentParameterCombinationSupportsBackgroundExecution</key>
-                                       <true/>
-                                       <key>INIntentParameterCombinationTitle</key>
-                                       <string>${operation} ${tunnel} tunnel ${state}</string>
-                                       <key>INIntentParameterCombinationTitleID</key>
-                                       <string>juDVvv</string>
-                                       <key>INIntentParameterCombinationUpdatesLinked</key>
-                                       <true/>
-                               </dict>
-                               <key>tunnel,operation</key>
-                               <dict>
-                                       <key>INIntentParameterCombinationSupportsBackgroundExecution</key>
-                                       <true/>
-                                       <key>INIntentParameterCombinationTitle</key>
-                                       <string>${operation} ${tunnel} tunnel</string>
-                                       <key>INIntentParameterCombinationTitleID</key>
-                                       <string>kMEAI0</string>
-                               </dict>
-                       </dict>
-                       <key>INIntentName</key>
-                       <string>SetTunnelStatus</string>
-                       <key>INIntentParameterCombinations</key>
-                       <dict>
-                               <key>state,tunnel,operation</key>
-                               <dict>
-                                       <key>INIntentParameterCombinationIsLinked</key>
-                                       <true/>
-                                       <key>INIntentParameterCombinationSupportsBackgroundExecution</key>
-                                       <true/>
-                                       <key>INIntentParameterCombinationTitle</key>
-                                       <string>${operation} ${tunnel} tunnel ${state}</string>
-                                       <key>INIntentParameterCombinationTitleID</key>
-                                       <string>SjQ7Ur</string>
-                               </dict>
-                       </dict>
-                       <key>INIntentParameters</key>
-                       <array>
-                               <dict>
-                                       <key>INIntentParameterConfigurable</key>
-                                       <true/>
-                                       <key>INIntentParameterDisplayName</key>
-                                       <string>Tunnel</string>
-                                       <key>INIntentParameterDisplayNameID</key>
-                                       <string>mbhQcc</string>
-                                       <key>INIntentParameterDisplayPriority</key>
-                                       <integer>1</integer>
-                                       <key>INIntentParameterMetadata</key>
-                                       <dict>
-                                               <key>INIntentParameterMetadataCapitalization</key>
-                                               <string>Sentences</string>
-                                               <key>INIntentParameterMetadataDefaultValueID</key>
-                                               <string>iBD5fT</string>
-                                       </dict>
-                                       <key>INIntentParameterName</key>
-                                       <string>tunnel</string>
-                                       <key>INIntentParameterPromptDialogs</key>
-                                       <array>
-                                               <dict>
-                                                       <key>INIntentParameterPromptDialogCustom</key>
-                                                       <true/>
-                                                       <key>INIntentParameterPromptDialogType</key>
-                                                       <string>Configuration</string>
-                                               </dict>
-                                               <dict>
-                                                       <key>INIntentParameterPromptDialogCustom</key>
-                                                       <true/>
-                                                       <key>INIntentParameterPromptDialogType</key>
-                                                       <string>Primary</string>
-                                               </dict>
-                                               <dict>
-                                                       <key>INIntentParameterPromptDialogCustom</key>
-                                                       <true/>
-                                                       <key>INIntentParameterPromptDialogFormatString</key>
-                                                       <string>There are ${count} options matching ‘${tunnel}’.</string>
-                                                       <key>INIntentParameterPromptDialogFormatStringID</key>
-                                                       <string>73NvEc</string>
-                                                       <key>INIntentParameterPromptDialogType</key>
-                                                       <string>DisambiguationIntroduction</string>
-                                               </dict>
-                                               <dict>
-                                                       <key>INIntentParameterPromptDialogCustom</key>
-                                                       <true/>
-                                                       <key>INIntentParameterPromptDialogFormatString</key>
-                                                       <string>Just to confirm, you wanted ‘${tunnel}’?</string>
-                                                       <key>INIntentParameterPromptDialogFormatStringID</key>
-                                                       <string>63TirR</string>
-                                                       <key>INIntentParameterPromptDialogType</key>
-                                                       <string>Confirmation</string>
-                                               </dict>
-                                       </array>
-                                       <key>INIntentParameterSupportsDynamicEnumeration</key>
-                                       <true/>
-                                       <key>INIntentParameterTag</key>
-                                       <integer>2</integer>
-                                       <key>INIntentParameterType</key>
-                                       <string>String</string>
-                               </dict>
-                               <dict>
-                                       <key>INIntentParameterConfigurable</key>
-                                       <true/>
-                                       <key>INIntentParameterDisplayName</key>
-                                       <string>Operation</string>
-                                       <key>INIntentParameterDisplayNameID</key>
-                                       <string>U9YFTG</string>
-                                       <key>INIntentParameterDisplayPriority</key>
-                                       <integer>2</integer>
-                                       <key>INIntentParameterEnumType</key>
-                                       <string>Operation</string>
-                                       <key>INIntentParameterEnumTypeNamespace</key>
-                                       <string>6NREiY</string>
-                                       <key>INIntentParameterMetadata</key>
-                                       <dict>
-                                               <key>INIntentParameterMetadataDefaultValue</key>
-                                               <string>turn</string>
-                                       </dict>
-                                       <key>INIntentParameterName</key>
-                                       <string>operation</string>
-                                       <key>INIntentParameterPromptDialogs</key>
-                                       <array>
-                                               <dict>
-                                                       <key>INIntentParameterPromptDialogCustom</key>
-                                                       <true/>
-                                                       <key>INIntentParameterPromptDialogType</key>
-                                                       <string>Configuration</string>
-                                               </dict>
-                                               <dict>
-                                                       <key>INIntentParameterPromptDialogCustom</key>
-                                                       <true/>
-                                                       <key>INIntentParameterPromptDialogType</key>
-                                                       <string>Primary</string>
-                                               </dict>
-                                               <dict>
-                                                       <key>INIntentParameterPromptDialogCustom</key>
-                                                       <true/>
-                                                       <key>INIntentParameterPromptDialogFormatString</key>
-                                                       <string>There are ${count} options matching ‘${operation}’.</string>
-                                                       <key>INIntentParameterPromptDialogFormatStringID</key>
-                                                       <string>SALmBF</string>
-                                                       <key>INIntentParameterPromptDialogType</key>
-                                                       <string>DisambiguationIntroduction</string>
-                                               </dict>
-                                               <dict>
-                                                       <key>INIntentParameterPromptDialogCustom</key>
-                                                       <true/>
-                                                       <key>INIntentParameterPromptDialogFormatString</key>
-                                                       <string>Just to confirm, you wanted ‘${operation}’?</string>
-                                                       <key>INIntentParameterPromptDialogFormatStringID</key>
-                                                       <string>vB13mD</string>
-                                                       <key>INIntentParameterPromptDialogType</key>
-                                                       <string>Confirmation</string>
-                                               </dict>
-                                       </array>
-                                       <key>INIntentParameterTag</key>
-                                       <integer>4</integer>
-                                       <key>INIntentParameterType</key>
-                                       <string>Integer</string>
-                               </dict>
-                               <dict>
-                                       <key>INIntentParameterConfigurable</key>
-                                       <true/>
-                                       <key>INIntentParameterDisplayName</key>
-                                       <string>State</string>
-                                       <key>INIntentParameterDisplayNameID</key>
-                                       <string>sLIh6r</string>
-                                       <key>INIntentParameterDisplayPriority</key>
-                                       <integer>3</integer>
-                                       <key>INIntentParameterEnumType</key>
-                                       <string>State</string>
-                                       <key>INIntentParameterEnumTypeNamespace</key>
-                                       <string>6NREiY</string>
-                                       <key>INIntentParameterMetadata</key>
-                                       <dict>
-                                               <key>INIntentParameterMetadataDefaultValue</key>
-                                               <string>on</string>
-                                       </dict>
-                                       <key>INIntentParameterName</key>
-                                       <string>state</string>
-                                       <key>INIntentParameterRelationship</key>
-                                       <dict>
-                                               <key>INIntentParameterRelationshipParentName</key>
-                                               <string>operation</string>
-                                               <key>INIntentParameterRelationshipPredicateName</key>
-                                               <string>EnumHasExactValue</string>
-                                               <key>INIntentParameterRelationshipPredicateValue</key>
-                                               <string>turn</string>
-                                       </dict>
-                                       <key>INIntentParameterTag</key>
-                                       <integer>1</integer>
-                                       <key>INIntentParameterType</key>
-                                       <string>Integer</string>
-                               </dict>
-                       </array>
-                       <key>INIntentResponse</key>
-                       <dict>
-                               <key>INIntentResponseCodes</key>
-                               <array>
-                                       <dict>
-                                               <key>INIntentResponseCodeName</key>
-                                               <string>success</string>
-                                               <key>INIntentResponseCodeSuccess</key>
-                                               <true/>
-                                       </dict>
-                                       <dict>
-                                               <key>INIntentResponseCodeName</key>
-                                               <string>failure</string>
-                                       </dict>
-                               </array>
-                               <key>INIntentResponseLastParameterTag</key>
-                               <integer>1</integer>
-                               <key>INIntentResponseParameters</key>
-                               <array>
-                                       <dict>
-                                               <key>INIntentResponseParameterConfigurable</key>
-                                               <true/>
-                                               <key>INIntentResponseParameterCustomDisambiguation</key>
-                                               <true/>
-                                               <key>INIntentResponseParameterDisplayName</key>
-                                               <string>State</string>
-                                               <key>INIntentResponseParameterDisplayNameID</key>
-                                               <string>TXqy56</string>
-                                               <key>INIntentResponseParameterDisplayPriority</key>
-                                               <integer>1</integer>
-                                               <key>INIntentResponseParameterEnumType</key>
-                                               <string>State</string>
-                                               <key>INIntentResponseParameterEnumTypeNamespace</key>
-                                               <string>6NREiY</string>
-                                               <key>INIntentResponseParameterName</key>
-                                               <string>state</string>
-                                               <key>INIntentResponseParameterSupportsResolution</key>
-                                               <true/>
-                                               <key>INIntentResponseParameterTag</key>
-                                               <integer>1</integer>
-                                               <key>INIntentResponseParameterType</key>
-                                               <string>Integer</string>
-                                       </dict>
-                               </array>
-                       </dict>
-                       <key>INIntentTitle</key>
-                       <string>Set Tunnel Status</string>
-                       <key>INIntentTitleID</key>
-                       <string>HULmDn</string>
-                       <key>INIntentType</key>
-                       <string>Custom</string>
-                       <key>INIntentVerb</key>
-                       <string>Toggle</string>
-               </dict>
        </array>
        <key>INTypes</key>
        <array/>
index fbf29c4fe95f075e8a7923bad72cddeb744f6984..7751e545792312f14d59cd86f2bfb681355d9592 100644 (file)
@@ -5,10 +5,6 @@ import Foundation
 import NetworkExtension
 import os.log
 
-#if os(iOS)
-import Intents
-#endif
-
 protocol TunnelsManagerListDelegate: AnyObject {
     func tunnelAdded(at index: Int)
     func tunnelModified(at index: Int)
@@ -311,12 +307,6 @@ class TunnelsManager {
         }
         #elseif os(iOS)
         (tunnelProviderManager.protocolConfiguration as? NETunnelProviderProtocol)?.destroyConfigurationReference()
-
-        INInteraction.delete(with: "com.wireguard.intents.tunnel.\(tunnel.name)") { error in
-            if let error = error {
-                wg_log(.error, message: "Error deleting donated interactions for tunnel \(tunnel.name): \(error.localizedDescription)")
-            }
-        }
         #else
         #error("Unimplemented")
         #endif
index 74a8149fe306d60eaa250da155b4b195ee726552..6aab559086731fbae34375513886924e3cfab53c 100644 (file)
@@ -2,7 +2,6 @@
 // Copyright © 2018-2021 WireGuard LLC. All Rights Reserved.
 
 import UIKit
-import Intents
 
 class MainViewController: UISplitViewController {
 
@@ -97,25 +96,10 @@ extension MainViewController {
             if let tunnel = tunnelsManager.tunnel(named: tunnelName) {
                 tunnelsListVC.showTunnelDetail(for: tunnel, animated: false)
                 if shouldToggleStatus {
-
-                    let intent = SetTunnelStatusIntent()
-                    intent.tunnel = tunnel.name
-                    intent.operation = .turn
-
                     if tunnel.status == .inactive {
                         tunnelsManager.startActivation(of: tunnel)
-                        intent.state = .on
                     } else if tunnel.status == .active {
                         tunnelsManager.startDeactivation(of: tunnel)
-                        intent.state = .off
-                    }
-
-                    let interaction = INInteraction(intent: intent, response: nil)
-                    interaction.groupIdentifier = "com.wireguard.intents.tunnel.\(tunnel.name)"
-                    interaction.donate { error in
-                        if let  error = error {
-                            wg_log(.error, message: "Error donating interaction for SetTunnelStatusIntent: \(error.localizedDescription)")
-                        }
                     }
                 }
             }
index 0361d44b8ea4738aa1d4f847d0759ba30725ca44..2c185d94c58d9e28268c8fb39ffd91b814f9d466 100644 (file)
@@ -388,18 +388,6 @@ extension TunnelDetailTableViewController {
         cell.onSwitchToggled = { [weak self] isOn in
             guard let self = self else { return }
 
-            let intent = SetTunnelStatusIntent()
-            intent.tunnel = self.tunnel.name
-            intent.operation = .turn
-            intent.state = isOn ? .on : .off
-            let interaction = INInteraction(intent: intent, response: nil)
-            interaction.groupIdentifier = "com.wireguard.intents.tunnel.\(self.tunnel.name)"
-            interaction.donate { error in
-                if let  error = error {
-                    wg_log(.error, message: "Error donating interaction for SetTunnelStatusIntent: \(error.localizedDescription)")
-                }
-            }
-
             if self.tunnel.hasOnDemandRules {
                 self.tunnelsManager.setOnDemandEnabled(isOn, on: self.tunnel) { error in
                     if error == nil && !isOn {
index c5132a207b59395a9341834288c94527c18921cf..85e64cea2eb825fad5f6fcbfb10f880de8cf2af2 100644 (file)
@@ -4,7 +4,6 @@
 import UIKit
 import MobileCoreServices
 import UserNotifications
-import Intents
 
 class TunnelsListTableViewController: UIViewController {
 
@@ -318,19 +317,6 @@ extension TunnelsListTableViewController: UITableViewDataSource {
             cell.tunnel = tunnel
             cell.onSwitchToggled = { [weak self] isOn in
                 guard let self = self, let tunnelsManager = self.tunnelsManager else { return }
-
-                let intent = SetTunnelStatusIntent()
-                intent.tunnel = tunnel.name
-                intent.operation = .turn
-                intent.state = isOn ? .on : .off
-                let interaction = INInteraction(intent: intent, response: nil)
-                interaction.groupIdentifier = "com.wireguard.intents.tunnel.\(tunnel.name)"
-                interaction.donate { error in
-                    if let  error = error {
-                        wg_log(.error, message: "Error donating interaction for SetTunnelStatusIntent: \(error.localizedDescription)")
-                    }
-                }
-
                 if tunnel.hasOnDemandRules {
                     tunnelsManager.setOnDemandEnabled(isOn, on: tunnel) { error in
                         if error == nil && !isOn {
index 6f1995144768cca3d7219270e1c36b66bf8d234f..a3527ef938adcec6211645c27584516a60258fd9 100644 (file)
@@ -12,8 +12,7 @@ class IntentHandler: INExtension {
 
     override func handler(for intent: INIntent) -> Any {
         guard intent is GetPeersIntent ||
-                intent is UpdateConfigurationIntent ||
-                intent is SetTunnelStatusIntent else {
+                intent is UpdateConfigurationIntent else {
             fatalError("Unhandled intent type: \(intent)")
         }
 
index 78004e07ad20b3f6a1642b6efd74655e3d5f79d8..1de3d46ade8b430b502a71d6a8ae6aea0c86a45c 100644 (file)
@@ -15,8 +15,6 @@ class IntentHandling: NSObject {
 
     var onTunnelsManagerReady: ((TunnelsManager) -> Void)?
 
-    var onTunnelStatusActivationReturn: ((Bool) -> Void)?
-
     override init() {
         super.init()
 
@@ -29,8 +27,6 @@ class IntentHandling: NSObject {
             case .success(let tunnelsManager):
                 self.tunnelsManager = tunnelsManager
 
-                self.tunnelsManager?.activationDelegate = self
-
                 self.onTunnelsManagerReady?(tunnelsManager)
                 self.onTunnelsManagerReady = nil
             }
@@ -174,129 +170,3 @@ extension IntentHandling: UpdateConfigurationIntentHandling {
     }
 
 }
-
-extension IntentHandling: SetTunnelStatusIntentHandling {
-
-    @available(iOSApplicationExtension 14.0, *)
-    func provideTunnelOptionsCollection(for intent: SetTunnelStatusIntent, with completion: @escaping (INObjectCollection<NSString>?, Error?) -> Void) {
-
-        self.allTunnelNames { tunnelsNames in
-            let tunnelsNamesObjects = (tunnelsNames ?? []).map { NSString(string: $0) }
-
-            let objectCollection = INObjectCollection(items: tunnelsNamesObjects)
-            completion(objectCollection, nil)
-        }
-    }
-
-    func handle(intent: SetTunnelStatusIntent, completion: @escaping (SetTunnelStatusIntentResponse) -> Void) {
-        guard let tunnelName = intent.tunnel else {
-            return completion(SetTunnelStatusIntentResponse(code: .failure, userActivity: nil))
-        }
-
-        let setTunnelStatusResultBlock: (Bool) -> Void = { result in
-            if result {
-                completion(SetTunnelStatusIntentResponse(code: .success, userActivity: nil))
-            } else {
-                completion(SetTunnelStatusIntentResponse(code: .failure, userActivity: nil))
-            }
-        }
-
-        let updateStatusBlock: (TunnelsManager) -> Void = { tunnelsManager in
-            guard let tunnel = tunnelsManager.tunnel(named: tunnelName) else {
-                completion(SetTunnelStatusIntentResponse(code: .failure, userActivity: nil))
-                return
-            }
-
-            let operation = intent.operation
-            let isOn: Bool
-
-            if operation == .toggle {
-                switch tunnel.status {
-                case .inactive:
-                    isOn = true
-                case .active:
-                    isOn = false
-                default:
-                    wg_log(.error, message: "SetTunnelStatusIntent action cannot be executed due to the current state of \(tunnelName) tunnel: \(tunnel.status)")
-                    completion(SetTunnelStatusIntentResponse(code: .failure, userActivity: nil))
-                    return
-                }
-
-            } else if operation == .turn {
-                if (tunnel.status == .inactive) || (tunnel.status == .active) {
-                    isOn = (intent.state == .on)
-
-                    if (isOn && tunnel.status == .active) || (!isOn && tunnel.status == .inactive) {
-                        wg_log(.debug, message: "Tunnel \(tunnelName) is already \(isOn ? "active" : "inactive")")
-                        completion(SetTunnelStatusIntentResponse(code: .success, userActivity: nil))
-                        return
-                    }
-                } else {
-                    wg_log(.error, message: "SetTunnelStatusIntent action cannot be executed due to the current state of \(tunnelName) tunnel: \(tunnel.status)")
-                    completion(SetTunnelStatusIntentResponse(code: .failure, userActivity: nil))
-                    return
-                }
-
-            } else {
-                wg_log(.error, message: "Invalid 'operation' option in action")
-                completion(SetTunnelStatusIntentResponse(code: .failure, userActivity: nil))
-                return
-            }
-
-            if tunnel.hasOnDemandRules {
-                tunnelsManager.setOnDemandEnabled(isOn, on: tunnel) { error in
-                    guard error == nil else {
-                        wg_log(.error, message: "Error setting OnDemand status: \(error!.localizedDescription).")
-                        completion(SetTunnelStatusIntentResponse(code: .failure, userActivity: nil))
-                        return
-                    }
-
-                    if !isOn {
-                        tunnelsManager.startDeactivation(of: tunnel)
-                    }
-
-                    completion(SetTunnelStatusIntentResponse(code: .success, userActivity: nil))
-                }
-            } else {
-                if isOn {
-                    self.onTunnelStatusActivationReturn = setTunnelStatusResultBlock
-                    tunnelsManager.startActivation(of: tunnel)
-                } else {
-                    tunnelsManager.startDeactivation(of: tunnel)
-                    completion(SetTunnelStatusIntentResponse(code: .success, userActivity: nil))
-                }
-            }
-        }
-
-        if let tunnelsManager = tunnelsManager {
-            updateStatusBlock(tunnelsManager)
-        } else {
-            if onTunnelsManagerReady != nil {
-                wg_log(.error, message: "Overriding onTunnelsManagerReady action in allTunnelPeers function. This should not happen.")
-            }
-            onTunnelsManagerReady = updateStatusBlock
-        }
-    }
-
-}
-
-extension IntentHandling: TunnelsManagerActivationDelegate {
-    func tunnelActivationAttemptFailed(tunnel: TunnelContainer, error: TunnelsManagerActivationAttemptError) {
-        wg_log(.error, message: "Tunnel Activation Attempt Failed with error: \(error.localizedDescription)")
-        self.onTunnelStatusActivationReturn?(false)
-    }
-
-    func tunnelActivationAttemptSucceeded(tunnel: TunnelContainer) {
-        // Nothing to do, we wait tunnelActivationSucceeded to be sure all activation logic has been executed
-    }
-
-    func tunnelActivationFailed(tunnel: TunnelContainer, error: TunnelsManagerActivationError) {
-        wg_log(.error, message: "Tunnel Activation Failed with error: \(error.localizedDescription)")
-        self.onTunnelStatusActivationReturn?(false)
-    }
-
-    func tunnelActivationSucceeded(tunnel: TunnelContainer) {
-        self.onTunnelStatusActivationReturn?(true)
-    }
-
-}