Octavi00 opened a new pull request, #10975:
URL: https://github.com/apache/gravitino/pull/10975
Co-authored with @Vonna9 and @jujusjuicebox
[Improvement] Add tlsConfigurer for Gravitino #9836
### What changes were proposed in this pull request?
The following changes are unit tests to test the proposed SSL certificate
authentication for clients.
### Why are the changes needed?
The following changes were added to replicate the issue, which was claimed
to be the lack of mTLS, and we decided to see if improvements toward SSL
certificates were actually needed. To be able to confirm or deny such, we
decided to add a couple of tests to the
common/src/test/java/org/apache/gravitino/server/web/TestJettyServer.java to
prove if SSL certificates fail in either keystores or during client
authentications. We found that mTLS is already automatically supported by the
Gravitino Server, and clients are able to respond to the mTLS request. In such
a case, that means this Improvement wasn't actually needed and was most likely
addressed in a separate pr.
Fix: #9836
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
Using the command
./gradlew :server-common:test --tests
"org.apache.gravitino.server.web.TestJettyServer", the build succeeded with the
newly added tests proving mTLS support.
--
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]
