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

[Doc] Add doc of OAuth2 authentication for Node.js client #14143

Closed
1 task done
momo-jun opened this issue Feb 7, 2022 · 1 comment · Fixed by #14659
Closed
1 task done

[Doc] Add doc of OAuth2 authentication for Node.js client #14143

momo-jun opened this issue Feb 7, 2022 · 1 comment · Fixed by #14659
Assignees
Labels
doc-required Your PR changes impact docs and you will update later. lifecycle/stale

Comments

@momo-jun
Copy link
Contributor

momo-jun commented Feb 7, 2022

What issue do you find in Pulsar docs?

File an issue here for the required doc effort.

Pulsar client node version 1.6 is about to release with the node.js client support of OAuth 2.0. See apache/pulsar-client-node#190 for more details.

What is your suggestion?

Add content for node.js client in the following OAuth 2.0 topic.
https://pulsar.apache.org/docs/en/security-oauth2/

Do you have any references?

apache/pulsar-client-node#190
https://github.com/streamnative/examples/pull/92/files
https://github.com/streamnative/doc-cloud/pull/130/files

Would you like to fix this issue?

Sure, I'd love to.

Note

  • I have researched my question.
@momo-jun momo-jun added the doc-required Your PR changes impact docs and you will update later. label Feb 7, 2022
@momo-jun momo-jun changed the title [Doc] [Doc] Add doc of OAuth2 authentication for Node.js client Feb 7, 2022
@github-actions
Copy link

The issue had no activity for 30 days, mark with Stale label.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-required Your PR changes impact docs and you will update later. lifecycle/stale
Projects
None yet
2 participants