From 4db6d7e4308b206e0f34e291f1d23e7a54ea254a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timo=20K=C3=B6sters?= <timo@koesters.xyz>
Date: Wed, 16 Sep 2020 18:10:17 +0200
Subject: [PATCH] fix: remove avatar url checks

They are not in the spec and maubot relies on that
---
 src/client_server/profile.rs | 12 ------------
 1 file changed, 12 deletions(-)

diff --git a/src/client_server/profile.rs b/src/client_server/profile.rs
index 53893c0f..686d4c3e 100644
--- a/src/client_server/profile.rs
+++ b/src/client_server/profile.rs
@@ -119,18 +119,6 @@ pub async fn set_avatar_url_route(
 ) -> ConduitResult<set_avatar_url::Response> {
     let sender_id = body.sender_id.as_ref().expect("user is authenticated");
 
-    if let Some(avatar_url) = &body.avatar_url {
-        if !avatar_url.starts_with("mxc://") {
-            return Err(Error::BadRequest(
-                ErrorKind::InvalidParam,
-                "avatar_url has to start with mxc://.",
-            ));
-        }
-
-        // TODO in the future when we can handle media uploads make sure that this url is our own server
-        // TODO also make sure this is valid mxc:// format (not only starting with it)
-    }
-
     db.users
         .set_avatar_url(&sender_id, body.avatar_url.clone())?;