From 133617ddd73d0bd52791ff23ac2128134f29fcb2 Mon Sep 17 00:00:00 2001 From: Mark Huang Date: Thu, 11 Jan 2007 06:01:55 +0000 Subject: [PATCH] - remove PLCCheckLocal functions, be more explicit about the error in each method - redact password from events message --- PLC/Methods/UpdatePerson.py | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/PLC/Methods/UpdatePerson.py b/PLC/Methods/UpdatePerson.py index bdca220..85c4a71 100644 --- a/PLC/Methods/UpdatePerson.py +++ b/PLC/Methods/UpdatePerson.py @@ -40,9 +40,10 @@ class UpdatePerson(Method): persons = Persons(self.api, [person_id_or_email]) if not persons: raise PLCInvalidArgument, "No such account" - person = persons[0] - PLCCheckLocalPerson(person,"UpdatePerson") + + if person['peer_id'] is not None: + raise PLCInvalidArgument, "Not a local account" # Authenticated function assert self.caller is not None @@ -56,9 +57,13 @@ class UpdatePerson(Method): # Logging variables self.object_ids = [person['person_id']] - self.message = 'Person %d updated: %s.' % \ - (person['person_id'], person_fields.keys()) + + # Redact password + if 'password' in person_fields: + person_fields['password'] = "Removed by API" + self.message = 'Person %d updated: %s.' % \ + (person['person_id'], person_fields.keys()) if 'enabled' in person_fields: - self.message += ' Person enabled' + self.message += ' Person enabled' return 1 -- 2.45.2