fix: Adding app lookup by id trait

Author: ruslanti

Cargo.toml

@@ -3,7 +3,7 @@ members = ["crates/*"]
 resolver = "2"
 
 [workspace.package]
-version = "0.3.8"
+version = "0.3.9"
 edition = "2021"
 publish = false
 authors = ["FastEdge Development Team"]

crates/http-service/src/lib.rs

@@ -234,7 +234,7 @@ where
             app_name,
             request.uri()
         );
-        let cfg = match self.context.lookup(&app_name).await {
+        let cfg = match self.context.lookup_by_name(&app_name).await {
             None => {
                 #[cfg(feature = "metrics")]
                 metrics::metrics(AppResult::UNKNOWN);
@@ -612,7 +612,7 @@ mod tests {
     }
 
     impl Router for TestContext {
-        async fn lookup(&self, _name: &str) -> Option<App> {
+        async fn lookup_by_name(&self, _name: &str) -> Option<App> {
             self.app.clone()
         }
     }

crates/runtime/src/lib.rs

@@ -314,7 +314,8 @@ pub trait ExecutorCache {
 }
 
 pub trait Router: Send + Sync {
-    fn lookup(&self, name: &str) -> impl std::future::Future<Output = Option<App>> + Send;
+    fn lookup_by_name(&self, name: &str) -> impl std::future::Future<Output = Option<App>> + Send;
+    fn lookup_by_id(&self, id: u64) -> impl std::future::Future<Output = Option<App>> + Send;
 }
 
 pub fn componentize_if_necessary(buffer: &[u8]) -> anyhow::Result<Cow<[u8]>> {

src/main.rs

@@ -246,9 +246,13 @@ impl ContextHeaders for CliContext<'_> {
 }
 
 impl Router for CliContext<'_> {
-    async fn lookup(&self, _name: &str) -> Option<App> {
+    async fn lookup_by_name(&self, _name: &str) -> Option<App> {
         self.app.to_owned()
     }
+
+    async fn lookup_by_id(&self, _id: u64) -> Option<App> {
+        unreachable!()
+    }
 }
 
 impl StatsWriter for CliContext<'_> {