Changeset d90f0fa for fedd

Timestamp:

Nov 24, 2008 2:17:29 PM (16 years ago)

Author:

Ted Faber <faber@…>

Branches:

axis_example, compt_changes, info-ops, master, version-1.30, version-2.00, version-3.01, version-3.02

Children:

516b7cc

Parents:

34bc05c

Message:

Bug checking multiple access requests. The code didn't notice when a request
succeeded

File:

• fedd/fedd_experiment_control.py (modified) (3 diffs)

fedd/fedd_experiment_control.py

@@ -938 +938 @@
 
         for p, u in access_user:
+            self.log.debug(("[get_access] Attempting access from (%s, %s) " + \
+                    "to %s") %  ((p or "None"), u, uri))
 
             if p:
@@ -985 +987 @@
             except service_error, e:
                 if e.code == service_error.access:
+                    self.log.debug("[get_access] Access denied")
                     r = None
                     continue
@@ -991 +994 @@
 
             if r.has_key('RequestAccessResponseBody'):
+                # Through to here we have a valid response, not a fault.
+                # Access denied is a fault, so something better or worse than
+                # access denied has happened.
                 r = r['RequestAccessResponseBody']
+                self.log.debug("[get_access] Access granted")
+                break
             else:
                 raise service_error(service_error.protocol,