From a38ba0abd015894d56d8da67536a4a4bd0dcefe5 Mon Sep 17 00:00:00 2001 From: Tigah <88289044+TBX3D@users.noreply.github.com> Date: Wed, 22 Jul 2026 16:18:22 -0700 Subject: [PATCH] fix(modules): read the chained-request body through the shared cap (#374) main does not build: internal/modules/executor.go:197 references MaxBodySize, which no longer exists in this package. nobody's patch is wrong on its own. #317 added the request-chaining loop with a capped read against the const as it stood, then #355 moved the cap to internal/httpx and updated every reference that existed at the time. neither diff touched the other, so both were green in isolation and only the merged tree is broken. use httpx.ReadCappedBody, which the same file already uses at line 423 and the frameworks detector uses too. --- internal/modules/executor.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/internal/modules/executor.go b/internal/modules/executor.go index a226945..2e165e0 100644 --- a/internal/modules/executor.go +++ b/internal/modules/executor.go @@ -194,7 +194,7 @@ func executeHTTPChain(ctx context.Context, client *http.Client, target string, d // a transport error breaks the chain; return whatever matched earlier. return result, nil } - respBody, err := io.ReadAll(io.LimitReader(resp.Body, MaxBodySize)) + respBody, err := httpx.ReadCappedBody(resp) resp.Body.Close() if err != nil { return result, nil