Stream: git-wasmtime

Topic: wasmtime / PR #10429 Draft: Expose Wasi-TLS handshake error


view this post on Zulip Wasmtime GitHub notifications bot (Mar 20 2025 at 14:40):

badeend edited PR #10429.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 20 2025 at 17:15):

jsturtevant submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 20 2025 at 17:15):

jsturtevant created PR review comment:

should we be return some information about the stream being closed with this error?

view this post on Zulip Wasmtime GitHub notifications bot (Mar 20 2025 at 17:15):

jsturtevant created PR review comment:

do we need logic to clean up these table entries?

view this post on Zulip Wasmtime GitHub notifications bot (Mar 20 2025 at 17:31):

badeend submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 20 2025 at 17:31):

badeend created PR review comment:

See https://github.com/WebAssembly/wasi-tls/pull/10

view this post on Zulip Wasmtime GitHub notifications bot (Mar 20 2025 at 17:35):

badeend submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 20 2025 at 17:35):

badeend created PR review comment:

You mean the table entries for the newly created wasi:io/error resources? The ownership of these resource is transferred into the guest one line below it. It's then the responsibility of the guest to drop it properly like any other resource. When the guest drops that resource, wasmtime removes it from the table in https://github.com/bytecodealliance/wasmtime/blob/main/crates/wasi-io/src/impls.rs#L114-L116

view this post on Zulip Wasmtime GitHub notifications bot (Mar 20 2025 at 18:04):

jsturtevant submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 20 2025 at 18:04):

jsturtevant created PR review comment:

thanks for the explanation

view this post on Zulip Wasmtime GitHub notifications bot (Mar 20 2025 at 19:52):

badeend updated PR #10429.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 20 2025 at 19:54):

badeend edited PR #10429:

Implements https://github.com/WebAssembly/wasi-tls/pull/10

And added a test case to verify the error information is indeed reaching the guest.

~PR is still draft, because above issue needs to be merged first. But feel free to review the design & implementation already.~

CC @jsturtevant

view this post on Zulip Wasmtime GitHub notifications bot (Mar 20 2025 at 19:54):

badeend has marked PR #10429 as ready for review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 20 2025 at 19:54):

badeend requested wasmtime-default-reviewers for a review on PR #10429.


Last updated: Apr 22 2025 at 16:04 UTC