michaeljmarshall opened a new pull request, #20353:
URL: https://github.com/apache/pulsar/pull/20353

   ### Motivation
   
   This PR builds on some of my recent changes here: 
   * https://github.com/apache/pulsar/pull/20289 
   * https://github.com/apache/pulsar/pull/20336
   * https://github.com/apache/pulsar/pull/20348
   
   This PR removes several TLS certs that are unnecessary and replaces them 
with ones already created in the `tests/certificate-authority` directory.
   
   The goal is to simplify our test framework so that future updates to 
certificates are easy.
   
   ### Modifications
   
   * Update several tests to use the `tests/certificate-authority` certs.
   * Update the way that `testWebserviceServiceTls` configures the HTTP client 
for TLS and add some requirements on the broker to ensure that the client is 
connecting and providing TLS certs.
   * Remove the unused certs
   
   ### Verifying this change
   
   The change is fairly straight forward and the fact that the tests pass is 
key.
   
   ### Documentation
   
   - [x] `doc-not-needed`
   
   ### Matching PR in forked repository
   
   PR in forked repository: https://github.com/michaeljmarshall/pulsar/pull/50


-- 
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]

Reply via email to