harshkha3417 commented on PR #7821: URL: https://github.com/apache/shenyu/pull/7821#issuecomment-6258763209
Thanks @Aias00 for the thorough review and approval! 1. **Rebase & master alignment:** Rebased the branch on the latest `master`. It applies cleanly without conflicts. 2. **UTF-8 comment:** Added an inline comment documenting that UTF-8 is used as a conservative upper bound across HTTP/1.1 and HTTP/2 without protocol branching. 3. **#7820 compatibility:** Since the two changes are functionally compatible and #7821 is approved, if #7821 lands first, #7820 can trivially rebase and retain `calculateHeaderSize(effectiveExchange.getRequest().getHeaders())`. -- 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]
