Make IndiePitcher sendable and use shared httpclient by default

IndiePitcher · Sep 30, 2024 · 7b132c0 · 7b132c0
1 parent 02c9543
commit 7b132c0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/Sources/IndiePitcherSwift/IndiePitcherSwift.swift b/Sources/IndiePitcherSwift/IndiePitcherSwift.swift
@@ -7,17 +7,17 @@ import NIOFoundationCompat
 
 /// IndiePitcher SDK.
 /// This SDK is only intended for server-side Swift use. Do not embed the secret API key in client-side code for security reasons.
-public struct IndiePitcher {
-    private let client: HTTPClient
+public struct IndiePitcher: Sendable {
+    private let client: HTTPClient // is sendable / thread-safe
     private let apiKey: String
     private let requestTimeout: TimeAmount = .seconds(30)
     private let maxResponseSize = 1024 * 1024 * 100
 
     /// Creates a new instance of IndiePitcher SDK
     /// - Parameters:
-    ///   - client: Vapor's client instance to use to perform network requests.
+    ///   - client: Vapor's client instance to use to perform network requests. Uses the  shared client by default.
     ///   - apiKey: Your project's secret key.
-    public init(client: HTTPClient, apiKey: String) {
+    public init(client: HTTPClient = .shared, apiKey: String) {
         self.client = client
         self.apiKey = apiKey
     }

diff --git a/Tests/IndiePitcherSwiftTests/IndiePitcherSwiftTests.swift b/Tests/IndiePitcherSwiftTests/IndiePitcherSwiftTests.swift
@@ -8,7 +8,7 @@ final class IndiePitcherSwiftTests: XCTestCase {
     var indiePitcher: IndiePitcher!
 
     override func setUp() async throws {
-        indiePitcher = IndiePitcher(client: .shared, apiKey: IP_SECRET_API_KEY)
+        indiePitcher = IndiePitcher(apiKey: IP_SECRET_API_KEY)
     }
 
     override func tearDown() async throws {