diff --git a/src/endpoint.rs b/src/endpoint.rs index 348513e..48bec16 100644 --- a/src/endpoint.rs +++ b/src/endpoint.rs @@ -120,7 +120,7 @@ impl<'a> Endpoint<'a> { let public_key = PublicKey::from_pem(remote_actor.public_key.pem.as_bytes())?; if ! (self.signature.verify(&public_key)?) { - return Err(Error::SignatureFail); + return Err(Error::SignatureFail(self.remote_actor_uri.clone())); } Ok(remote_actor) diff --git a/src/error.rs b/src/error.rs index 639d83d..9fdcbcd 100644 --- a/src/error.rs +++ b/src/error.rs @@ -9,7 +9,7 @@ pub enum Error { #[error("Signature error")] Signature(#[from] Arc), #[error("Signature verification failure")] - SignatureFail, + SignatureFail(String), #[error("HTTP request error")] HttpReq(#[from] Arc), #[error("HTTP client error")] diff --git a/src/main.rs b/src/main.rs index 0f1b3a6..6174cec 100644 --- a/src/main.rs +++ b/src/main.rs @@ -138,6 +138,7 @@ async fn post_relay( Ok(remote_actor) => remote_actor, Err(e) => { track_request("POST", "relay", "bad_actor"); + tracing::error!("post_relay bad actor: {e:?}"); return ( StatusCode::BAD_REQUEST, format!("Bad actor: {:?}", e) @@ -160,6 +161,7 @@ async fn post_relay( Ok(action) => action, Err(e) => { track_request("POST", "relay", "bad_action"); + tracing::error!("post_relay bad action: {e:?}"); return ( StatusCode::BAD_REQUEST, format!("Bad action: {:?}", e) diff --git a/src/send.rs b/src/send.rs index 2e2aeb4..7bb0802 100644 --- a/src/send.rs +++ b/src/send.rs @@ -55,7 +55,6 @@ pub async fn send_raw( Ok(()) } else { histogram!("relay_http_response_duration", t3 - t2, "res" => "err", "host" => host); - tracing::error!("send_raw {} response HTTP {}", url, res.status()); let response = res.text().await?; Err(Error::Response(response)) }