PR changes

Author: gjcairo

Sources/GRPCNIOTransportHTTP2Posix/TLSConfig.swift

@@ -294,10 +294,10 @@ extension HTTP2ClientTransport.Posix.Config {
     ///   - serverHostname: An optional server hostname to use when verifying certificates.
     public init(
       certificateChain: [TLSConfig.CertificateSource],
-      privateKey: TLSConfig.PrivateKeySource? = nil,
+      privateKey: TLSConfig.PrivateKeySource?,
       serverCertificateVerification: TLSConfig.CertificateVerification,
       trustRoots: TLSConfig.TrustRootsSource,
-      serverHostname: String? = nil
+      serverHostname: String?
     ) {
       self.certificateChain = certificateChain
       self.privateKey = privateKey
@@ -330,6 +330,16 @@ extension HTTP2ClientTransport.Posix.Config {
       return config
     }
 
+    /// Create a new HTTP2 NIO Posix transport TLS config, with some values defaulted:
+    /// - `certificateChain` equals `[]`
+    /// - `privateKey` equals `nil`
+    /// - `serverCertificateVerification` equals `fullVerification`
+    /// - `trustRoots` equals `systemDefault`
+    /// - `serverHostname` equals `nil`
+    public static var defaults: Self {
+      Self.defaults()
+    }
+
     /// Create a new HTTP2 NIO Posix transport TLS config, with some values defaulted to match
     /// the requirements of mTLS:
     /// - `trustRoots` equals `systemDefault`
@@ -349,7 +359,8 @@ extension HTTP2ClientTransport.Posix.Config {
         certificateChain: certificateChain,
         privateKey: privateKey,
         serverCertificateVerification: .fullVerification,
-        trustRoots: .systemDefault
+        trustRoots: .systemDefault,
+        serverHostname: nil
       )
       configure(&config)
       return config

Tests/GRPCNIOTransportHTTP2Tests/HTTP2TransportNIOPosixTests.swift

@@ -410,7 +410,7 @@ final class HTTP2TransportNIOPosixTests: XCTestCase {
   }
 
   func testClientTLSConfig_Defaults() throws {
-    let grpcTLSConfig = HTTP2ClientTransport.Posix.Config.TLS.defaults()
+    let grpcTLSConfig = HTTP2ClientTransport.Posix.Config.TLS.defaults
     let nioSSLTLSConfig = try TLSConfiguration(grpcTLSConfig)
 
     XCTAssertEqual(nioSSLTLSConfig.certificateChain, [])
@@ -422,7 +422,7 @@ final class HTTP2TransportNIOPosixTests: XCTestCase {
   }
 
   func testClientTLSConfig_CustomCertificateChainAndPrivateKey() throws {
-    var grpcTLSConfig = HTTP2ClientTransport.Posix.Config.TLS.defaults()
+    var grpcTLSConfig = HTTP2ClientTransport.Posix.Config.TLS.defaults
     grpcTLSConfig.certificateChain = [
       .bytes(Array(Self.samplePemCert.utf8), format: .pem)
     ]
@@ -451,7 +451,7 @@ final class HTTP2TransportNIOPosixTests: XCTestCase {
   }
 
   func testClientTLSConfig_CustomTrustRoots() throws {
-    var grpcTLSConfig = HTTP2ClientTransport.Posix.Config.TLS.defaults()
+    var grpcTLSConfig = HTTP2ClientTransport.Posix.Config.TLS.defaults
     grpcTLSConfig.trustRoots = .certificates([.bytes(Array(Self.samplePemCert.utf8), format: .pem)])
     let nioSSLTLSConfig = try TLSConfiguration(grpcTLSConfig)
 
@@ -467,7 +467,7 @@ final class HTTP2TransportNIOPosixTests: XCTestCase {
   }
 
   func testClientTLSConfig_CustomCertificateVerification() throws {
-    var grpcTLSConfig = HTTP2ClientTransport.Posix.Config.TLS.defaults()
+    var grpcTLSConfig = HTTP2ClientTransport.Posix.Config.TLS.defaults
     grpcTLSConfig.serverCertificateVerification = .noHostnameVerification
     let nioSSLTLSConfig = try TLSConfiguration(grpcTLSConfig)