-
Notifications
You must be signed in to change notification settings - Fork 133
Add support for client auth using TLS #325
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
base: master
Are you sure you want to change the base?
Conversation
@lhotari can we get assistance in getting this PR merged? |
Thanks for the contribution @alesharik. Would it be possible to add some tests to validate the added feature? |
@freeznet @mattisonchao Please review this PR. |
Hi, @lhotari |
@alesharik Thanks! There doesn't seem to be very many tests in the project currently. Improving on that in this area could require more effort than usual because of the lack of examples of how things are handled in pulsar-rs. |
@lhotari hi! |
Closes #197
Fork of PR #267 , and it's rebase attempt, because seems like original PR is unmaintained