Fix CI

Author: martinsaposnic

lightning-liquidity/src/lsps5/client.rs

@@ -28,7 +28,9 @@ use core::ops::Deref;
 use crate::prelude::{new_hash_map, HashMap, String};
 
 use super::msgs::{Lsps5AppName, Lsps5WebhookUrl};
-use super::service::{from_rfc3339, DefaultTimeProvider, TimeProvider};
+#[cfg(feature = "std")]
+use super::service::DefaultTimeProvider;
+use super::service::{from_rfc3339, TimeProvider};
 use super::url_utils::Url;
 use core::time::Duration;
 use lightning::sign::EntropySource;
@@ -43,22 +45,25 @@ pub struct LSPS5ClientConfig {
 	/// Maximum age in seconds for cached responses (default: 3600 - 1 hour)
 	pub response_max_age_secs: u64,
 	/// Time provider for LSPS5 service
-	pub time_provider: Arc<dyn TimeProvider>,
+	pub time_provider: Option<Arc<dyn TimeProvider>>,
 }
 
 impl Default for LSPS5ClientConfig {
 	fn default() -> Self {
 		Self {
 			response_max_age_secs: DEFAULT_RESPONSE_MAX_AGE_SECS,
-			time_provider: Arc::new(DefaultTimeProvider),
+			#[cfg(feature = "std")]
+			time_provider: Some(Arc::new(DefaultTimeProvider)),
+			#[cfg(not(feature = "std"))]
+			time_provider: None,
 		}
 	}
 }
 
 impl LSPS5ClientConfig {
 	/// Set a custom time provider for the LSPS5 service
 	pub fn with_time_provider(mut self, time_provider: impl TimeProvider + 'static) -> Self {
-		self.time_provider = Arc::new(time_provider);
+		self.time_provider = Some(Arc::new(time_provider));
 		self
 	}
 }
@@ -136,9 +141,9 @@ where
 	/// event queue, and LSPS5ClientConfig
 	pub(crate) fn new(
 		entropy_source: ES, pending_messages: Arc<MessageQueue>, pending_events: Arc<EventQueue>,
-		config: LSPS5ClientConfig,
+		mut config: LSPS5ClientConfig,
 	) -> Self {
-		let time_provider = config.time_provider.clone();
+		let time_provider = config.time_provider.take().expect("Time provider should be present");
 		Self {
 			pending_messages,
 			pending_events,
@@ -525,12 +530,12 @@ where
 			Ok(timestamp_dt) => {
 				// Check timestamp is within 10 minutes of current time
 				let now = time_provider.now();
-				let diff;
-				if now > timestamp_dt {
-					diff = now.checked_sub(timestamp_dt).unwrap();
+
+				let diff = if now > timestamp_dt {
+					now.checked_sub(timestamp_dt).unwrap()
 				} else {
-					diff = timestamp_dt.checked_sub(now).unwrap();
-				}
+					timestamp_dt.checked_sub(now).unwrap()
+				};
 				if diff > Duration::from_secs(600) {
 					// 10 minutes
 					return Err(LightningError {

lightning-liquidity/src/lsps5/service.rs

@@ -192,7 +192,7 @@ pub struct LSPS5ServiceConfig {
 	/// Configuration for signature storage
 	pub signature_config: SignatureStorageConfig,
 	/// Time provider for LSPS5 service
-	pub time_provider: Arc<dyn TimeProvider>,
+	pub time_provider: Option<Arc<dyn TimeProvider>>,
 	/// HTTP client for webhook delivery
 	#[doc(hidden)] // Hide from docs since it uses Arc and can't be directly created
 	pub(crate) http_client: Option<Arc<dyn HttpClient>>,
@@ -206,7 +206,10 @@ impl Default for LSPS5ServiceConfig {
 			notification_cooldown_hours: 24,
 			signature_config: SignatureStorageConfig::default(),
 			http_client: None,
-			time_provider: Arc::new(DefaultTimeProvider),
+			#[cfg(feature = "std")]
+			time_provider: Some(Arc::new(DefaultTimeProvider)),
+			#[cfg(not(feature = "std"))]
+			time_provider: None,
 		}
 	}
 }
@@ -220,7 +223,7 @@ impl LSPS5ServiceConfig {
 
 	/// Set a custom time provider for the LSPS5 service
 	pub fn with_time_provider(mut self, time_provider: impl TimeProvider + 'static) -> Self {
-		self.time_provider = Arc::new(time_provider);
+		self.time_provider = Some(Arc::new(time_provider));
 		self
 	}
 }
@@ -268,7 +271,7 @@ impl LSPS5ServiceHandler {
 		let http_client: Arc<dyn HttpClient> =
 			config.http_client.take().expect("HTTP client should be present");
 		let max_signatures = config.signature_config.max_signatures.clone();
-		let time_provider = config.time_provider.clone();
+		let time_provider = config.time_provider.take().expect("Time provider should be present");
 		Self {
 			config,
 			webhooks: Arc::new(Mutex::new(new_hash_map())),