make HTTPVersion public and set to .automatic by default

Author: dnadoba

Diff for: Sources/AsyncHTTPClient/HTTPClient.swift

@@ -603,10 +603,8 @@ public class HTTPClient {
         /// Ignore TLS unclean shutdown error, defaults to `false`.
         public var ignoreUncleanSSLShutdown: Bool
 
-        // TODO: make public
-        // TODO: set to automatic by default
-        /// HTTP/2 is by default disabled
-        internal var httpVersion: HTTPVersion
+        /// is set to `.automatic` by default which will use HTTP/2 if run over https and the server supports it, otherwise HTTP/1
+        public var httpVersion: HTTPVersion
 
         public init(
             tlsConfiguration: TLSConfiguration? = nil,
@@ -624,13 +622,11 @@ public class HTTPClient {
                 proxy: proxy,
                 ignoreUncleanSSLShutdown: ignoreUncleanSSLShutdown,
                 decompression: decompression,
-                // TODO: set to automatic by default
-                httpVersion: .http1Only
+                httpVersion: .automatic
             )
         }
 
-        // TODO: make public
-        internal init(
+        public init(
             tlsConfiguration: TLSConfiguration? = nil,
             redirectConfiguration: RedirectConfiguration? = nil,
             timeout: Timeout = Timeout(),
@@ -862,18 +858,17 @@ extension HTTPClient.Configuration {
         }
     }
 
-    // TODO: make this struct and its static properties public
-    internal struct HTTPVersion {
+    public struct HTTPVersion {
         internal enum Configuration {
             case http1Only
             case automatic
         }
 
         /// we only use HTTP/1, even if the server would supports HTTP/2
-        internal static let http1Only: Self = .init(configuration: .http1Only)
+        public static let http1Only: Self = .init(configuration: .http1Only)
 
         /// HTTP/2 is used if we connect to a server with HTTPS and the server supports HTTP/2, otherwise we use HTTP/1
-        internal static let automatic: Self = .init(configuration: .automatic)
+        public static let automatic: Self = .init(configuration: .automatic)
 
         internal var configuration: Configuration
     }

Diff for: Tests/AsyncHTTPClientTests/HTTP2ClientTests.swift

@@ -12,8 +12,7 @@
 //
 //===----------------------------------------------------------------------===//
 
-// TODO: remove @testable once we officially support HTTP/2
-@testable import AsyncHTTPClient // Tests that really need @testable go into HTTP2ClientInternalTests.swift
+/* NOT @testable */ import AsyncHTTPClient // Tests that really need @testable go into HTTP2ClientInternalTests.swift
 #if canImport(Network)
     import Network
 #endif