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

update gokrb5 to v8.4.1 #1762

Closed
wants to merge 1 commit into from
Closed

Conversation

kmhagan
Copy link

@kmhagan kmhagan commented Jul 25, 2020

Hello!

This updates gokrb5 dependency to v8.4.1 which fixes an issue I ran into when a KDC server was unresponsive: jcmturner/gokrb5#399

@kmhagan kmhagan requested a review from bai as a code owner July 25, 2020 16:08
@ghost ghost added the cla-needed label Jul 25, 2020
@ghost ghost removed the cla-needed label Jul 25, 2020
@bai bai requested review from dnwe and d1egoaz July 25, 2020 17:14
Copy link
Contributor

@bai bai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your contribution! LGTM but I'll let another reviewer to take a look 👀

@dnwe
Copy link
Collaborator

dnwe commented Jul 25, 2020

@bai I’m pretty sure I attempted this upgrade as part of #1626 and had to revert in #1630 because it broke some users.

Essentially:

Switching to go module based import paths for dependencies would require every consumer of the client to have also switched. We should wait for Sarama/v2 before doing that

@bai
Copy link
Contributor

bai commented Jul 25, 2020

Oh good point, I missed that one 👍

@dnwe
Copy link
Collaborator

dnwe commented Jul 25, 2020

I’m happy for us to just start prepping a v2 though 😄

@kmhagan
Copy link
Author

kmhagan commented Jul 25, 2020

Looking forward to v2! I'll go ahead and close this PR and in the meantime I'll use my branch to handle this scenario

@kmhagan kmhagan closed this Jul 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants