perf: avoid full URL parse and HashMap alloc in param()

Author: Algorithm5838

src/utils.rs

@@ -17,7 +17,7 @@ use rust_embed::RustEmbed;
 use serde::{Deserialize, Deserializer, Serialize, Serializer};
 use serde_json::Value;
 use serde_json_path::{JsonPath, JsonPathExt};
-use std::collections::{HashMap, HashSet};
+use std::collections::HashSet;
 use std::env;
 use std::io::{Read, Write};
 use std::string::ToString;
@@ -890,15 +890,10 @@ pub async fn parse_post(post: &Value) -> Post {
 
 /// Grab a query parameter from a url
 pub fn param(path: &str, value: &str) -> Option<String> {
-	Some(
-		Url::parse(format!("https://libredd.it/{path}").as_str())
-			.ok()?
-			.query_pairs()
-			.into_owned()
-			.collect::<HashMap<_, _>>()
-			.get(value)?
-			.clone(),
-	)
+	let query = path.splitn(2, '?').nth(1).unwrap_or_default();
+	url::form_urlencoded::parse(query.as_bytes())
+		.find(|(k, _)| k == value)
+		.map(|(_, v)| v.into_owned())
 }
 
 /// Retrieve the value of a setting by name