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

Support SASL/GSSAPI authentication #287

Closed
wants to merge 1 commit into from

Conversation

mkomitee
Copy link

@mkomitee mkomitee commented Mar 3, 2015

To trigger SASL/GSSAPI authentication, add an auth_data tuple of ("gssapi", None) to the KazooClient. It uses the users current Kerberos credential cache to authenticate.

This was developed against 1.3.1, and so I fully expect it to not merge against the current HEAD of kazoo.

Furthermore it was developed against an older version of PyGSSAPI which included a bug. I fixed that bug in my own copy of PyGSSAPI, but when I went to submit the fix upstream, PyGSSAPI had been significantly modified and it was unbuildable at the time.

I'm only submitting this PR to serve as a reference for #130 and #284.

To trigger SASL/GSSAPI authentication, add an auth_data tuple
of (gssapi, None) to the KazooClient.
@bbangert
Copy link
Member

Ok, so I'm closing this then since there's no intention to update it to cleanly merge, tests, etc. It also add's the requirement to kazoo to install PyGSSAPI, which many people don't need so it shouldn't be there.

@bbangert bbangert closed this Mar 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants