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

net/http: test WebAssembly Fetch implementation of Transport #26358

Open
odeke-em opened this issue Jul 12, 2018 · 4 comments

Comments

Projects
None yet
4 participants
@odeke-em
Copy link
Member

commented Jul 12, 2018

While taking a look at CL https://go-review.googlesource.com/c/go/+/123537 for #26349, a recommendation was that we should add tests to avoid regressions such as in that issue raised. @bradfitz added a reminder in #26051 (comment)

@odeke-em odeke-em added the Arch-Wasm label Jul 12, 2018

@odeke-em odeke-em added this to the Go1.12 milestone Jul 12, 2018

@odeke-em

This comment has been minimized.

Copy link
Member Author

commented Jul 12, 2018

@odeke-em

This comment has been minimized.

Copy link
Member Author

commented Jul 12, 2018

Also a superset of #25933

@bradfitz bradfitz changed the title all: add tests for WASM code when wasm browser-based builders are available net/http: test WebAssembly Fetch implementation of Transport Jul 12, 2018

@bradfitz

This comment has been minimized.

Copy link
Member

commented Jul 12, 2018

Retitled. This depends on #26051. Once #26051, then this shouldn't be too much work.

@dmitshur

This comment has been minimized.

Copy link
Member

commented Apr 29, 2019

This might've helped with #31736 too.

@andybons andybons modified the milestones: Go1.13, Go1.14 Jul 8, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.