Soundness fixes

Author: czechboy0

Sources/OpenAPIRuntime/Conversion/Configuration.swift

@@ -166,7 +166,8 @@ public struct Configuration: Sendable {
     ///   - jsonEncodingOptions: The options for the underlying JSON encoder.
     ///   - multipartBoundaryGenerator: The generator to use when creating mutlipart bodies.
     ///   - xmlCoder: Custom XML coder for encoding and decoding xml bodies. Only required when using XML body payloads.
-    ///   - errorMapper: An error mapping closure to allow customizing the final error thrown.
+    ///   - clientErrorMapper: An error mapping closure to allow customizing the error thrown by the client.
+    ///   - serverErrorMapper: An error mapping closure to allow customizing the error thrown by the server.
     public init(
         dateTranscoder: any DateTranscoder = .iso8601,
         jsonEncodingOptions: JSONEncodingOptions = [.sortedKeys, .prettyPrinted],

Sources/OpenAPIRuntime/Deprecated/Deprecated.swift

@@ -67,8 +67,12 @@ extension Configuration {
     ///   - jsonEncodingOptions: The options for the underlying JSON encoder.
     ///   - multipartBoundaryGenerator: The generator to use when creating mutlipart bodies.
     ///   - xmlCoder: Custom XML coder for encoding and decoding xml bodies. Only required when using XML body payloads.
-    @available(*, deprecated, renamed: "init(dateTranscoder:jsonEncodingOptions:multipartBoundaryGenerator:xmlCoder:clientErrorMapper:serverErrorMapper:)")
-    @_disfavoredOverload public init(
+    @available(
+        *,
+        deprecated,
+        renamed:
+            "init(dateTranscoder:jsonEncodingOptions:multipartBoundaryGenerator:xmlCoder:clientErrorMapper:serverErrorMapper:)"
+    ) @_disfavoredOverload public init(
         dateTranscoder: any DateTranscoder = .iso8601,
         jsonEncodingOptions: JSONEncodingOptions = [.sortedKeys, .prettyPrinted],
         multipartBoundaryGenerator: any MultipartBoundaryGenerator = .random,

Sources/OpenAPIRuntime/Interface/UniversalServer.swift

@@ -180,9 +180,7 @@ import struct Foundation.URLComponents
                 }
             }
         }
-        do {
-            return try await next(request, requestBody, metadata)
-        } catch {
+        do { return try await next(request, requestBody, metadata) } catch {
             if let errorMapper, let serverError = error as? ServerError {
                 throw errorMapper(serverError)
             } else {

Tests/OpenAPIRuntimeTests/Interface/Test_UniversalClient.swift

@@ -122,12 +122,10 @@ final class Test_UniversalClient: Test_Runtime {
     func testErrorPropagation_customErrorMapper() async throws {
         do {
             let client = UniversalClient(
-                configuration: .init(
-                    clientErrorMapper: { clientError in
-                        // Don't care about the extra context, just wants the underlyingError
-                        clientError.underlyingError
-                    }
-                ),
+                configuration: .init(clientErrorMapper: { clientError in
+                    // Don't care about the extra context, just wants the underlyingError
+                    clientError.underlyingError
+                }),
                 transport: MockClientTransport.failing
             )
             try await client.send(
@@ -136,9 +134,7 @@ final class Test_UniversalClient: Test_Runtime {
                 serializer: { input in (HTTPRequest(soar_path: "/", method: .post), MockClientTransport.requestBody) },
                 deserializer: { response, body in fatalError() }
             )
-        } catch {
-            XCTAssertTrue(error is TestError, "Threw an unexpected error: \(type(of: error))")
-        }
+        } catch { XCTAssertTrue(error is TestError, "Threw an unexpected error: \(type(of: error))") }
     }
 
     func testErrorPropagation_middlewareOnResponse() async throws {

Tests/OpenAPIRuntimeTests/Interface/Test_UniversalServer.swift

@@ -133,12 +133,10 @@ final class Test_UniversalServer: Test_Runtime {
         do {
             let server = UniversalServer(
                 handler: MockHandler(shouldFail: true),
-                configuration: .init(
-                    serverErrorMapper: { serverError in
-                        // Don't care about the extra context, just wants the underlyingError
-                        serverError.underlyingError
-                    }
-                )
+                configuration: .init(serverErrorMapper: { serverError in
+                    // Don't care about the extra context, just wants the underlyingError
+                    serverError.underlyingError
+                })
             )
             _ = try await server.handle(
                 request: .init(soar_path: "/", method: .post),
@@ -152,9 +150,7 @@ final class Test_UniversalServer: Test_Runtime {
                 },
                 serializer: { output, _ in fatalError() }
             )
-        } catch {
-            XCTAssertTrue(error is TestError, "Threw an unexpected error: \(type(of: error))")
-        }
+        } catch { XCTAssertTrue(error is TestError, "Threw an unexpected error: \(type(of: error))") }
     }
 
     func testErrorPropagation_serializer() async throws {