Don't panic on SerializationFailure | InternalFailure while getting p…

…roperty Signed-off-by: Alex Snaps <alex@wcgw.dev>
proxy-wasm · Jul 3, 2024 · 51b0584 · 51b0584
1 parent 427bea0
commit 51b0584
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/src/hostcalls.rs b/src/hostcalls.rs
@@ -414,8 +414,10 @@ pub fn get_property(path: Vec<&str>) -> Result<Option<Bytes>, Status> {
                 } else {
                     Ok(None)
                 }
-            }
+            },
             Status::NotFound => Ok(None),
+            Status::SerializationFailure => Err(Status::SerializationFailure),
+            Status::InternalFailure => Err(Status::InternalFailure),
             status => panic!("unexpected status: {}", status as u32),
         }
     }

diff --git a/src/types.rs b/src/types.rs
@@ -46,14 +46,9 @@ pub enum Status {
     BadArgument = 2,
     SerializationFailure = 3,
     ParseFailure = 4,
-    BadExpression = 5,
-    InvalidMemoryAccess = 6,
     Empty = 7,
     CasMismatch = 8,
-    ResultMismatch = 9,
     InternalFailure = 10,
-    BrokenConnection = 11,
-    Unimplemented = 12,
 }
 
 #[repr(u32)]