Octavi00 opened a new pull request, #12449: URL: https://github.com/apache/gravitino/pull/12449
[Subtask] M2: TLS support at the HTTPClient layer #12300 ### What changes were proposed in this pull request? - Added TLSConfigurer: Default TLS interface. One note to the reviewer: the hostname verifier still uses an import org.apache.hc.client5.http.ssl.HttpsSupport to provide a default hostname verifier. SSLContext follows the requested "no types from implementation-scoped dependencies in its public signatures or default method bodies". I want to clarify if an hc5 import would meet or fail this request. - Added TLSConfigurers: contains all builder methods for easier ux. -Modified HTTPClient: Added optional TLS configuration - Added TestHTTPClientTLS.java: covers 6 requested test + 2 extra early fail tests that further prove requested functionality - Modified TestHttpsServerAuthentication to avoid redundant code in TestHTTPClientTLS by shifting a few functions into TestTlsServerUtils - Modified clients/client-java/build.gradle.kts: added test implementation to include testArtifacts published in M1 Fix: #12300 ### Does this PR introduce _any_ user-facing change? Yes, addition of TLSConfigurer + TLSConfigurer as described in M0. ### How was this patch tested? Ran: ./gradlew rat Passed ./gradlew :server-common:test --tests "org.apache.gravitino.server.web.TestHttpsServerAuthentication" Passed ./gradlew :clients:client-java:test --tests "org.apache.gravitino.client.TestHTTPClientTLS" Passed -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
