Skip to content

Commit 52bc85d

Browse files
authored
Cleanup gssapi code; fix bug report re AttributeError. (#1262)
1 parent cd4d01c commit 52bc85d

File tree

1 file changed

+9
-10
lines changed

1 file changed

+9
-10
lines changed

kafka/conn.py

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -532,20 +532,19 @@ def _try_authenticate_plain(self, future):
532532
return future.success(True)
533533

534534
def _try_authenticate_gssapi(self, future):
535-
data = b''
536-
gssname = self.config['sasl_kerberos_service_name'] + '@' + self.hostname
537-
ctx_Name = gssapi.Name(gssname, name_type=gssapi.NameType.hostbased_service)
538-
ctx_CanonName = ctx_Name.canonicalize(gssapi.MechType.kerberos)
539-
log.debug('%s: canonical Servicename: %s', self, ctx_CanonName)
540-
ctx_Context = gssapi.SecurityContext(name=ctx_CanonName, usage='initiate')
541-
log.debug("%s: initiator name: %s", self, ctx_Context.initiator_name)
535+
gssapi_name = gssapi.Name(
536+
self.config['sasl_kerberos_service_name'] + '@' + self.hostname,
537+
name_type=gssapi.NameType.hostbased_service
538+
).canonicalize(gssapi.MechType.kerberos)
539+
log.debug('%s: GSSAPI name: %s', self, gssapi_name)
542540

543541
# Exchange tokens until authentication either succeeds or fails
542+
client_ctx = gssapi.SecurityContext(name=gssapi_name, usage='initiate')
544543
received_token = None
545544
try:
546-
while not ctx_Context.complete:
545+
while not client_ctx.complete:
547546
# calculate an output token from kafka token (or None if first iteration)
548-
output_token = ctx_Context.step(received_token)
547+
output_token = client_ctx.step(received_token)
549548

550549
# pass output token to kafka
551550
try:
@@ -570,7 +569,7 @@ def _try_authenticate_gssapi(self, future):
570569
except Exception as e:
571570
return future.failure(e)
572571

573-
log.info('%s: Authenticated as %s via GSSAPI', self, gssname)
572+
log.info('%s: Authenticated as %s via GSSAPI', self, gssapi_name)
574573
return future.success(True)
575574

576575
def blacked_out(self):

0 commit comments

Comments
 (0)