Merge pull request #11 from LarsJK/async-will-send

Make willSend async
kean · Dec 16, 2021 · 731689f · 731689f
2 parents 71aeac4 + 3cd7bff
commit 731689f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/Sources/APIClient/APIClient.swift b/Sources/APIClient/APIClient.swift
@@ -5,7 +5,7 @@
 import Foundation
 
 public protocol APIClientDelegate {
-    func client(_ client: APIClient, willSendRequest request: inout URLRequest)
+    func client(_ client: APIClient, willSendRequest request: inout URLRequest) async
     func shouldClientRetry(_ client: APIClient, withError error: Error) async -> Bool
     func client(_ client: APIClient, didReceiveInvalidResponse response: HTTPURLResponse, data: Data) -> Error
 }
@@ -104,7 +104,7 @@ public actor APIClient {
 
     private func actuallySend(_ request: URLRequest) async throws -> Response<Data> {
         var request = request
-        delegate.client(self, willSendRequest: &request)
+        await delegate.client(self, willSendRequest: &request)
         let (data, response) = try await session.data(for: request, delegate: nil)
         try validate(response: response, data: data)
         let httpResponse = (response as? HTTPURLResponse) ?? HTTPURLResponse() // The right side should never be executed

diff --git a/Tests/APIClientTests/APIClientTests.swift b/Tests/APIClientTests/APIClientTests.swift
@@ -219,7 +219,7 @@ private func json(named name: String) -> Data {
 private final class MockAuthorizatingDelegate: APIClientDelegate {
     var token = "expired-token"
 
-    func client(_ client: APIClient, willSendRequest request: inout URLRequest) {
+    func client(_ client: APIClient, willSendRequest request: inout URLRequest) async {
         request.allHTTPHeaderFields = ["Authorization": "Bearer: \(token)"]
     }