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

updating configs to resolve #1263 #1264

Merged
merged 1 commit into from
Oct 17, 2017
Merged

Conversation

natedogs911
Copy link
Contributor

updated producer and consumer configs to resolve assertion error when using sasl kerberos service name
see #1263

@dpkp dpkp merged commit 13752d7 into dpkp:master Oct 17, 2017
@natedogs911 natedogs911 deleted the config-update branch October 18, 2017 03:57
@dpkp
Copy link
Owner

dpkp commented Oct 21, 2017

@natedogs911 With this change landed is SASL/GSSAPI/kerberos now working for you?

@natedogs911
Copy link
Contributor Author

natedogs911 commented Oct 21, 2017 via email

@dpkp
Copy link
Owner

dpkp commented Oct 21, 2017

Ok, let me know if you find anything. I am very interested in getting this to work smoothly.

@dpkp
Copy link
Owner

dpkp commented Oct 22, 2017

fyi, I noticed that this PR was missing an update to KafkaClient. That is required to make sure the configuration is pushed down to the client's BrokerConnections. I pushed the fixup to master.

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.

2 participants