char/conduit
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

fix: remove avatar url checks

Browse source 
They are not in the spec and maubot relies on that
This commit is contained in:
Timo Kösters 2020-09-16 18:10:17 +02:00
parent a567cd81d5
commit 4db6d7e430
No known key found for this signature in database
GPG key ID: 24DA7517711A2BA4
1 changed files with 0 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
src/client_server/profile.rs
View file

@ -119,18 +119,6 @@ pub async fn set_avatar_url_route(
) -> ConduitResult<set_avatar_url::Response> { ) -> ConduitResult<set_avatar_url::Response> {
let sender_id = body.sender_id.as_ref().expect("user is authenticated"); 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 db.users
.set_avatar_url(&sender_id, body.avatar_url.clone())?; .set_avatar_url(&sender_id, body.avatar_url.clone())?;