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

Ipv6环境下,io.seata.common.util.NetUtil的toInetSocketAddress方法没办法正确解析ipv6格式的地址 #5607

Closed
1 task done
xiaoqiang-github-123 opened this issue May 25, 2023 · 1 comment · Fixed by #5902
Labels
type: bug Category issues or prs related to bug.

Comments

@xiaoqiang-github-123
Copy link

xiaoqiang-github-123 commented May 25, 2023

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

Ⅰ. Issue Description

当seataf服务部署在ipv6环境时,注册到注册中心上的seata服务地址是ipv6形式的,当服务导入seata依赖时,发现如图的这个工具类里对ip地址的解析有问题,会出现截取的host和port不正确

image

Ⅱ. Describe what happened

If there is an exception, please attach the exception trace:

 i.s.c.r.netty.NettyClientChannelManager  : 0101 can not connect to 2202:0:0:0:0:0:0:133:9908 cause:can not register RM,err:For input string: "0:0:0:0:0:0:133:9908"
io.seata.common.exception.FrameworkException: can not register RM,err:For input string: "0:0:0:0:0:0:133:9908"
....

Ⅲ. Describe what you expected to happen

Ⅳ. How to reproduce it (as minimally and precisely as possible)

  1. xxx
  2. xxx
  3. xxx

Minimal yet complete reproducer code (or URL to code):

Ⅴ. Anything else we need to know?

Ⅵ. Environment:

  • JDK version(e.g. java -version): 1.8.0_191-b12
  • Seata client/server version: latest
  • Database version:
  • OS(e.g. uname -a):
  • Others:
@slievrly
Copy link
Member

@xiaoqiang-github-123 can you submit a PR to fix it?

@funky-eyes funky-eyes added the type: bug Category issues or prs related to bug. label May 25, 2023
xiaoqiang-github-123 added a commit to xiaoqiang-github-123/seata that referenced this issue Jul 22, 2023
fix bug: Ipv6环境下,io.seata.common.util.NetUtil的toInetSocketAddress方法没办法正确解析ipv6格式的地址 

issue: apache#5607
@Ifdevil Ifdevil mentioned this issue Oct 6, 2023
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Category issues or prs related to bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants