Galsza opened a new pull request, #7162: URL: https://github.com/apache/ozone/pull/7162
## What changes were proposed in this pull request? SCMSecurityProtocol consistently returns encoded versions of certificates which causes this encoding to spread around the code. With the ongoing crypto-compliance efforts it's best to limit the places that use BouncyCastle dependencies. With this refactor it's ensured that only the proto layer translators use it on SCM side. An additional later benefit of this pull request is that the CAServer can eventually implement the SCMSecurityProtocol, which will be useful when the default certificate server is separated to root and sub ca servers. [HDDS-11417](https://issues.apache.org/jira/browse/HDDS-11417) ## How was this patch tested? No functional changes in code, existing CI runs should suffice. Here is a CI run from my fork that uses the same changeset (Although different branch because the Jira ticket was created later) https://github.com/Galsza/ozone/actions/runs/10699894112 -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
