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.

Sign up for GitHub

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

Jump to bottom

[Bug][DataProxy] The IP config was invalid #6602

Closed
1 of 2 tasks
FvvSuu opened this issue Nov 22, 2022 · 2 comments · Fixed by #6603
Closed
1 of 2 tasks

[Bug][DataProxy] The IP config was invalid #6602

FvvSuu opened this issue Nov 22, 2022 · 2 comments · Fixed by #6603
Assignees
@gosonzhang
Labels
component/dataproxy type/bug Something is wrong
Milestone
1.5.0

Comments

@FvvSuu
Copy link

FvvSuu commented Nov 22, 2022

What happened

ip config not valid.
It may be a regular expression validation problem

What you expected to happen

image

How to reproduce

Fill in the actual IP of the server in dataproxy-pulsar.conf

Environment

No response

InLong version

1.4.0

InLong Component

InLong DataProxy

Are you willing to submit PR?

  • Yes, I am willing to submit a PR!

Code of Conduct
@FvvSuu FvvSuu added the type/bug Something is wrong label Nov 22, 2022
@gosonzhang
Copy link
Contributor

Thanks @FvvSuu , let me see it

@gosonzhang
Copy link
Contributor

isValidIp() has a bug, let me fix it.

image
https://github.com/apache/inlong/blob/master/inlong-dataproxy/dataproxy-source/src/main/java/org/apache/inlong/dataproxy/utils/ConfStringUtils.java#L33

@gosonzhang gosonzhang self-assigned this Nov 22, 2022
@gosonzhang gosonzhang mentioned this issue Nov 22, 2022
Merged
@dockerzhang dockerzhang added this to the 1.5.0 milestone Nov 22, 2022
@healchow healchow changed the title [Bug][DataProxy] ip config not valid [Bug][DataProxy] The IP config was invalid Nov 22, 2022
@gosonzhang gosonzhang mentioned this issue Nov 22, 2022
Closed
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gosonzhang gosonzhang

Labels
component/dataproxy type/bug Something is wrong
Projects
None yet
Milestone
1.5.0
Development

Successfully merging a pull request may close this issue.

[INLONG-6602][DataProxy] Fix the IP config was invalid gosonzhang/inlong
3 participants
@gosonzhang @dockerzhang @FvvSuu