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][TubeMQ] Fix misuse of SECT_TOKEN_BROKER #6446

Closed
2 tasks done
gosonzhang opened this issue Nov 7, 2022 · 0 comments · Fixed by #6447
Closed
2 tasks done

[Bug][TubeMQ] Fix misuse of SECT_TOKEN_BROKER #6446

gosonzhang opened this issue Nov 7, 2022 · 0 comments · Fixed by #6447
Assignees
@gosonzhang
Labels
component/tubemq type/bug Something is wrong
Milestone
1.4.0

Comments

@gosonzhang
Copy link
Contributor

What happened

When review the code, a reference error was found, and the configuration item Token of the Broker was used in the Master's Configure[1]

image
https://github.com/apache/inlong/blob/master/inlong-tubemq/tubemq-server/src/main/java/org/apache/inlong/tubemq/server/master/MasterConfig.java#L508

What you expected to happen

Use SECT_TOKEN_MASTER, not SECT_TOKEN_BROKER

How to reproduce

review code

Environment

Linux

InLong version

master

InLong Component

InLong TubeMQ

Are you willing to submit PR?

  • Yes, I am willing to submit a PR!

Code of Conduct
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gosonzhang gosonzhang

Labels
component/tubemq type/bug Something is wrong
Projects
None yet
Milestone
1.4.0
Development

Successfully merging a pull request may close this issue.

[INLONG-6446][TubeMQ] Fix misuse of SECT_TOKEN_BROKER
2 participants
@gosonzhang @dockerzhang