From 25f2f40a096c7768757db0262d39e048c79ae24b Mon Sep 17 00:00:00 2001 From: Sergey Timoshin Date: Mon, 21 Jul 2025 15:24:54 +0100 Subject: [PATCH] fix: don't panic if PagerDuty routing key is undefined --- forester/src/epoch_manager.rs | 31 +++++++++++++++---------------- 1 file changed, 15 insertions(+), 16 deletions(-) diff --git a/forester/src/epoch_manager.rs b/forester/src/epoch_manager.rs index 90cb9a397f..7ec18ee435 100644 --- a/forester/src/epoch_manager.rs +++ b/forester/src/epoch_manager.rs @@ -545,23 +545,22 @@ impl EpochManager { if attempt < max_retries - 1 { sleep(retry_delay).await; } else { - if let Err(alert_err) = send_pagerduty_alert( - &self - .config - .external_services - .pagerduty_routing_key - .clone() - .unwrap(), - &format!( - "Forester failed to register for epoch {} after {} attempts", - epoch, max_retries - ), - "critical", - &format!("Forester {}", self.config.payer_keypair.pubkey()), - ) - .await + if let Some(pagerduty_key) = + self.config.external_services.pagerduty_routing_key.clone() { - error!("Failed to send PagerDuty alert: {:?}", alert_err); + if let Err(alert_err) = send_pagerduty_alert( + &pagerduty_key, + &format!( + "Forester failed to register for epoch {} after {} attempts", + epoch, max_retries + ), + "critical", + &format!("Forester {}", self.config.payer_keypair.pubkey()), + ) + .await + { + error!("Failed to send PagerDuty alert: {:?}", alert_err); + } } return Err(e); }