agentjido / req_llm / 540062a58b524cd85ee9e76f24f86d924cc3ed33-PR-113

51%

main: 49%

Build Type

Pull #113

github

Committed by neilberkman

Commit Message

Fix Finch HTTP/2 flow control bug with large request bodies

Changes default Finch pool from [:http2, :http1] to [:http1] to avoid
HTTP/2 flow control bug when sending request bodies larger than 64KB.

This is a workaround for https://github.com/sneako/finch/issues/265
where mixed HTTP/1+HTTP/2 protocols cause exceeds_window_size errors
for large request bodies.

Adds safety validation that prevents HTTP/2 pools from attempting large
requests. If user configures HTTP/2 and tries to send >64KB body, returns
helpful error with configuration instructions and README link.

Affects all providers (Anthropic, Bedrock, etc.) when sending large
prompts or context windows. Users can override to HTTP/2 for small
payloads if needed.

Pull Request Pull Request #113: Fix exceeds_window_size errors with large request bodies

Run Details

7 of 22 new or added lines in 2 files covered. (31.82%)

3 existing lines in 1 file now uncovered.

3285 of 6534 relevant lines covered (50.28%)

445.32 hits per line