@@ -349,0 +351,4 @@
+			..Default::default() /* TODO(nex): This weird manual field de/serialisation just
+			                      * needs replacing with JSON storage to avoid this mess.
+			                      * This is consequently intentionally dropping information
+			                      * added later (like image alt text). I'll do it later. */

@@ -301,0 +336,4 @@
+				preview_data.image_height = image_data.image_height;
+				preview_data.image_type = obj
+					.properties
+					.get("image:type")

@@ -301,0 +344,4 @@
+					.flatten();
+				preview_data.image_alt = obj
+					.properties
+					.get("image:alt")

@@ -307,0 +360,4 @@
+				preview_data.video_size = video_data.video_size;
+				preview_data.video_type = obj
+					.properties
+					.get("video:type")

@@ -307,0 +368,4 @@
+					.flatten();
+				preview_data.video_alt = obj
+					.properties
+					.get("video:alt")

@@ -307,0 +355,4 @@
+				|e| info!(error=?e, asset_url=%obj.url, "Could not download URL preview video"),
+			) {
+				preview_data.video = video_data.video;
+				preview_data.video_width = video_data.video_width;

@@ -219,6 +219,7 @@ pub(crate) async fn get_media_preview_route(
 		)));
 	}
 
+	info!("Fetching a URL preview for {url} on behalf of {sender_user}");

@@ -307,0 +356,4 @@
+			) {
+				preview_data.video = video_data.video;
+				preview_data.video_width = video_data.video_width;
+				preview_data.video_height = video_data.video_height;

@@ -319,0 +390,4 @@
+		preview_data.determiner = props.get("determiner").cloned();
+		preview_data.locale = props.get("locale").cloned();
+		preview_data.site_name = props.get("site_name").cloned();
+		preview_data.site_type = props.get("type").cloned();

@@ -319,0 +391,4 @@
+		preview_data.locale = props.get("locale").cloned();
+		preview_data.site_name = props.get("site_name").cloned();
+		preview_data.site_type = props.get("type").cloned();
+		preview_data.url = props.get("url").cloned().or_else(|| Some(url.to_owned()));

@@ -84,3 +84,3 @@
 				.timeout(Duration::from_secs(config.url_preview_timeout))
 				.redirect(redirect::Policy::limited(3))
-				.user_agent(url_preview_user_agent)
+				.user_agent(url_preview_user_agent.replace("<version>", conduwuit::version_ua()))

@@ -24,1 +25,4 @@
 pub struct UrlPreviewData {
+	// From https://ogp.me/
+	/// The title of your object as it should appear within the graph
+	/// "The Rock".

@@ -10,2 +10,3 @@
-use conduwuit::{Err, Result, debug, err, utils::response::LimitReadExt};
+use conduwuit::{Err, Result, debug, err, utils::response::LimitReadExt, info};
 use ipaddress::IPAddress;
+use lettre::message::header::ContentType;

@@ -219,10 +219,15 @@ pub(crate) async fn get_media_preview_route(
 		)));
 	}
 
+	info!("Fetching a URL preview for {url} on behalf of {sender_user}");

@@ -115,3 +154,1 @@
-			| audio if audio.starts_with("audio/") =>
-				self.download_audio(url.as_str(), None).await?,
-			| _ => return Err!(Request(Unknown("Unsupported Content-Type"))),
+			| img if img.starts_with("image/") => self.download_image(url.as_str()).await?,

@@ -300,3 +345,1 @@
-			preview_data = self.download_video(&obj.url, Some(preview_data)).await?;
-			preview_data.video_width = obj.properties.get("width").and_then(|v| v.parse().ok());
-			preview_data.video_height = obj.properties.get("height").and_then(|v| v.parse().ok());
+			info!(asset_url=%obj.url, "Downloading preview video");

@@ -315,0 +381,4 @@
+			preview_data.video_type = None;
+			preview_data.video_alt = None;
+			preview_data.video_height = None;
+			preview_data.video_width = None;