Nicer error message on admin not being a group.

[invirt/packages/invirt-web.git] / code / cache_acls.py

diff --git a/code/cache_acls.py b/code/cache_acls.py
index 34d5e1e..f7575e1 100644 (file)
--- a/code/cache_acls.py
+++ b/code/cache_acls.py
@@ -28,12 +28,19 @@ def expandName(name):
         if isUser(name):
             return [name]
         name = 'system:'+name
-    return getafsgroups.getAfsGroupMembers(name, 'athena.mit.edu')
+    try:
+        return getafsgroups.getAfsGroupMembers(name, 'athena.mit.edu')
+    except getafsgroups.AfsProcessError:
+        return []
 
-def refreshMachine(m):
+def accessList(m):
     people = set()
     people.update(expandLocker(m.owner))
     people.update(expandName(m.administrator))
+    return people
+
+def refreshMachine(m):
+    people = accessList(m)
     old_people = set(a.user for a in m.acl)
     for removed in old_people - people:
         ma = [x for x in m.acl if x.user == removed][0]