Merge pull request #126 from Arjentix/master

fix(core): do not ignore error while sending starting message
elfo-rs · May 11, 2024 · f61dbf7 · f61dbf7
2 parents d7e6390 + 8abf4c0
commit f61dbf7
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/elfo-core/src/init.rs b/elfo-core/src/init.rs
@@ -53,8 +53,7 @@ async fn start_entrypoints(ctx: &Context, topology: &Topology, is_check_only: bo
             match response {
                 Ok(Ok(())) => Ok(()),
                 Ok(Err(e)) => Err(StartError::single(group.name.clone(), e.reason)),
-                Err(RequestError::Ignored) => Ok(()),
-                Err(RequestError::Failed) => Err(StartError::single(
+                Err(RequestError::Ignored) | Err(RequestError::Failed) => Err(StartError::single(
                     group.name.clone(),
                     "config cannot be delivered to the entrypoint".into(),
                 )),
@@ -77,8 +76,7 @@ async fn start_entrypoints(ctx: &Context, topology: &Topology, is_check_only: bo
                         .collect();
                     Err(StartError::multiple(group_errors))
                 }
-                Err(RequestError::Ignored) => Ok(()),
-                Err(RequestError::Failed) => Err(StartError::single(
+                Err(RequestError::Ignored) | Err(RequestError::Failed) => Err(StartError::single(
                     group.name,
                     "starting message cannot be delivered to the entrypoint".into(),
                 )),