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

Evaluate options for Kerberos auth #15619

Closed
andresrc opened this issue Jan 16, 2020 · 3 comments
Closed

Evaluate options for Kerberos auth #15619

andresrc opened this issue Jan 16, 2020 · 3 comments
Assignees
Labels
Team:Integrations Label for the Integrations team Team:Services (Deprecated) Label for the former Integrations-Services team

Comments

@andresrc
Copy link
Contributor

We have outstanding requests (#5413, #7404) to add Kerberos support for the communication with both Elasticsearch and Kafka.

The goal of this issue is to evaluate the currently available options and define an action plan.

Some considerations:

  • The client library that we are using for Kafka seems to have added some support
  • The preference would be to use the same library for both Elasticsearch and Kafka
@andresrc andresrc added [zube]: Inbox Team:Services (Deprecated) Label for the former Integrations-Services team [zube]: Ready and removed [zube]: Inbox labels Jan 27, 2020
@kvch kvch self-assigned this Jan 31, 2020
@kvch
Copy link
Contributor

kvch commented Feb 24, 2020

Two years ago there was another round of research regarding which Kerberos library to use: #7404

The lib selected by @andrewkroh is called github.com/jcmturner/gokrb5. Since then, the lib has matured a lot and is still maintained. New releases are available regularly. Furthermore, sarama uses this library of v7.5.0. So by choosing this lib, we would be able to support Elasticserach and Kafka using one dependency.

There are a few honourable mentions out there both pure Go and with CGO. But all of those are abandoned. So I would not opt for those.

All in all, I agree with the choice of Andrew.

@urso WDYT?

@urso
Copy link

urso commented Feb 24, 2020

Sounds good. +1 using gokbr5.

@kvch
Copy link
Contributor

kvch commented Feb 25, 2020

Closing as a desicion has been made to use github.com/jcmturner/gokrb5.

@kvch kvch closed this as completed Feb 25, 2020
@kvch kvch mentioned this issue Feb 25, 2020
10 tasks
@andresrc andresrc added Team:Integrations Label for the Integrations team and removed [zube]: Done labels Mar 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Integrations Label for the Integrations team Team:Services (Deprecated) Label for the former Integrations-Services team
Projects
None yet
Development

No branches or pull requests

3 participants