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

[Feature]: Add SSL/TLS authentication support with certs for external kafka #27563

Open
1 task done
lashsingh opened this issue Oct 9, 2023 · 3 comments
Open
1 task done
Assignees
Labels
kind/feature Issues related to feature request from users

Comments

@lashsingh
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Is your feature request related to a problem? Please describe.

Please see here: #27486

Describe the solution you'd like.

Add the option to enable SSL/TLS authentication for external kafka along with the option to provide the certifications location.

Describe an alternate solution.

None found to make it work with the Milvus release v2.3.0

Anything else? (Additional Context)

It'll be great to get the timeline to add this support, so we use Milvus becuase our current setup uses Kafka as a Service and only allows authentication via certificates. Thanks.

@lashsingh lashsingh added the kind/feature Issues related to feature request from users label Oct 9, 2023
@xiaofan-luan
Copy link
Collaborator

@jaime0815
did we already support this feature?

@prrs
Copy link

prrs commented Dec 30, 2024

Referring these:

  1. https://github.com/apache/kafka/pull/10007/files, and
  2. 2009c3c

It looks to me if we are presenting the client certificate it's mTLS. TLS wouldn't require client to present it's certificate.

If this is true, this feature should be mark as complete as current milvus(even 2.3.x) support mTLS.

@xiaofan-luan @jaime0815 @yhmo

@xiaofan-luan
Copy link
Collaborator

Referring these:

  1. https://github.com/apache/kafka/pull/10007/files, and
  2. 2009c3c

It looks to me if we are presenting the client certificate it's mTLS. TLS wouldn't require client to present it's certificate.

If this is true, this feature should be mark as complete as current milvus(even 2.3.x) support mTLS.

@xiaofan-luan @jaime0815 @yhmo

Yes, that is what we thoughts. mTLS means user specify certification and we already support it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Issues related to feature request from users
Projects
None yet
Development

No branches or pull requests

3 participants