add a note in GetPersons about a possible privacy leak

diff --git a/PLC/Accessors/Factory.py b/PLC/Accessors/Factory.py
index 9b86a8d..c6e87f3 100644 (file)
--- a/PLC/Accessors/Factory.py
+++ b/PLC/Accessors/Factory.py
@@ -45,6 +45,7 @@ taggable_classes = { Node : {'table_class' : Nodes,
 
 # xxx probably defined someplace else
 admin_roles = ['admin']
+person_roles = [ 'admin', 'pi', 'tech', 'user' ]
 all_roles = [ 'admin', 'pi', 'tech', 'user', 'node' ]
 tech_roles = [ 'admin', 'pi', 'tech' ]
 

diff --git a/PLC/Filter.py b/PLC/Filter.py
index 5dc9f9b..42589e0 100644 (file)
--- a/PLC/Filter.py
+++ b/PLC/Filter.py
@@ -1,5 +1,6 @@
-# $Id$
-# $URL$
+#
+# Thierry Parmentelat - INRIA
+#
 from types import StringTypes
 try:
     set

diff --git a/PLC/Methods/GetPersonTags.py b/PLC/Methods/GetPersonTags.py
index 44fbdea..be85340 100644 (file)
--- a/PLC/Methods/GetPersonTags.py
+++ b/PLC/Methods/GetPersonTags.py
@@ -1,10 +1,6 @@
-# $Id: GetPersonTags.py 14587 2009-07-19 13:18:50Z thierry $
-# $URL: http://svn.planet-lab.org/svn/PLCAPI/tags/PLCAPI-4.3-27/PLC/Methods/GetPersonTags.py $
 #
 # Thierry Parmentelat - INRIA
 #
-# $Revision: 14587 $
-#
 from PLC.Faults import *
 from PLC.Method import Method
 from PLC.Parameter import Parameter, Mixed

diff --git a/PLC/Methods/GetPersons.py b/PLC/Methods/GetPersons.py
index 6302765..a209b87 100644 (file)
--- a/PLC/Methods/GetPersons.py
+++ b/PLC/Methods/GetPersons.py
@@ -52,6 +52,8 @@ class GetPersons(Method):
             if not valid_person_ids:
                 return []
 
+            # xxx this looks suspicious
+            # we need to add this restriction even if person_filter is defined
             if person_filter is None:
                 person_filter = valid_person_ids
 

diff --git a/PLC/Persons.py b/PLC/Persons.py
index e0e4c84..060a46f 100644 (file)
--- a/PLC/Persons.py
+++ b/PLC/Persons.py
@@ -4,9 +4,6 @@
 # Mark Huang <mlhuang@cs.princeton.edu>
 # Copyright (C) 2006 The Trustees of Princeton University
 #
-# $Id$
-# $URL$
-#
 
 from types import StringTypes
 try:

diff --git a/PLC/TagTypes.py b/PLC/TagTypes.py
index c045a63..f552435 100644 (file)
--- a/PLC/TagTypes.py
+++ b/PLC/TagTypes.py
@@ -1,6 +1,3 @@
-# $Id$
-# $Id$
-# $URL$
 #
 # Thierry Parmentelat - INRIA
 #