harshkha3417 commented on PR #7821: URL: https://github.com/apache/shenyu/pull/7821#issuecomment-6330391404
Thanks @Aias00 for the thorough review and explanation regarding the flaky `shenyu-integrated-test-sdk-http` run! I have addressed all non-blocking observations: 1. **Release Notes Documentation:** Documented the `headerMaxSize` calculation change under `Behavior Changes` in `RELEASE-NOTES.md` so operators with tight limits are aware of the wire-accurate counting. 2. **Javadoc Clarification:** Clarified that `calculateHeaderSize` measures the header field section (excluding request line / pseudo-headers) and explicitly returns `0L` for null/empty headers. 3. **Defensive Cleanup:** Removed redundant `StringUtils.defaultString` on the non-null `HttpHeaders` map keys. 4. Pushed updates to trigger a clean CI workflow run. -- 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]
