[rich7420]

What changes were proposed in this pull request?

The S3 Gateway parses signed chunk sizes as long and maps overflow to InvalidRequest, avoiding uncaught NumberFormatException. Parser-detected truncation returns IncompleteBody, and malformed trailers return MalformedTrailerError. Signature/checksum error precedence and commit guards are preserved; shared Content-Length mismatch handling remains unchanged.

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-17476

How was this patch tested?

• Local JDK 21: the full S3 Gateway unit suite passed 2065 cases; the final parser suites passed 170 cases, with no failures, errors or skips. Both build caches were disabled.
• Restoring the base parsers caused all 54 selected regression cases to fail (50 assertion failures and 4 NumberFormatException errors); restoring the fix passed.
• checkstyle.sh and module PMD passed. The configured SpotBugs 3.1.12.2 cannot analyze Java 17 class files; SpotBugs 4.8.6.8 analysis of the changed production classes reported no new findings against the base and retained one existing constructor warning.
• Fork CI: https://github.com/rich7420/ozone/actions/runs/42346941773 succeeded for head 19dd8238e0 (113 jobs passed, 2 skipped). S3 Gateway unit tests passed 2065 cases and S3 integration tests passed 383 cases, with no failures, errors or skips in either suite.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

[adoroszlai]

CLI shows the missing build-branch (fork) check:

$ gh pr checks https://github.com/apache/ozone/pull/11974 
All checks were successful
0 cancelled, 0 failing, 3 successful, 0 skipped, and 0 pending checks

   NAME                                             DESCRIPTION  ELAPSED  URL                                                                      
✓  pull request/build-branch (fork) (pull_request)               1h6m48s  https://github.com/apache/ozone/runs/128326913975
✓  Fork CI ping/ping (pull_request)                              3s       https://github.com/apache/ozone/actions/runs/42361625893/job/128371049224
✓  pull request/title (pull_request)                             8s       https://github.com/apache/ozone/actions/runs/42351747802/job/128341174003

[rich7420]

@adoroszlai thanks!!

[rich7420]

Moving this to draft for now. HDDS-17478 pulls the shared chunk and trailer parsing into a common base class, so I'd like to land that first and then move these error changes there.