Octavi00 opened a new pull request, #12620: URL: https://github.com/apache/gravitino/pull/12620
[Subtask] M3: Public TLS API on the client builders, documentation, and e2e #12301 ### What changes were proposed in this pull request? Added withTlsConfigurer to the Gravitino client builder to match the M0 design doc, in order to avoid internal types in public signatures. The docs/how-to-use-gravitino-client.md was updated to include these new methods, as well as docs/security/how-to-use-https.md to use the new client TLS API instead of relying on JVM-wide truststore configuration. Added E2E tests, which were achieved by creating a real gravitino test server and utilizing a metalake, as well as a metalake operation, to test real user endpoints. This method is separate from the M1 server setup, which was a lightweight server setup; however, all fixtures were reused from the M1 test artifacts. ### Why are the changes needed? Changes were directly requested as described in M3 Fix: #12301 ### Does this PR introduce _any_ user-facing change? Yes, added tlsConfigurer to GravitinoClientBase, GravitinoAdminClient, and GravitinoClient as described in the included documentation file. ### How was this patch tested? ./gradlew rat Passed ./gradlew :clients:client-java:test --tests "org.apache.gravitino.client.TestGravitinoClientTLS" 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]
