archive/pillar
0
0
Fork 0
Code Issues 42 Pull Requests Projects Releases Wiki Activity

Orgs: PATCH op to batch-add emails as members now strip()s emails

Browse Source 
It also refuses to add empty emails.
This commit is contained in:
Sybren A. Stüvel 2017-08-23 15:37:03 +02:00
parent 5d17d892a4
commit cfde720b1d
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
pillar/api/organizations/patch.py
View File

@ -33,8 +33,10 @@ class OrganizationPatchHandler(patch_handler.AbstractPatchHandler):
except KeyError: except KeyError:
raise wz_exceptions.BadRequest('No key "email" in patch.') raise wz_exceptions.BadRequest('No key "email" in patch.')
if not all(isinstance(email, str) for email in emails): # Skip empty emails.
raise wz_exceptions.BadRequest('Invalid list of email addresses') 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', log.info('User %s uses PATCH to add users to organization %s',
current_user().user_id, org_id) current_user().user_id, org_id)