Fix wrong error code when subscribing to a non-existing path

eclipse · Jun 28, 2023 · 204596b · 204596b
1 parent 9ab495b
commit 204596b
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/kuksa_databroker/databroker/src/grpc/kuksa_val_v1/val.rs b/kuksa_databroker/databroker/src/grpc/kuksa_val_v1/val.rs
@@ -24,6 +24,7 @@ use tracing::debug;
 
 use crate::broker;
 use crate::broker::ReadError;
+use crate::broker::SubscriptionError;
 use crate::permissions::Permissions;
 
 #[tonic::async_trait]
@@ -326,10 +327,16 @@ impl proto::val_server::Val for broker::DataBroker {
                 let stream = convert_to_proto_stream(stream);
                 Ok(tonic::Response::new(Box::pin(stream)))
             }
-            Err(e) => Err(tonic::Status::new(
+            Err(SubscriptionError::NotFound) => {
+                Err(tonic::Status::new(tonic::Code::NotFound, "Path not found"))
+            }
+            Err(SubscriptionError::InvalidInput) => Err(tonic::Status::new(
                 tonic::Code::InvalidArgument,
-                format!("{e:?}"),
+                "Invalid Argument",
             )),
+            Err(SubscriptionError::InternalError) => {
+                Err(tonic::Status::new(tonic::Code::Internal, "Internal Error"))
+            }
         }
     }