From b423e4d5243c4b2321f774d897e6cc372e06adde Mon Sep 17 00:00:00 2001 From: Brandon Thomas Date: Wed, 25 Mar 2026 14:51:54 -0400 Subject: [PATCH] error --- crates/lib/pager/src/client/pager_builder.rs | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/crates/lib/pager/src/client/pager_builder.rs b/crates/lib/pager/src/client/pager_builder.rs index fac4829058..49cc20eb3a 100644 --- a/crates/lib/pager/src/client/pager_builder.rs +++ b/crates/lib/pager/src/client/pager_builder.rs @@ -5,6 +5,8 @@ use rootly_client::creds::rootly_api_key::RootlyApiKey; use crate::client::pager::Pager; use crate::client::pager_client::{PagerClient, PagerClientConfig}; +use crate::error::pager_client_error::PagerClientError; +use crate::error::pager_error::PagerError; use crate::worker::pager_worker_message_queue::{new_shared_queue, new_shared_queue_with_capacity, SharedMessageQueue}; use crate::worker::pager_worker_thread::PagerWorkerThread; @@ -74,11 +76,9 @@ impl PagerBuilder { /// /// If `with_worker()` was called, this also creates a `PagerWorkerThread`. /// The caller is responsible for spawning the worker thread (via `Pager::worker()`). - pub fn build(self) -> Pager { - let api_key = self.api_key.unwrap_or_else(|| { - log::warn!("PagerBuilder: No API key set. Pages will fail at send time."); - RootlyApiKey::new(String::new()) - }); + pub fn build(self) -> Result { + let api_key = self.api_key + .ok_or(PagerClientError::NotConfigured("no api key set".to_string()))?; let source = self.source.unwrap_or_else(|| "unknown".to_string()); @@ -106,6 +106,6 @@ impl PagerBuilder { (None, None, None) }; - Pager::new(client, queue, worker, worker_shutdown) + Ok(Pager::new(client, queue, worker, worker_shutdown)) } }