diff options
| author | Andrew Morgan <1342360+anoadragon453@users.noreply.github.com> | 2019-12-09 14:54:33 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-12-09 14:54:33 +0000 |
| commit | 5e8abe9013427e8ad452c4652dfcb40da05c246e (patch) | |
| tree | a354359636e037ebfc57d666c1819beea023b74b /synapse/rest/client/v1 | |
| parent | Back out perf regression from get_cross_signing_keys_from_cache. (#6494) (diff) | |
| download | synapse-5e8abe9013427e8ad452c4652dfcb40da05c246e.tar.xz | |
Better errors regarding changing avatar_url (#6497)
Diffstat (limited to 'synapse/rest/client/v1')
| -rw-r--r-- | synapse/rest/client/v1/profile.py | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/synapse/rest/client/v1/profile.py b/synapse/rest/client/v1/profile.py index 1eac8a44c5..4f47562c1b 100644 --- a/synapse/rest/client/v1/profile.py +++ b/synapse/rest/client/v1/profile.py @@ -103,11 +103,16 @@ class ProfileAvatarURLRestServlet(RestServlet): content = parse_json_object_from_request(request) try: - new_name = content["avatar_url"] + new_avatar_url = content.get("avatar_url") except Exception: - return 400, "Unable to parse name" + return 400, "Unable to parse avatar_url" + + if new_avatar_url is None: + return 400, "Missing required key: avatar_url" - await self.profile_handler.set_avatar_url(user, requester, new_name, is_admin) + await self.profile_handler.set_avatar_url( + user, requester, new_avatar_url, is_admin + ) return 200, {} |