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

Add ability to connect to bookkeeper using tls #3376

Closed
bputt opened this issue Jan 16, 2019 · 9 comments
Closed

Add ability to connect to bookkeeper using tls #3376

bputt opened this issue Jan 16, 2019 · 9 comments
Labels
area/security help wanted type/feature The PR added a new feature or issue requested a new feature

Comments

@bputt
Copy link

bputt commented Jan 16, 2019

Is your feature request related to a problem? Please describe.
I have a bookkeeper cluster that has TLS enabled and need pulsar clients to communicate to my cluster using tls

Describe the solution you'd like
The ability to specify tls settings for my broker to connect to bookkeeper

Describe alternatives you've considered
Using stunnel + iptables to redirect

@bputt bputt added the type/feature The PR added a new feature or issue requested a new feature label Jan 16, 2019
@sijie
Copy link
Member

sijie commented Jan 17, 2019

@bputt just try to clarify your requirement here:

are you looking for "connecting your pulsar clients to brokers via TLS" or "connecting your pulsar broker to bookies via TLS"?

@bputt
Copy link
Author

bputt commented Jan 17, 2019

@sijie "connecting your pulsar broker to bookies via TLS"

@sijie
Copy link
Member

sijie commented Jan 17, 2019

@bputt gotcha, thank you for clarification.

@joefk
Copy link
Contributor

joefk commented Jan 17, 2019

👍

@one70six
Copy link

one70six commented Apr 9, 2019

This functionality would be very useful for running a secure cluster and allowing a separate Pulsar functions cluster to also communicate with BK. Super interested in having that capability.

@jiazhai
Copy link
Member

jiazhai commented Jul 30, 2019

For TLS, It need these configs:
http://bookkeeper.apache.org/docs/latest/security/tls/#configuring-clients

We need add and support these config in BookKeeperClientFactoryImpl, which is similar to bk client config in it.

@david-streamlio
Copy link
Contributor

Any update on this issue? Is there an ETA for this?

@rdhabalia
Copy link
Contributor

@david-streamlio
I think we have added broker to bookie mTLS support with this PR: #5042

https://github.com/apache/pulsar/blob/master/pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java#L792-L818

Is that you are looking for?

@david-streamlio
Copy link
Contributor

Yes, thank you!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/security help wanted type/feature The PR added a new feature or issue requested a new feature
Projects
None yet
Development

No branches or pull requests

7 participants