Throw error on wrong API param

WalletConnect · Sep 1, 2022 · e00368f · e00368f
1 parent 927c388
commit e00368f
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 4 deletions.
diff --git a/Sources/Auth/AuthClient.swift b/Sources/Auth/AuthClient.swift
@@ -11,7 +11,7 @@ import WalletConnectRelay
 /// Access via `Auth.instance`
 public class AuthClient {
     enum Errors: Error {
-        case malformedPairingURI
+        case pairingUriWrongApiParam
         case unknownWalletAddress
         case noPairingMatchingTopic
     }
@@ -92,6 +92,9 @@ public class AuthClient {
     /// - When URI is invalid format or missing params
     /// - When topic is already in use
     public func pair(uri: WalletConnectURI) async throws {
+        guard uri.api = .auth else {
+            throw Errors.pairingUriWrongApiParam
+        }
         try await walletPairService.pair(uri)
     }
 

diff --git a/Sources/WalletConnectSign/Sign/SignClient.swift b/Sources/WalletConnectSign/Sign/SignClient.swift
@@ -169,6 +169,9 @@ public final class SignClient {
     /// - When URI has invalid format or missing params
     /// - When topic is already in use
     public func pair(uri: WalletConnectURI) async throws {
+        guard uri.api = .sign else {
+            throw WalletConnectError.pairingUriWrongApiParam
+        }
         try await pairEngine.pair(uri)
     }
 

diff --git a/Sources/WalletConnectSign/WalletConnectError.swift b/Sources/WalletConnectSign/WalletConnectError.swift
@@ -1,7 +1,7 @@
 enum WalletConnectError: Error {
 
     case pairingProposalFailed
-    case malformedPairingURI
+    case pairingUriWrongApiParam
     case noPairingMatchingTopic(String)
     case noSessionMatchingTopic(String)
     case sessionNotAcknowledged(String)
@@ -29,8 +29,8 @@ extension WalletConnectError {
         switch self {
         case .pairingProposalFailed:
             return "Pairing proposal failed."
-        case .malformedPairingURI:
-            return "Pairing URI string is invalid."
+        case .pairingUriWrongApiParam:
+            return "Pairing URI containt wrong API param"
         case .noPairingMatchingTopic(let topic):
             return "There is no existing pairing matching the topic: \(topic)."
         case .noSessionMatchingTopic(let topic):