fix(agent): fix updating agent when some instances are live and some aren't

Author: joelwurtz

src/action/mod.rs

@@ -131,10 +131,7 @@ pub struct WithTargetUnitTrace {
 
 impl WithTargetUnitTrace {
     fn add_unit_id(&mut self, target: &str, unit_id: &str) {
-        let unit_ids = self
-            .unit_ids_applied_by_key
-            .entry(target.to_string())
-            .or_insert_with(LinkedHashSet::new);
+        let unit_ids = self.unit_ids_applied_by_key.entry(target.to_string()).or_default();
         unit_ids.insert(unit_id.to_string());
     }
 

src/api/impact.rs

@@ -241,7 +241,15 @@ impl ImpactOutput {
         'outer: for i in 1..=max_hops {
             let new_example = example.with_url(current_url.clone()).with_method(Some(current_method.clone()));
 
-            let request = Request::from_example(&router.config, &new_example).unwrap();
+            let request = match Request::from_example(&router.config, &new_example) {
+                Ok(request) => request,
+                Err(err) => {
+                    log::warn!("cannot create request from new target: {:?} : {}", new_example, err);
+
+                    break;
+                }
+            };
+
             let routes = router.match_request(&request);
             let mut action = Action::from_routes_rule(routes, &request, None);