router_condition: Fix unconditional route check in routing

The PR #11055 removed the
routes_mask guard, which exposed a design flaw in the previous logic.
Unconditional routes were being forced through prepare_logs() first.
The proper fix is to allow unconditional routes to short-circuit early,
so that chunk_trace routes are wired correctly.
With this change, the core_chunk_trace unit test will no longer fail.

Signed-off-by: Hiroshi Hatake <[email protected]>

Author: cosmo0920

src/flb_router_condition.c

@@ -265,6 +265,10 @@ int flb_router_path_should_route(struct flb_event_chunk *chunk,
         return FLB_FALSE;
     }
 
+    if (!path->route) {
+        return FLB_TRUE;
+    }
+
     if (chunk && chunk->type == FLB_EVENT_TYPE_LOGS) {
         if (!context) {
             return FLB_FALSE;
@@ -275,10 +279,6 @@ int flb_router_path_should_route(struct flb_event_chunk *chunk,
         }
     }
 
-    if (!path->route) {
-        return FLB_TRUE;
-    }
-
     return flb_route_condition_eval(chunk, context, path->route);
 }