fixup! Update JS filter config using MQTT

didier-wenzek · didier-wenzek · commit 975b8be421a7 · 2025-06-03T11:01:40.000+02:00
diff --git a/crates/extensions/tedge_gen_mapper/src/js_filter.rs b/crates/extensions/tedge_gen_mapper/src/js_filter.rs
@@ -307,11 +307,9 @@ impl<'js> FromJs<'js> for JsonValue {
         }
         if let Some(object) = value.as_object() {
             let mut js_object = serde_json::Map::new();
-            for key in object.keys::<String>() {
-                if let Ok(k) = key {
-                    if let Ok(JsonValue(v)) = object.get(&k) {
-                        js_object.insert(k, v.clone());
-                    }
+            for key in object.keys::<String>().flatten() {
+                if let Ok(JsonValue(v)) = object.get(&key) {
+                    js_object.insert(key, v.clone());
                 }
             }
             return Ok(JsonValue(serde_json::Value::Object(js_object)));

Original file line number	Diff line number	Diff line change
`@@ -307,11 +307,9 @@ impl<'js> FromJs<'js> for JsonValue {`
`307`	`307`	`}`
`308`	`308`	`if let Some(object) = value.as_object() {`
`309`	`309`	`let mut js_object = serde_json::Map::new();`
`310`		`- for key in object.keys::<String>() {`
`311`		`- if let Ok(k) = key {`
`312`		`- if let Ok(JsonValue(v)) = object.get(&k) {`
`313`		`- js_object.insert(k, v.clone());`
`314`		`- }`
	`310`	`+ for key in object.keys::<String>().flatten() {`
	`311`	`+ if let Ok(JsonValue(v)) = object.get(&key) {`
	`312`	`+ js_object.insert(key, v.clone());`
`315`	`313`	`}`
`316`	`314`	`}`
`317`	`315`	`return Ok(JsonValue(serde_json::Value::Object(js_object)));`