michaeljmarshall opened a new pull request, #16045: URL: https://github.com/apache/pulsar/pull/16045
### Motivation When reading through the Proxy code, I noticed that the `inboundChannel` was used where it probably makes sense to use the `outboundChannel`. I am not sure if I've proposed the correct solution, so I'll need someone more familiar to verify this PR's correctness. Note that a few lines above the changed code, you can see that there is this assertion: `outboundChannel.localAddress() instanceof InetSocketAddress`. ### Modifications * Replace `inboundChannel.remoteAddress` with `outboundChannel.localAddress`. ### Verifying this change I probably need to add a test, but I haven't focused on that yet. I'd like to confirm that this is indeed the right solution first. -- 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]
