fix: Invalid type in config cause parse failure

deepflowio · Nov 1, 2024 · 52ba9b8 · 52ba9b8
1 parent dbe93b7
commit 52ba9b8
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/agent/src/config/config.rs b/agent/src/config/config.rs
@@ -2177,8 +2177,8 @@ impl Default for Socket {
 #[derive(Clone, Debug, Deserialize, PartialEq, Eq)]
 #[serde(default)]
 pub struct FlowLogFilters {
-    pub l4_capture_network_types: Vec<u16>,
-    pub l7_capture_network_types: Vec<u16>,
+    pub l4_capture_network_types: Vec<i16>,
+    pub l7_capture_network_types: Vec<i16>,
     pub l4_ignored_observation_points: Vec<u16>,
     pub l7_ignored_observation_points: Vec<u16>,
 }
@@ -2886,12 +2886,12 @@ impl From<&RuntimeConfig> for UserConfig {
                         l4_capture_network_types: rc
                             .l4_log_store_tap_types
                             .iter()
-                            .map(|t| *t as u16)
+                            .map(|t| *t as i16)
                             .collect(),
                         l7_capture_network_types: rc
                             .l7_log_store_tap_types
                             .iter()
-                            .map(|t| *t as u16)
+                            .map(|t| *t as i16)
                             .collect(),
                         l4_ignored_observation_points: rc
                             .l4_log_ignore_tap_sides

diff --git a/agent/src/config/handler.rs b/agent/src/config/handler.rs
@@ -2007,7 +2007,7 @@ impl TryFrom<(Config, UserConfig, DynamicConfig)> for ModuleConfig {
                         .l7_capture_network_types
                         .iter()
                     {
-                        if t >= u16::from(CaptureNetworkType::Max) {
+                        if t >= u16::from(CaptureNetworkType::Max) as i16 {
                             warn!("invalid tap type: {}", t);
                         } else {
                             tap_types[t as usize] = true;