From 42fe8c30838862193e63bae6643c6c82e62704b7 Mon Sep 17 00:00:00 2001 From: Affaan Mustafa Date: Thu, 11 Jun 2026 01:26:59 -0400 Subject: [PATCH] fix(ecc2): port webhook sender to ureq 3 Agent API (#2231) #2209 bumped ureq to 3.x but the AgentBuilder-based webhook sender was not ported (branch update raced the merge). ureq 3 replaces AgentBuilder with Agent::config_builder(); timeouts are Option-wrapped and status() returns http::StatusCode. --- ecc2/src/notifications.rs | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/ecc2/src/notifications.rs b/ecc2/src/notifications.rs index f7d51883..7e98e9a5 100644 --- a/ecc2/src/notifications.rs +++ b/ecc2/src/notifications.rs @@ -403,16 +403,17 @@ fn run_notification_command(_program: &str, _args: &[String]) -> Result<()> { #[cfg(not(test))] fn send_webhook_request(target: &WebhookTarget, payload: serde_json::Value) -> Result<()> { - let agent = ureq::AgentBuilder::new() - .timeout_connect(std::time::Duration::from_secs(5)) - .timeout_read(std::time::Duration::from_secs(5)) - .build(); + let agent = ureq::Agent::config_builder() + .timeout_connect(Some(std::time::Duration::from_secs(5))) + .timeout_recv_response(Some(std::time::Duration::from_secs(5))) + .build() + .new_agent(); let response = agent .post(&target.url) .send_json(payload) .with_context(|| format!("POST {}", target.url))?; - if response.status() >= 200 && response.status() < 300 { + if response.status().is_success() { Ok(()) } else { anyhow::bail!("{} returned {}", target.url, response.status());