fix: changes to fix lifetime issues when multiple sdks are in use (#390)

Author: kneekey23

Packages/ClientRuntime/Sources/Networking/Http/CRT/CRTClientEngine.swift

@@ -21,6 +21,7 @@ public class CRTClientEngine: HttpClientEngine {
 
     private let windowSize: Int
     private let maxConnectionsPerEndpoint: Int
+    private let sharedDefaultIO: SDKDefaultIO = SDKDefaultIO.shared
 
     init(config: CRTClientEngineConfig = CRTClientEngineConfig()) {
         self.maxConnectionsPerEndpoint = config.maxConnectionsPerEndpoint
@@ -147,8 +148,4 @@ public class CRTClientEngine: HttpClientEngine {
 
         return (requestOptions, future)
     }
-    
-    deinit {
-        AwsCommonRuntimeKit.cleanUp()
-    }
 }

Packages/ClientRuntime/Sources/Networking/Http/CRT/SDKDefaultIO.swift

@@ -9,12 +9,23 @@ import AwsCommonRuntimeKit
 import class Foundation.ProcessInfo
 
 public final class SDKDefaultIO {
-    public static let shared = SDKDefaultIO()
+    static weak var privateShared: SDKDefaultIO? = nil
 
-    public let eventLoopGroup: EventLoopGroup
-    public let hostResolver: DefaultHostResolver
-    public let clientBootstrap: ClientBootstrap
-    public let tlsContext: TlsContext
+    // TODO: revisit this and verify that it is thread safe.
+    public static var shared: SDKDefaultIO {
+        if let shared = privateShared {
+            return shared
+        } else {
+            let shared = SDKDefaultIO()
+            privateShared = shared
+            return shared
+        }
+    }
+    
+    public var eventLoopGroup: EventLoopGroup
+    public var hostResolver: DefaultHostResolver
+    public var clientBootstrap: ClientBootstrap
+    public var tlsContext: TlsContext
 
     private init() {
         AwsCommonRuntimeKit.initialize()

Packages/ClientRuntime/Sources/Retries/SDKRetryer.swift

@@ -8,6 +8,7 @@ import AwsCommonRuntimeKit
 
 public class SDKRetryer: Retryer {
     let crtRetryStrategy: CRTAWSRetryStrategy
+    private let sharedDefaultIO: SDKDefaultIO = SDKDefaultIO.shared
 
     public init(options: RetryOptions) throws {
         self.crtRetryStrategy = try CRTAWSRetryStrategy(options: options.toCRTType())