From cfde720b1dc5cea22683878b208b736618a606ed Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sybren=20A=2E=20St=C3=BCvel?= Date: Wed, 23 Aug 2017 15:37:03 +0200 Subject: [PATCH] Orgs: PATCH op to batch-add emails as members now strip()s emails It also refuses to add empty emails. --- pillar/api/organizations/patch.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/pillar/api/organizations/patch.py b/pillar/api/organizations/patch.py index 98d8c563..7cbce1a1 100644 --- a/pillar/api/organizations/patch.py +++ b/pillar/api/organizations/patch.py @@ -33,8 +33,10 @@ class OrganizationPatchHandler(patch_handler.AbstractPatchHandler): except KeyError: raise wz_exceptions.BadRequest('No key "email" in patch.') - if not all(isinstance(email, str) for email in emails): - raise wz_exceptions.BadRequest('Invalid list of email addresses') + # Skip empty emails. + emails = [stripped + for stripped in (email.strip() for email in emails) + if stripped] log.info('User %s uses PATCH to add users to organization %s', current_user().user_id, org_id)