Stream: git-wasmtime

Topic: wasmtime / PR #5929 Begin implementation of wasi-http


view this post on Zulip Wasmtime GitHub notifications bot (Mar 13 2023 at 03:57):

brendandburns edited PR #5929 from wasi_http to main:

This is a major WIP, no need to review currently, mostly here so I can visualize the diff and checkpoint it.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 13 2023 at 03:58):

brendandburns edited PR #5929 from wasi_http to main:

Initial draft.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 13 2023 at 04:01):

brendandburns edited PR #5929 from wasi_http to main:

Initial draft.

Known issues:

view this post on Zulip Wasmtime GitHub notifications bot (Mar 13 2023 at 04:02):

brendandburns edited PR #5929 from wasi_http to main:

Initial draft, working, but with limitations.

Known issues:

view this post on Zulip Wasmtime GitHub notifications bot (Mar 13 2023 at 04:06):

brendandburns edited PR #5929 from wasi_http to main:

Initial draft, working, but with limitations.

Known issues:

view this post on Zulip Wasmtime GitHub notifications bot (Mar 13 2023 at 04:57):

brendandburns updated PR #5929 from wasi_http to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 13 2023 at 15:48):

brendandburns edited PR #5929 from wasi_http to main:

Initial draft, working, but with limitations.

Known issues:

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 01:36):

brendandburns updated PR #5929 from wasi_http to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 03:19):

pchickey submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 03:19):

pchickey submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 03:19):

pchickey created PR review comment:

This shouldn't be a macro - lets write it as a regular function?

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 03:19):

pchickey created PR review comment:

alignment is set to 0 here, which will trap in some allocators

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 03:19):

pchickey created PR review comment:

this method isn't expressible in wit, so we don't have an off-ramp into the component runtime for it. I do not think we should leave it in, since the idea is we will replace all of the functionality with the component runtime once the other pieces are ready.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 03:19):

pchickey created PR review comment:

We have a project policy against use of unsafe - additionally, this will be incorrect on big-endian (s390x). Can you instead use u32::to_le_bytes() to write these values into a byte array?

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 03:19):

pchickey created PR review comment:

We use

    authors.workspace = true

to pickup the "The Wasmtime Project Developers" author

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 03:19):

pchickey created PR review comment:

I'm not sure if we'll want to use this name for the host implementation, or reserve it for guest side bindings, but either way I grabbed the name on crates.io and we can land it with this name for now

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 03:19):

pchickey created PR review comment:

    edition.workspace = true
````
~~~

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 03:19):

pchickey created PR review comment:

    thiserror = { workspace = true }

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 03:19):

pchickey created PR review comment:

    anyhow = { workspace = true }

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 03:19):

pchickey created PR review comment:

Project uses Apache-2.0 WITH LLVM-exception

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 03:19):

pchickey created PR review comment:

I'm a bit surprised all these functions landed in the types module name, that may be something we have to look into...

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:38):

brendandburns submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:38):

brendandburns created PR review comment:

The compiler doesn't like this function b/c of Rust borrow semantics when I write it as a function:

error[E0382]: borrow of moved value: `caller`
   --> crates/wasi-http/src/component_impl.rs:343:37
    |
330 |         move |mut caller: Caller<'_, T>, stream: u32, len: u64, ptr: u32| {
    |               ---------- move occurs because `caller` has type `Caller<'_, T>`, which does not implement the `Copy` trait
...
339 |             let out_ptr = allocate_guest_pointer(caller, body_len);
    |                                                  ------ value moved here
...
343 |             let memory = memory_get(&mut caller).unwrap();
    |                                     ^^^^^^^^^^^ value borrowed here after move

error[E0382]: use of moved value: `caller`
   --> crates/wasi-http/src/component_impl.rs:390:55
    |
377 |         move |mut caller: Caller<'_, T>, fields: u32, out_ptr: u32| {
    |               ---------- move occurs because `caller` has type `Caller<'_, T>`, which does not implement the `Copy` trait
...
382 |             let tuple_ptr = allocate_guest_pointer(caller, (16 * header_len).try_into().unwrap());
    |                                                    ------ value moved here
...
390 |                 let name_ptr = allocate_guest_pointer(caller, name_len);
    |                                                       ^^^^^^ value used here after move

error[E0382]: use of moved value: `caller`
   --> crates/wasi-http/src/component_impl.rs:391:56
    |
377 |         move |mut caller: Caller<'_, T>, fields: u32, out_ptr: u32| {
    |               ---------- move occurs because `caller` has type `Caller<'_, T>`, which does not implement the `Copy` trait
...
390 |                 let name_ptr = allocate_guest_pointer(caller, name_len);
    |                                                       ------ value moved here
391 |                 let value_ptr = allocate_guest_pointer(caller, value_len);
    |                                                        ^^^^^^ value used here after move

error[E0382]: borrow of moved value: `caller`
   --> crates/wasi-http/src/component_impl.rs:393:41
    |
377 |         move |mut caller: Caller<'_, T>, fields: u32, out_ptr: u32| {
    |               ---------- move occurs because `caller` has type `Caller<'_, T>`, which does not implement the `Copy` trait
...
391 |                 let value_ptr = allocate_guest_pointer(caller, value_len);
    |                                                        ------ value moved here
392 |
393 |                 let memory = memory_get(&mut caller).unwrap();
    |                                         ^^^^^^^^^^^ value borrowed here after move

For more information about this error, try `rustc --explain E0382`.
error: could not compile `wasi-http` due to 4 previous errors

Admittedly I'm a Rust newb, so if there is a way to resolve this, please let me know. I tried a few things but I couldn't see a way to do it.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:41):

brendandburns edited PR review comment.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:41):

brendandburns updated PR #5929 from wasi_http to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:41):

brendandburns submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:41):

brendandburns created PR review comment:

switched to 4

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:42):

brendandburns submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:42):

brendandburns created PR review comment:

removed, this was stale old code.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:42):

brendandburns submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:42):

brendandburns created PR review comment:

Done (here and below)

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:42):

brendandburns submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:42):

brendandburns created PR review comment:

done.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:42):

brendandburns created PR review comment:

done.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:42):

brendandburns submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:42):

brendandburns submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:42):

brendandburns created PR review comment:

done.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:43):

brendandburns submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:43):

brendandburns created PR review comment:

done.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:43):

brendandburns submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:43):

brendandburns created PR review comment:

acknowledged.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:43):

brendandburns submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:43):

brendandburns created PR review comment:

done.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:43):

brendandburns submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 14 2023 at 04:43):

brendandburns created PR review comment:

yeah, surprised me to, seems like some sort of aliasing in wit-bindgen

view this post on Zulip Wasmtime GitHub notifications bot (Mar 15 2023 at 04:15):

brendandburns updated PR #5929 from wasi_http to main.

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

brendandburns updated PR #5929 from wasi_http to main.

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

brendandburns created PR review comment:

I figured out how to fix this, so this is now fixed in the PR.

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

brendandburns submitted PR review.

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

brendandburns edited PR #5929 from wasi_http to main:

Initial draft, working, but with limitations.

Known issues:

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

brendandburns updated PR #5929 from wasi_http to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 15 2023 at 22:35):

Mossaka submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 15 2023 at 22:35):

Mossaka submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 15 2023 at 22:35):

Mossaka created PR review comment:

            _ => return bail!("unknown method!"),

view this post on Zulip Wasmtime GitHub notifications bot (Mar 15 2023 at 22:35):

Mossaka created PR review comment:

If you set async: true then you can use reqwest::Client instead of reqwest::blocking::Client.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 15 2023 at 23:05):

pchickey submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 15 2023 at 23:05):

pchickey created PR review comment:

Agreed - really, the only time we want to use the sync traits is to support hosts that arent running tokio. Since the reqwest blocking client is actually using tokio under the hood, using the sync trait means this library can't compose with the rest of the wasi libraries when run on tokio, but it also can't work for hosts that don't want to incur tokio, which is the worst of both worlds.

And once we make that change, I'd much prefer to drop down to just plain old hyper, because I don't think there's a ton of value in the reqwest async interface over the hyper primitives for this use case, and reqwest is responsible for at least half of the dependencies that need to be cargo veted in order to land this.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 15 2023 at 23:06):

pchickey edited PR review comment.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 15 2023 at 23:11):

pchickey edited PR review comment.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 16 2023 at 04:38):

brendandburns submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 16 2023 at 04:38):

brendandburns created PR review comment:

I looked at hyper. It makes you do things like explicitly open a TCP connection before talking HTTP, it's ergonomics as a library are not great at first glance anyway.

I'm happy to do this as a later optimization, but I don't think that it impacts this PR, so I'd prefer to merge and iterate.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 16 2023 at 17:12):

pchickey submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 16 2023 at 17:12):

pchickey created PR review comment:

In order to land the PR I have to audit all unvetted dependencies for supply chain safety. These audits are very time consuming - I have to review the source of the entire crate. We can only accept these audits if they are performed by core team members. I can't make any kind of exception for this PR, or breeze through things because reqwest will be swapped out soon, because our audits are imported to be part of the supply chain safety of other projects.

Right now this PR introduces 49 unvetted dependencies into the project https://github.com/bytecodealliance/wasmtime/actions/runs/4430843524/jobs/7773090032?pr=5929. That represents at least a full day of very tedious work for me. Many of those dependencies are pulled in exclusively by the reqwest library, in order to provide functionality we don't need. (Because of the use by other projects, vetting considers all dependencies for all features and targets, even though our project doesn't need them.)

Therefore, we need to iterate on this before landing the PR. Like I said, I can send a PR to your branch to help with this transformation, because my time would be better spent on making the changes that need to happen anyway vs auditing packages we don't care about.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 16 2023 at 22:05):

brendandburns submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 16 2023 at 22:05):

brendandburns created PR review comment:

The need to reduce the scope of rust vet makes total sense to me. I will look into switching this to Hyper.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 18 2023 at 05:16):

brendandburns updated PR #5929 from wasi_http to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 18 2023 at 05:28):

brendandburns updated PR #5929 from wasi_http to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 18 2023 at 15:46):

brendandburns updated PR #5929 from wasi_http to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 21 2023 at 03:31):

brendandburns updated PR #5929 from wasi_http to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 21 2023 at 03:32):

brendandburns edited PR #5929 from wasi_http to main:

Initial draft, working, but with limitations.

Known issues:

view this post on Zulip Wasmtime GitHub notifications bot (Mar 21 2023 at 03:44):

brendandburns updated PR #5929 from wasi_http to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 21 2023 at 03:45):

brendandburns updated PR #5929 from wasi_http to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 23 2023 at 22:59):

eduardomourar submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 23 2023 at 22:59):

eduardomourar submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 23 2023 at 22:59):

eduardomourar created PR review comment:

i believe this was a typo

    http-body = "1.0.0-rc.2"

view this post on Zulip Wasmtime GitHub notifications bot (Mar 23 2023 at 22:59):

eduardomourar created PR review comment:

we don't need the full set of features

    tokio = { version = "1", default-features = false, features = ["net", "rt-multi-thread", "time"] }

view this post on Zulip Wasmtime GitHub notifications bot (Mar 23 2023 at 23:48):

brendandburns updated PR #5929 from wasi_http to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 23 2023 at 23:49):

brendandburns updated PR #5929 from wasi_http to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 24 2023 at 01:13):

eduardomourar submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 24 2023 at 16:50):

Mossaka submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 24 2023 at 16:50):

Mossaka submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 24 2023 at 16:50):

Mossaka created PR review comment:

It doesn't seem like the len parameter has been used here to specify how much bytes to read. I feel like for an initial implementation this should be okay, but if the response body is too big to fit in host's memory then we probably need to use BufRead to read bytes into a buffer incrementally.

I remember you added read_into function before, but I can't find it anymore.

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

brendandburns updated PR #5929 from wasi_http to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 24 2023 at 20:16):

brendandburns submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 24 2023 at 20:16):

brendandburns created PR review comment:

read_into was removed after feedback from @pchickey and others.

I fixed this function to actually implement read correctly.

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

brendandburns updated PR #5929 from wasi_http to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 24 2023 at 22:20):

brendandburns updated PR #5929 from wasi_http to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 27 2023 at 16:24):

brendandburns updated PR #5929 from wasi_http to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 28 2023 at 03:19):

brendandburns updated PR #5929 from wasi_http to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 28 2023 at 04:34):

brendandburns updated PR #5929 from wasi_http to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 28 2023 at 04:35):

brendandburns updated PR #5929 from wasi_http to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 29 2023 at 20:39):

brendandburns requested pchickey for a review on PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 29 2023 at 20:39):

brendandburns updated PR #5929 from wasi_http to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 29 2023 at 20:39):

brendandburns requested wasmtime-default-reviewers for a review on PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 29 2023 at 20:39):

brendandburns requested wasmtime-core-reviewers for a review on PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2023 at 15:38):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2023 at 15:56):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2023 at 15:57):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2023 at 19:02):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2023 at 19:59):

pchickey submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2023 at 19:59):

pchickey submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2023 at 19:59):

pchickey created PR review comment:

    let builder = WasiCtxBuilder::new().inherit_stdio().arg(bin_name)?;

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2023 at 19:59):

pchickey created PR review comment:

    let config = Config::new();

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2023 at 19:59):

pchickey created PR review comment:

    _workspace: Option<&Path>,

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2023 at 19:59):

pchickey updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2023 at 19:59):

pchickey updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2023 at 19:59):

pchickey updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2023 at 21:28):

pchickey submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2023 at 21:47):

eduardomourar submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2023 at 21:47):

eduardomourar created PR review comment:

I know this is a little bit late, but can we change this package name to wasmtime-wasi-http, please?

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2023 at 22:09):

eduardomourar deleted PR review comment.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2023 at 00:05):

brendandburns updated PR #5929.

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

pchickey submitted PR review.

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

pchickey submitted PR review.

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

pchickey created PR review comment:

#![allow(dead_code, unused_imports)]

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

pchickey updated PR #5929.

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

pchickey has enabled auto merge for PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2023 at 21:23):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2023 at 21:25):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2023 at 21:25):

rvolosatovs submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2023 at 21:25):

rvolosatovs submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2023 at 21:25):

rvolosatovs created PR review comment:

Shouldn't need SSL anymore, right?

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2023 at 21:26):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2023 at 21:26):

brendandburns submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2023 at 21:26):

brendandburns created PR review comment:

done.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2023 at 22:21):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 31 2023 at 22:22):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 03 2023 at 16:32):

eduardomourar created PR review comment:

we should actually be returning the future incoming response id here and not the response id

view this post on Zulip Wasmtime GitHub notifications bot (Apr 03 2023 at 16:32):

eduardomourar submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 03 2023 at 20:00):

brendandburns submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 03 2023 at 20:00):

brendandburns created PR review comment:

Future's aren't implemented (yet) in this PR so the response_id is the same as the "future" id. If you look at the future-incoming-response-get in component_impl.rs, you'll see that it just returns the passed in future as the response id.

https://github.com/bytecodealliance/wasmtime/pull/5929/files/f02db0f19f99eeb82c369b511cb56743f31b7f60#diff-902e7d68ea9ed72ae165cb94771bad9528553b6b1a04661f731bbe40460ff1beR215

Eventually we will implement futures and this will change, but not in this PR.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 03 2023 at 20:49):

pchickey submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 03 2023 at 21:16):

eduardomourar submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 03 2023 at 21:16):

eduardomourar created PR review comment:

after this is merged, i will submit a pr that will at least prevent the host from crashing if the emulated future is assumed by the guest like here: https://github.com/bytecodealliance/jco/blob/2b3c9cd7839b3728f74af2cbf6722cf09287f956/packages/preview2-shim/lib/http/wasi-http.js#L263-L296

view this post on Zulip Wasmtime GitHub notifications bot (Apr 04 2023 at 04:07):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 04 2023 at 19:00):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 05 2023 at 03:19):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 05 2023 at 03:33):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 05 2023 at 03:40):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 05 2023 at 04:04):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 05 2023 at 13:53):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 05 2023 at 14:36):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 05 2023 at 15:39):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 05 2023 at 15:55):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 05 2023 at 16:39):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 05 2023 at 17:10):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 05 2023 at 17:10):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 05 2023 at 17:48):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 05 2023 at 18:09):

brendandburns updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 05 2023 at 19:59):

pchickey submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 05 2023 at 19:59):

pchickey created PR review comment:

# The `ring` crate, used to implement TLS, does not build on riscv64 or s390x
[target.'cfg(not(any(target_arch = "riscv64", target_arch = "s390x")))'.dependencies]

view this post on Zulip Wasmtime GitHub notifications bot (Apr 05 2023 at 20:00):

pchickey updated PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 05 2023 at 20:00):

pchickey submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 05 2023 at 20:00):

pchickey has enabled auto merge for PR #5929.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 05 2023 at 21:08):

pchickey merged PR #5929.


Last updated: Dec 23 2024 at 13:07 UTC