From b94eaa65c3213bd79015f8e3b3f52c7d44a03c53 Mon Sep 17 00:00:00 2001 From: yuneng-jiang Date: Fri, 2 Oct 2026 16:12:15 -0700 Subject: [PATCH] test(integration): opt the config pass-through spend-log case into auth (#44265) The failure spend-log row from #42695 is written for pass-through routes that run as LLM API routes, which a config route only does with auth: true. The test omitted auth and passed only while config wins (#41779) registered config entries through the typed model, where auth defaults to true. #43962 restored the pre-config-wins registration, so the route lost that status and the row was never written. Set auth: true on the route so the test covers the logging it was written for without depending on that side effect --- .../observability/test_passthrough_upstream_error_visibility.py | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/integration/observability/test_passthrough_upstream_error_visibility.py b/tests/integration/observability/test_passthrough_upstream_error_visibility.py index bb18add2f2f..c29ce4f1c62 100644 --- a/tests/integration/observability/test_passthrough_upstream_error_visibility.py +++ b/tests/integration/observability/test_passthrough_upstream_error_visibility.py @@ -238,6 +238,7 @@ def test_config_pass_through_route_logs_body_and_strips_query(gateway: Gateway, "target": f"{wire.url}/upstream?trace=secret-q", "include_subpath": True, "headers": {"Authorization": "Bearer scripted"}, + "auth": True, } ] path.write_text(yaml.safe_dump(config))