From ccfc243c2c1fec9b859ab6accec1246fa63aef94 Mon Sep 17 00:00:00 2001
From: Andrej Kacian <andrej@kacian.sk>
Date: Thu, 27 Jan 2022 17:00:08 +0100
Subject: [PATCH 1/2] Make get_remote_content() return Result instead of
 ConduitResult

---
 src/client_server/media.rs | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/client_server/media.rs b/src/client_server/media.rs
index bd73cff5..dd8e7b0f 100644
--- a/src/client_server/media.rs
+++ b/src/client_server/media.rs
@@ -79,7 +79,7 @@ pub async fn get_remote_content(
     mxc: &str,
     server_name: &ruma::ServerName,
     media_id: &str
-) -> ConduitResult<get_content::Response> {
+) -> Result<get_content::Response, Error> {
     let content_response = db
         .sending
         .send_federation_request(
@@ -103,7 +103,7 @@ pub async fn get_remote_content(
         )
         .await?;
 
-    Ok(content_response.into())
+    Ok(content_response)
 }
 
 /// # `GET /_matrix/media/r0/download/{serverName}/{mediaId}`
@@ -141,7 +141,7 @@ pub async fn get_content_route(
             &body.server_name,
             &body.media_id
             ).await?;
-        Ok(remote_content_response)
+        Ok(remote_content_response.into())
     } else {
         Err(Error::BadRequest(ErrorKind::NotFound, "Media not found."))
     }
@@ -185,8 +185,8 @@ pub async fn get_content_as_filename_route(
 
         Ok(get_content_as_filename::Response {
             content_disposition: Some(format!("inline: filename={}", body.filename)),
-            content_type: remote_content_response.0.content_type,
-            file: remote_content_response.0.file
+            content_type: remote_content_response.content_type,
+            file: remote_content_response.file
         }
         .into())
     } else {

From 0f6d232cb1117d959a1984551ee2872558172767 Mon Sep 17 00:00:00 2001
From: Andrej Kacian <andrej@kacian.sk>
Date: Thu, 27 Jan 2022 17:08:04 +0100
Subject: [PATCH 2/2] Style fixes from 'cargo fmt'

---
 src/client_server/media.rs | 22 +++++++---------------
 1 file changed, 7 insertions(+), 15 deletions(-)

diff --git a/src/client_server/media.rs b/src/client_server/media.rs
index dd8e7b0f..a827d64f 100644
--- a/src/client_server/media.rs
+++ b/src/client_server/media.rs
@@ -78,7 +78,7 @@ pub async fn get_remote_content(
     db: &DatabaseGuard,
     mxc: &str,
     server_name: &ruma::ServerName,
-    media_id: &str
+    media_id: &str,
 ) -> Result<get_content::Response, Error> {
     let content_response = db
         .sending
@@ -88,7 +88,7 @@ pub async fn get_remote_content(
             get_content::Request {
                 allow_remote: false,
                 server_name,
-                media_id
+                media_id,
             },
         )
         .await?;
@@ -135,12 +135,8 @@ pub async fn get_content_route(
         }
         .into())
     } else if &*body.server_name != db.globals.server_name() && body.allow_remote {
-        let remote_content_response = get_remote_content(
-            &db,
-            &mxc,
-            &body.server_name,
-            &body.media_id
-            ).await?;
+        let remote_content_response =
+            get_remote_content(&db, &mxc, &body.server_name, &body.media_id).await?;
         Ok(remote_content_response.into())
     } else {
         Err(Error::BadRequest(ErrorKind::NotFound, "Media not found."))
@@ -176,17 +172,13 @@ pub async fn get_content_as_filename_route(
         }
         .into())
     } else if &*body.server_name != db.globals.server_name() && body.allow_remote {
-        let remote_content_response = get_remote_content(
-            &db,
-            &mxc,
-            &body.server_name,
-            &body.media_id
-            ).await?;
+        let remote_content_response =
+            get_remote_content(&db, &mxc, &body.server_name, &body.media_id).await?;
 
         Ok(get_content_as_filename::Response {
             content_disposition: Some(format!("inline: filename={}", body.filename)),
             content_type: remote_content_response.content_type,
-            file: remote_content_response.file
+            file: remote_content_response.file,
         }
         .into())
     } else {