From 931b8e11f8258e4bb0056c1de2c70d312b5a9438 Mon Sep 17 00:00:00 2001 From: sneak Date: Wed, 11 Nov 2026 04:39:44 +0000 Subject: [PATCH] rproxy tests: give the over-1 MiB answers a 60 s time limit (closes #323) TestAnAnswerThatCannotBePassedOnIsAnswered502 and TestAnAnswerThatCannotBePassedOnGetsNoReport left SWRP_TIMEOUT_MS at its default of 2 s. On a loaded host rproxy had not read the whole answer of 1 MiB and one byte within that time, so it used SWRP_DEFAULT_DISPOSITION and the tests, which want 502, failed. Both now set SWRP_TIMEOUT_MS to 60000, as the file's other tests do where the time limit is not what they test. Model: opus-5-5 --- internal/rproxy/rproxy_test.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/internal/rproxy/rproxy_test.go b/internal/rproxy/rproxy_test.go index 0e6e2e8..3f0a7a9 100644 --- a/internal/rproxy/rproxy_test.go +++ b/internal/rproxy/rproxy_test.go @@ -1117,6 +1117,7 @@ func TestAnAnswerThatCannotBePassedOnIsAnswered502(t *testing.T) { t.Error("the app was asked") }) addr, out := startRProxy(t, backend(t), app.URL, map[string]string{ + timeoutSetting: "60000", defaultDisposition: setting, }) @@ -1140,7 +1141,9 @@ func TestAnAnswerThatCannotBePassedOnGetsNoReport(t *testing.T) { t.Parallel() standIn := startWAF(t, overOneMiB(dispositionDeny)) - addr, out := startRProxy(t, standIn.URL, nowhere, nil) + addr, out := startRProxy(t, standIn.URL, nowhere, map[string]string{ + timeoutSetting: "60000", + }) if got := get(t, addr, "/"); got.status != http.StatusBadGateway { t.Errorf("answered %+v, want 502", got) -- 2.54.0