fix: don't http sign proxy cloaks

upub/core/src/traits/fetch.rs

@@ -99,6 +99,13 @@ pub trait Fetcher {
 
 	async fn fetch_thread(&self, id: &str, tx: &impl ConnectionTrait) -> Result<(), RequestError>;
 
+	fn client(domain: &str) -> reqwest::Client {
+		reqwest::Client::builder()
+			.user_agent(format!("upub+{} ({domain})", crate::VERSION))
+			.build()
+			.expect("failed building http client, check system tls or resolver")
+	}
+
 	async fn request(
 		method: reqwest::Method,
 		url: &str,
@@ -130,11 +137,10 @@ pub trait Fetcher {
 			.build_manually(&method.to_string().to_lowercase(), &path, headers_map)
 			.sign(key)?;
 
-		let response = reqwest::Client::new()
+		let response = Self::client(domain)
-			.request(method.clone(), url)
+			.request(method, url)
 			.header(ACCEPT, "application/ld+json; profile=\"https://www.w3.org/ns/activitystreams\"")
 			.header(CONTENT_TYPE, "application/ld+json; profile=\"https://www.w3.org/ns/activitystreams\"")
-			.header(USER_AGENT, format!("upub+{} ({domain})", crate::VERSION))
 			.header("Host", host.clone())
 			.header("Date", date.clone())
 			.header("Digest", digest)

upub/routes/src/activitypub/application.rs

@@ -3,7 +3,7 @@ use axum::{extract::{Path, Query, State}, http::HeaderMap, response::{IntoRespon
 use reqwest::Method;
 use upub::{traits::{Cloaker, Fetcher}, Context};
 
-use crate::{builders::JsonLD, ApiError, ApiResult, AuthIdentity, Identity};
+use crate::{builders::JsonLD, ApiError, AuthIdentity};
 
 
 pub async fn view(
@@ -76,20 +76,18 @@ pub async fn cloak_proxy(
 	let uri = ctx.uncloak(&hmac, &uri)
 		.ok_or_else(ApiError::unauthorized)?;
 
-	let resp = Context::request(
+	let resp = Context::client(ctx.domain())
-			Method::GET,
+		.get(uri)
-			&uri,
+		.send()
-			None,
+		.await?
-			ctx.base(),
+		.error_for_status()?;
-			ctx.pkey(),
-			&format!("{}+proxy", ctx.domain()),
-		)
-			.await?
-			.error_for_status()?;
 
 	let headers = resp.headers().clone();
+	// TODO can we stream the response body as it comes?
 	let body = resp.bytes().await?.to_vec();
 
+	// TODO not so great to just try parsing json, but this should be a cheap check as most things we
+	// proxy are not json (as in, dont start with '{')
 	if serde_json::from_slice::<serde_json::Value>(&body).is_ok() {
 		return Err(ApiError::forbidden());
 	}