Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[wasi] wasi:http + WasiHttpHandler #103752

Merged
merged 15 commits into from
Aug 1, 2024
Merged

Conversation

pavelsavara
Copy link
Member

@pavelsavara pavelsavara commented Jun 20, 2024

Stealing code and ideas from dotnet/runtimelab#2614

  • System.Threading.WasiEventLoop made accessible via Thread class
  • WasiHttpHandler and generated WasiHttpWorld.wit.imports
  • HTTP sample

credits to @dicej & @SingleAccretion

Contributes to #96419
Contributes to #98957

Parts are moved to

@pavelsavara pavelsavara added arch-wasm WebAssembly architecture area-Build-mono os-wasi Related to WASI variant of arch-wasm labels Jun 20, 2024
@pavelsavara pavelsavara self-assigned this Jun 20, 2024
Copy link
Contributor

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

src/mono/wasi/wasi.proj Outdated Show resolved Hide resolved
@pavelsavara pavelsavara changed the title [wasi] upgrade to WASIp2 [wasi] upgrade to WASIp2 + wasi:http Jul 2, 2024
@pavelsavara

This comment was marked as resolved.

Copy link
Member

@akoeplinger akoeplinger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, one nit that can be addressed in a future PR :)

@@ -455,6 +455,22 @@
Link="Common\System\Net\Http\HttpHandlerDefaults.cs" />
</ItemGroup>

<ItemGroup Condition="'$(TargetPlatformIdentifier)' == 'wasi'">
<Reference Include="System.Threading.Thread" />
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: please move this to the ItemGroup below which has all the Reference items

@pavelsavara pavelsavara merged commit 0e8db73 into dotnet:main Aug 1, 2024
157 of 160 checks passed
@pavelsavara pavelsavara deleted the wasip2-2 branch August 1, 2024 20:05
@github-actions github-actions bot locked and limited conversation to collaborators Sep 1, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
arch-wasm WebAssembly architecture area-Build-mono os-wasi Related to WASI variant of arch-wasm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants