fix(client): use string literals for wreq header lookups + convert Method via str

Author: Algorithm5838

src/client.rs

@@ -147,14 +147,15 @@ pub async fn canonical_path(path: String, tries: i8) -> Result<Option<String>, S
 
 	let res = res.ok_or_else(|| "Unable to make HEAD request to Reddit.".to_string())?;
 	let status = res.status().as_u16();
-	let policy_error = res.headers().get(header::RETRY_AFTER).is_some();
+	// Use string literals: wreq uses http v1.x, hyper::header constants are http v0.2
+	let policy_error = res.headers().get("retry-after").is_some();
 
 	match status {
 		// If Reddit responds with a 2xx, then the path is already canonical.
 		200..=299 => Ok(Some(path)),
 
 		// If Reddit responds with a 301, then the path is redirected.
-		301 => match res.headers().get(header::LOCATION) {
+		301 => match res.headers().get("location") {
 			Some(val) => {
 				let Ok(original) = val.to_str() else {
 					return Err("Unable to decode Location header.".to_string());
@@ -179,7 +180,7 @@ pub async fn canonical_path(path: String, tries: i8) -> Result<Option<String>, S
 		_ => Ok(
 			res
 				.headers()
-				.get(header::LOCATION)
+				.get("location")
 				.map(|val| percent_encode(val.as_bytes(), CONTROLS).to_string().trim_start_matches(REDDIT_URL_BASE).to_string()),
 		),
 	}
@@ -283,7 +284,9 @@ fn request(method: Method, path: String, redirect: bool, quarantine: bool, base_
 	fastrand::shuffle(&mut headers);
 
 	async move {
-		let mut builder = CLIENT.request(method.clone(), &url);
+		// Convert hyper::Method (http v0.2) to wreq::Method (http v1.x) via string
+		let wreq_method = wreq::Method::from_bytes(method.as_str().as_bytes()).unwrap_or(wreq::Method::GET);
+		let mut builder = CLIENT.request(wreq_method, &url);
 
 		for (key, value) in &headers {
 			builder = builder.header(key.as_str(), value.as_str());
@@ -296,7 +299,8 @@ fn request(method: Method, path: String, redirect: bool, quarantine: bool, base_
 						return resp.into_hyper().await;
 					}
 
-					let location = resp.headers().get(header::LOCATION).map(|v| v.to_str().unwrap_or_default().to_string());
+					// Use string literal: wreq uses http v1.x, hyper::header constants are http v0.2
+					let location = resp.headers().get("location").map(|v| v.to_str().unwrap_or_default().to_string());
 
 					if location.as_deref() == Some(ALTERNATIVE_REDDIT_URL_BASE) {
 						return Err("Reddit response was invalid".to_string());