rich7420 opened a new pull request, #11460: URL: https://github.com/apache/ozone/pull/11460
## What changes were proposed in this pull request? Canonical URI construction currently compiles a regex to split URI path segments for each S3 signature. Use the existing commons-lang3 `StringUtils.splitPreserveAllTokens` with a slash separator and remove the regex helper. Leading, trailing, and repeated separators remain preserved, and each segment is encoded as before. Parameterized tests cover empty paths, repeated separators, dot segments, percent escapes, spaces, and Unicode. ## What is the link to the Apache JIRA? https://issues.apache.org/jira/browse/HDDS-16749 ## How was this patch tested? - Six S3 signature test classes: 68 tests passed locally; checkstyle and S3 Gateway PMD passed. - [Latest fork CI](https://github.com/rich7420/ozone/actions/runs/37800764520): OM integration failed in `testClientWithLocalLeaseEnabled`, client integration failed in `TestFailureHandlingByClient.testContainerExclusionWithClosedContainerException`, and Kubernetes failed in the Freon Ozone Client Key Generator and Key Validator. These paths do not use S3 canonical URI construction. Other checks that ran passed. Generated-by: Codex (GPT-6) -- 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]
