--- a/urllib2_kerberos.py
+++ b/urllib2_kerberos.py
@@ -34,7 +34,7 @@ class AbstractKerberosAuthHandler:
"""checks for "Negotiate" in proper auth header
"""
authreqs = headers.getheaders(self.auth_header)
- log.debug('authreqs = {}'.format(authreqs))
+ log.debug('authreqs = %s', authreqs)
if authreqs:
@@ -44,10 +44,10 @@ class AbstractKerberosAuthHandler:
if mo:
return mo.group(1)
else:
- log.debug("regex failed on: %s" % authreq)
+ log.debug("regex failed on: %s", authreq)
else:
- log.debug("%s header not found" % self.auth_header)
+ log.debug("%s header not found", self.auth_header)
return None
@@ -57,10 +57,10 @@ class AbstractKerberosAuthHandler:
def generate_request_header(self, req, headers, neg_value):
self.retried += 1
- log.debug("retry count: %d" % self.retried)
+ log.debug("retry count: %d", self.retried)
host = req.get_host()
- log.debug("req.get_host() returned %s" % host)
+ log.debug("req.get_host() returned %s", host)
tail, sep, head = host.rpartition(':')
domain = tail if tail else head
@@ -68,7 +68,7 @@ class AbstractKerberosAuthHandler:
result, self.context = k.authGSSClientInit("HTTP@%s" % domain)
if result < 1:
- log.warning("authGSSClientInit returned result %d" % result)
+ log.warning("authGSSClientInit returned result %d", result)
return None
log.debug("authGSSClientInit() succeeded")
@@ -76,7 +76,7 @@ class AbstractKerberosAuthHandler:
result = k.authGSSClientStep(self.context, neg_value)
if result < 0:
- log.warning("authGSSClientStep returned result %d" % result)
+ log.warning("authGSSClientStep returned result %d", result)
return None
log.debug("authGSSClientStep() succeeded")
@@ -97,7 +97,7 @@ class AbstractKerberosAuthHandler:
if result < 1:
# this is a critical security warning
# should change to a raise --Tim
- log.critical("mutual auth failed: authGSSClientStep returned result %d" % result)
+ log.critical("mutual auth failed: authGSSClientStep returned result %d", result)
pass
def clean_context(self):
@@ -133,7 +133,7 @@ class AbstractKerberosAuthHandler:
return resp
except k.GSSError, e:
- log.critical("GSSAPI Error: %s/%s" % (e[0][0], e[1][0]))
+ log.critical("GSSAPI Error: %s/%s", (e[0][0], e[1][0]))
return None
finally: