-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[Doc] Update websocket docs to suggest passing the token as headers rather than query parameters #23625
Comments
I want to do it if no one want. could you assign it to me? @lhotari |
sure, please go ahead. Just a tip: using generative AI such as ChatGPT is good for fixing grammar issues. |
@walkinggo You should first update the documentation files in the docs directory and then in the versioned_docs ONLY for supported versions (version-4.0.x and version-3.0.x). More details in https://github.com/apache/pulsar-site?tab=readme-ov-file#updating-versioned-docs and https://pulsar.apache.org/contribute/document-contribution/#update-versioned-docs . |
Thank you ! I will try my best to finish it. |
I would like to resolve this issue, but could you please explain the reason? You suggested passing the token as a header rather than as a query parameter, but why is that the case? @lhotari |
take a look,please. pr981 |
Search before asking
What issue do you find in Pulsar docs?
https://pulsar.apache.org/docs/4.0.x/client-libraries-websocket/#authentication
What is your suggestion?
suggest passing the token as headers rather than query parameters
Any reference?
No response
Are you willing to submit a PR?
The text was updated successfully, but these errors were encountered: