Skip to content

WASIP3 HTTP return error for TooManyFields/TotalSizeTooBig#12788

Merged
alexcrichton merged 2 commits intobytecodealliance:mainfrom
ricochet:http-too-many
Mar 17, 2026
Merged

WASIP3 HTTP return error for TooManyFields/TotalSizeTooBig#12788
alexcrichton merged 2 commits intobytecodealliance:mainfrom
ricochet:http-too-many

Conversation

@ricochet
Copy link
Contributor

@ricochet ricochet commented Mar 16, 2026

Return new error type from wasi-http

@ricochet ricochet requested a review from a team as a code owner March 16, 2026 20:41
@alexcrichton
Copy link
Member

Thanks! This'll require adjustments to some tests and expectations, but implementation looks good to me

p2 cases unchanged (they trap before break is reached).

p3 cases assert the guest exited successfully and
 that it printed "error received", confirming the limit was actually enforced.
@ricochet ricochet requested a review from a team as a code owner March 16, 2026 23:11
@ricochet ricochet requested review from cfallin and removed request for a team March 16, 2026 23:11
@alexcrichton alexcrichton added this pull request to the merge queue Mar 17, 2026
Merged via the queue into bytecodealliance:main with commit a1a74d3 Mar 17, 2026
46 checks passed
@ricochet ricochet deleted the http-too-many branch March 17, 2026 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants