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][Manager] No token field in the return result of getAllConfig #7810

Closed
2 tasks done
gosonzhang opened this issue Apr 10, 2023 · 2 comments · Fixed by #7811
Closed
2 tasks done

[Bug][Manager] No token field in the return result of getAllConfig #7810

gosonzhang opened this issue Apr 10, 2023 · 2 comments · Fixed by #7811
Assignees
Labels
Milestone

Comments

@gosonzhang
Copy link
Contributor

What happened

When DataProxy accesses a Pulsar cluster with authentication and authorization enabled, DataProxy will report an authorization error. Through inspection, it is found that when the Manager obtains the metadata information of the Pulsar cluster through the getConfig method, the returned result carries the token authorization access information of the Pulsar cluster, but through When the getAllConfig method obtains the metadata information of the Pulsar cluster, the token authorization access information of the Pulsar cluster is not carried

What you expected to happen

When the Pulsar cluster metadata information is obtained through the getAllConfig method, the returned result carries the token authorization access information of the Pulsar cluster

How to reproduce

Enable Pulsar cluster authentication authorization check, and then report data to the Pulsar cluster through DataProxy

Environment

Linux

InLong version

master

InLong Component

InLong Manager, InLong DataProxy

Are you willing to submit PR?

  • Yes, I am willing to submit a PR!

Code of Conduct

@fuweng11
Copy link
Contributor

I will deal with this problme. And add token information to CacheClusterObject and CacheCluster, convert the token information to json, and send it to dataproxy.

@gosonzhang
Copy link
Contributor Author

Thanks @fuweng11

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