bbejeck opened a new pull request, #23571: URL: https://github.com/apache/kafka/pull/23571
`applyRouting` set the top-level host/port on every FindCoordinator response, but only v0-3 have those fields. On v4+ the response failed to re-serialize and was forwarded unrewritten, so group coordinator requests bypassed the proxy and its faults. Now v0-3 responses get the top-level host/port rewritten and v4+ responses get the `coordinators` list rewritten. Added `KafkaProtocolFaultProxyTest.shouldRewriteFindCoordinatorResponsesToPointAtTheProxy` over all FindCoordinator versions; v4-6 fail without the fix. -- 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]
