Skip to content

Commit b664c80

Browse files
committed
add error handling to create_user function
1 parent 1017431 commit b664c80

File tree

1 file changed

+10
-3
lines changed

1 file changed

+10
-3
lines changed

dspace_rest_client/client.py

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1335,9 +1335,16 @@ def create_user(self, user, token=None, embeds=None):
13351335
params = parse_params(embeds=embeds)
13361336
if token is not None:
13371337
params = {"token": token}
1338-
return User(
1339-
api_resource=parse_json(self.create_dso(url, params=params, data=data))
1340-
)
1338+
1339+
api_resource = self.create_dso(url, params=params, data=data)
1340+
if api_resource.status_code == 201 and api_resource.text:
1341+
return User(api_resource=parse_json(api_resource))
1342+
elif api_resource.status_code in [400, 401, 403, 422]:
1343+
logging.error("User creation failed: %s", api_resource.text)
1344+
return None
1345+
else:
1346+
logging.error("User creation failed: %s", api_resource.text)
1347+
return None
13411348

13421349
def delete_user(self, user):
13431350
if not isinstance(user, User):

0 commit comments

Comments
 (0)