aboutsummaryrefslogtreecommitdiffstats
path: root/dont-panick-if-we-succeed.patch
diff options
context:
space:
mode:
authorMatěj Cepl <mcepl@cepl.eu>2023-04-29 19:46:45 +0200
committerMatěj Cepl <mcepl@cepl.eu>2023-04-29 19:46:45 +0200
commit8d87c8065d6762c0a32a4d7cbe11535bb3fcb148 (patch)
tree65812e7ad4abdee38d84d9e308fbf34c309e965c /dont-panick-if-we-succeed.patch
parent1fcb556ed8b14f6f13144df0102ea2b86eaea884 (diff)
downloadurllib2_kerberos-8d87c8065d6762c0a32a4d7cbe11535bb3fcb148.tar.gz
Incorporate dont-panick-if-we-succeed.patch
Diffstat (limited to 'dont-panick-if-we-succeed.patch')
-rw-r--r--dont-panick-if-we-succeed.patch79
1 files changed, 0 insertions, 79 deletions
diff --git a/dont-panick-if-we-succeed.patch b/dont-panick-if-we-succeed.patch
deleted file mode 100644
index 050a023..0000000
--- a/dont-panick-if-we-succeed.patch
+++ /dev/null
@@ -1,79 +0,0 @@
---- a/urllib2_kerberos.py
-+++ b/urllib2_kerberos.py
-@@ -42,6 +42,7 @@ class AbstractKerberosAuthHandler:
- """checks for "Negotiate" in proper auth header
- """
- authreqs = headers.getheaders(self.auth_header)
-+ log.debug('authreqs = %s', authreqs)
-
- if authreqs:
-
-@@ -51,10 +52,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
-
-@@ -64,10 +65,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
-@@ -75,7 +76,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")
-@@ -83,7 +84,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")
-@@ -104,7 +105,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):
-@@ -134,12 +135,13 @@ class AbstractKerberosAuthHandler:
- req.add_unredirected_header(self.authz_header, neg_hdr)
- resp = self.parent.open(req)
-
-- self.authenticate_server(resp.info())
-+ if resp.getcode() != 200:
-+ self.authenticate_server(resp.info())
-
- 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: