89799969 opened a new pull request, #11052: URL: https://github.com/apache/rocketmq/pull/11052
### Which Issue(s) This PR Fixes - Fixes #10745 ### Brief Description The HTTP/2 proxy previously closed only the channel returned by a failed write, leaving its paired channel open. The exception paths had the same asymmetric cleanup. This can retain an unusable frontend or backend connection after the other side has failed. This change closes both paired channels on write failures and handler exceptions in `Http2ProxyBackendHandler` and `Http2ProxyFrontendHandler`. It also adds focused regression coverage for all four failure paths. ### How Did You Test This Change? ```text mvn -Dmaven.repo.local=D:\projects\agentUniverse\tmp\m2-rocketmq -pl proxy -am -Dtest=Http2ProxyChannelCleanupTest -Dsurefire.failIfNoSpecifiedTests=false -DskipITs test Tests run: 4, Failures: 0, Errors: 0, Skipped: 0 Reactor modules: 11/11 SUCCESS BUILD SUCCESS ``` The same run completed Checkstyle and SpotBugs successfully for the reactor modules, including `rocketmq-proxy`. AI-assisted contribution; implementation and tests were reviewed and verified locally. -- 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]
