Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Error parsing ipv6 from address in HAProxyMessageForwarder #8259

Closed
3 tasks done
yuz10 opened this issue Jun 5, 2024 · 0 comments · Fixed by #8260
Closed
3 tasks done

[Bug] Error parsing ipv6 from address in HAProxyMessageForwarder #8259

yuz10 opened this issue Jun 5, 2024 · 0 comments · Fixed by #8260

Comments

@yuz10
Copy link
Member

yuz10 commented Jun 5, 2024

Before Creating the Bug Report

  • I found a bug, not just asking a question, which should be created in GitHub Discussions.

  • I have searched the GitHub Issues and GitHub Discussions of this repository and believe that this is not a duplicate.

  • I have confirmed that this bug belongs to the current repository, not other repositories of RocketMQ.

Runtime platform environment

Ubuntu 24.04

RocketMQ version

5.2.0

JDK Version

1.8

Describe the Bug

image

Steps to Reproduce

use ipv6 localhost (::1)

What Did You Expect to See?

succuss

What Did You See Instead?

Forward HAProxyMessage from Remoting to gRPC server error.
java.lang.IllegalArgumentException: invalid IPv4 address: 0
at io.netty.handler.codec.haproxy.HAProxyMessage.checkAddress(HAProxyMessage.java:444)
at io.netty.handler.codec.haproxy.HAProxyMessage.(HAProxyMessage.java:101)
at test.HAProxyMessageForwarder.buildHAProxyMessage(HAProxyMessageForwarder.java:151)
at test.HAProxyMessageForwarder.forwardHAProxyMessage(HAProxyMessageForwarder.java:76)
at test.HAProxyMessageForwarder.channelRead(HAProxyMessageForwarder.java:61)

Additional Context

No response

yuz10 added a commit that referenced this issue Sep 12, 2024
* fix parse ipv6 from address in haproxy

* more
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant