chore: remove socket as the default transport layer (#179)

MetaMask · Aug 22, 2024 · 28cb29a · 28cb29a
1 parent 749953d
commit 28cb29a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/Sources/metamask-ios-sdk/Classes/CommunicationLayer/CommLayer.swift b/Sources/metamask-ios-sdk/Classes/CommunicationLayer/CommLayer.swift
@@ -5,8 +5,13 @@
 
 import Foundation
 
+/**
+ An enum representing the communication types supported for communication with MetaMask wallet
+ **/
 public enum Transport: CaseIterable, Identifiable, Hashable {
+    /// Uses socket.io as a transport mechanism
     case socket
+    /// Uses deeplinking as transport mechanism. Recommended. Requires setting URI scheme
     case deeplinking(dappScheme: String)
 
     public var id: String {

diff --git a/Sources/metamask-ios-sdk/Classes/SDK/MetaMaskSDK.swift b/Sources/metamask-ios-sdk/Classes/SDK/MetaMaskSDK.swift
@@ -81,7 +81,7 @@ public class MetaMaskSDK: ObservableObject {
     }
 
     public static func shared(_ appMetadata: AppMetadata,
-                              transport: Transport = .socket,
+                              transport: Transport,
                               enableDebug: Bool = true,
                               sdkOptions: SDKOptions?) -> MetaMaskSDK {
         guard let sdk = SDKWrapper.shared.sdk else {