Stream: git-wasmtime

Topic: wasmtime / PR #15047 http: reject messages with both Tran...

view this post on Zulip Wasmtime GitHub notifications bot (Oct 30 2026 at 10:20):

xia-chao updated PR #15047.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 30 2026 at 10:20):

xia-chao edited PR #15047.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 30 2026 at 11:01):

xia-chao updated PR #15047.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 30 2026 at 11:06):

xia-chao commented on PR #15047:

also updated the two forward_duplicate_content_length tests - they depended on fetching a TE+CL response, which the client rejects now.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 30 2026 at 12:04):

xia-chao updated PR #15047.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 30 2026 at 12:06):

xia-chao deleted a comment on PR #15047:

also updated the two forward_duplicate_content_length tests - they depended on fetching a TE+CL response, which the client rejects now.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 30 2026 at 12:09):

xia-chao commented on PR #15047:

your forward_duplicate_content_length tests from #15070 kind of got in the way here - they relied on a TE+CL response, which the client rejects now.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 30 2026 at 12:09):

xia-chao deleted a comment on PR #15047:

your forward_duplicate_content_length tests from #15070 kind of got in the way here - they relied on a TE+CL response, which the client rejects now.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 30 2026 at 12:11):

xia-chao commented on PR #15047:

@pchickey your forward_duplicate_content_length tests from #15070 broke here, they relied on a TE+CL response, and the client rejects that now. So I had to update them.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 30 2026 at 15:15):

xia-chao updated PR #15047.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 30 2026 at 15:16):

xia-chao deleted a comment on PR #15047:

@pchickey your forward_duplicate_content_length tests from #15070 broke here, they relied on a TE+CL response, and the client rejects that now. So I had to update them.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 30 2026 at 15:18):

xia-chao edited PR #15047:

Requests and responses that carry both transfer-encoding and content-length are rejected before they reach the guest: 400 for a request and an HttpProtocolError for a response.

Hyper passes both headers through, so a guest was getting a content-length that did not match the body it read.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 30 2026 at 15:18):

xia-chao commented on PR #15047:

the duplicate content length tests relied on a TE+CL response, so i changed the fixture

Last updated: Oct 31 2026 at 23:03 UTC