aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorUnknown <tolsen@limebits.com>2008-07-11 12:29:18 -0400
committerMatěj Cepl <mcepl@cepl.eu>2023-04-29 18:19:45 +0200
commitbe6a143eb5456c09c9c2828693e99ba60bcf213d (patch)
tree9fa2f09a791a7a8737e9c8908ff4bf88f1cb85f7
parentc17e173bcb6414f0492bfa242d06a361c3294cf3 (diff)
downloadurllib2_kerberos-be6a143eb5456c09c9c2828693e99ba60bcf213d.tar.gz
actually set the result variable
-rw-r--r--urllib2_kerberos.py8
1 files changed, 4 insertions, 4 deletions
diff --git a/urllib2_kerberos.py b/urllib2_kerberos.py
index 7711a4e..fd7b9b1 100644
--- a/urllib2_kerberos.py
+++ b/urllib2_kerberos.py
@@ -97,11 +97,11 @@ class AbstractKerberosAuthHandler:
log.critical("mutual auth failed. No negotiate header")
return None
- if k.authGSSClientStep(self.context, neg_value) < 1:
- #TODO pyflakes flagged this so I commented it out --Gar
- #
+ result = k.authGSSClientStep(self.context, neg_value)
+
+ if result < 1:
# this is a critical security warning
- # uncommenting. will change to a raise soon --Tim
+ # should change to a raise --Tim
log.critical("mutual auth failed: authGSSClientStep returned result %d" % result)
pass