Merge pull request #2231 from karthik2804/improve_redis_trigger_warn

Warn instead of trace when errors happen in the spin_redis_engine
spinframework · Jan 18, 2024 · c25b685 · c25b685
2 parents be88d81 + 061cdeb
commit c25b685
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/crates/redis/src/lib.rs b/crates/redis/src/lib.rs
@@ -94,7 +94,11 @@ impl TriggerExecutor for RedisTrigger {
         let mut stream = pubsub.on_message();
         loop {
             match stream.next().await {
-                Some(msg) => drop(self.handle(msg).await),
+                Some(msg) => {
+                    if let Err(err) = self.handle(msg).await {
+                        tracing::warn!("Error handling message: {err}");
+                    }
+                }
                 None => {
                     tracing::trace!("Empty message");
                     if !client.check_connection() {