posborne edited PR #13345.
alexcrichton requested alexcrichton for a review on PR #13345.
alexcrichton unassigned pchickey from PR #13345 wasi-http: p3 single-instance-per-request coverage and fixes.
:thumbs_up: alexcrichton submitted PR review:
Everything here looks reasonable to me, but I also get tangled up in the futures here. CI looks like it's failing also on a channel/future-related thing which seems like it might be either a preexisting race or one accidentally introduced here?
:speech_balloon: alexcrichton created PR review comment:
It's ok to drop the mention of the old code here, no need to document what-once-was and I think the first part of these docs stand well on their own
posborne commented on PR #13345:
CI looks like it's failing also on a channel/future-related thing which seems like it might be either a preexisting race or one accidentally introduced here?
I'm working on reproducing this; it's similar to some of the kind of races I saw in the existing code which I had hoped had been addressed but some more digging is clearly warranted.
posborne updated PR #13345.
alexcrichton commented on PR #13345:
@posborne the most recent commit looks reasonable to me, but did you want to test/adjust more other than that? If not this'll need a rebase after https://github.com/bytecodealliance/wasmtime/pull/13404, and if so just lemme know when you feel this is ready
posborne commented on PR #13345:
@alexcrichton I think it's ready to go, was just waiting on CI yesterday -- I did get a local repro so was fairly confident in the patch.
I can do the rebase and put it into the merge queue when ready.
posborne updated PR #13345.
posborne updated PR #13345.
Last updated: Jun 01 2026 at 09:49 UTC