Skip to content

Support for TLS Authentication #197

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
Open
Tracked by #224
jmanno01 opened this issue May 4, 2022 · 3 comments · May be fixed by #267 or #325
Open
Tracked by #224

Support for TLS Authentication #197

jmanno01 opened this issue May 4, 2022 · 3 comments · May be fixed by #267 or #325
Assignees
@DonghunLouisLee

Comments

@jmanno01
Copy link

jmanno01 commented May 4, 2022 •
edited
Loading

In the current pulsar crate, it is mentioned as a feature: TLS connection,
but when using fn with_auth() of the Struct pulsar::producer::Producer::Builder, it does not support TLS Authentication.

The current Authentication support:pub struct Authentication { /// Authentication kid. Use "token" for JWT pub name: String, /// Authentication data pub data: Vec<u8>, }

coming from connection.rs of the crate

Is there a way to use a TLS Authentication method, because there is no clear way to do so as for now
@DonghunLouisLee DonghunLouisLee self-assigned this Jun 8, 2022
@DonghunLouisLee DonghunLouisLee added good first issue Good for newcomers and removed good first issue Good for newcomers labels Jun 8, 2022
@DonghunLouisLee
Copy link
Collaborator

DonghunLouisLee commented Jun 10, 2022 •
edited
Loading

@jmanno01 could you elaborate on what TLS Authenticatin means? 'with_certificate_chain()' method should be used for tls authentication for the client. Authentication in the crate refers to "authenticating" user such as jwt or Oauth2

@DonghunLouisLee DonghunLouisLee mentioned this issue Aug 28, 2022
Open
7 tasks
@afonsosribeiro
Copy link
Contributor

afonsosribeiro commented Mar 16, 2023 •
edited
Loading

The client being able to validate the server certificate is already supported through with_certificate_chain().
But the client still can't provide it's certificate and private key to the server.

afonsosribeiro added a commit to afonsosribeiro/pulsar-rs that referenced this issue Mar 16, 2023
@afonsosribeiro
Added support for client TLS authentication closes streamnative#197
79c8b40
@afonsosribeiro afonsosribeiro linked a pull request Mar 21, 2023 that will close this issue
Open
@alesharik alesharik linked a pull request Sep 18, 2024 that will close this issue
Open
@lightistor
Copy link

@lhotari the linked PR is from a colleague of mine who had implemented it for our internal use, can we anyone to review and merge it?

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

@DonghunLouisLee DonghunLouisLee

Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@lightistor @jmanno01 @DonghunLouisLee @afonsosribeiro