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

Implement wasm target support for cloudflare workers #6

Merged
merged 12 commits into from Mar 7, 2022

Conversation

Tricked-dev
Copy link
Member

No description provided.

@proudmuslim-dev
Copy link
Member

Gonna figure this out tomorrow, seems like most of the errors are coming from the socket2 library

@Tricked-dev
Copy link
Member Author

pure pain

The wasm target won't build, we need to do a lot of stuff with features to ensure that tokio doesn't get compiled when attempting to use the target
@proudmuslim-dev
Copy link
Member

pure pain

Facts

@proudmuslim-dev
Copy link
Member

Reqwest has some horribly half-baked support for wasm at the moment- I think we'll have to use another HTTP client for the worker feature.

seanmonstar/reqwest#1095
seanmonstar/reqwest#1383
seanmonstar/reqwest#1135

@Tricked-dev
Copy link
Member Author

hmm thats annoying

@proudmuslim-dev proudmuslim-dev changed the title feat: begin cloudflare worker stuff Implement wasm target support for cloudflare workers Feb 21, 2022
There's still a lot of work to be done, I plan on clearing out most of the errors tomorrow
…sent in `wasm_bindgen`

See the TODO on `RequestInitWrapper` for more information
@proudmuslim-dev proudmuslim-dev force-pushed the cloudflare branch 2 times, most recently from 0615e79 to d775c14 Compare March 3, 2022 23:10
See the todos I made for the minor stuff that still needs to be added in- I might have missed some stuff, but I'm too happy to care right now
@Tricked-dev
Copy link
Member Author

🎉

@proudmuslim-dev
Copy link
Member

Merging since everything critical to the wasm portion is done

@proudmuslim-dev proudmuslim-dev merged commit 9b4bde9 into main Mar 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants