From: Thierry Parmentelat <thierry.parmentelat@inria.fr>
Date: Tue, 9 Apr 2019 20:06:44 +0000 (+0200)
Subject: some odd code was obviously broken, it was using a str variable and then
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=b18eb6202dbcaccf80d53c533a22f7d4bf733c54;p=sfa.git

some odd code was obviously broken, it was using a str variable and then
checking if that was instance of .. str
---

diff --git a/sfa/trust/credential.py b/sfa/trust/credential.py
index 92c114e3..8a6c0bbc 100644
--- a/sfa/trust/credential.py
+++ b/sfa/trust/credential.py
@@ -286,21 +286,19 @@ class Credential(object):
                 self.version = cred['geni_version']
 
         if string or filename:
-            if string:
-                str = string
-            elif filename:
+            if not string:
                 with open(filename) as infile:
-                    str = infile.read()
+                    string = infile.read()
 
             # if this is a legacy credential, write error and bail out
-            if isinstance(str, str) and str.strip().startswith("-----"):
+            if isinstance(string, str) and string.strip().startswith("-----"):
                 logger.error(
                     "Legacy credentials not supported any more "
                     "- giving up with {}..."
                     .format(str[:10]))
                 return
             else:
-                self.xml = str
+                self.xml = string
                 self.decode()
         # not strictly necessary but won't hurt either
         self.get_xmlsec1_path()