Skip to content

Updates - #7

Closed
calvinrp wants to merge 8 commits into
bytecodealliance:mainfrom
calvinrp:refactor
Closed

Updates#7
calvinrp wants to merge 8 commits into
bytecodealliance:mainfrom
calvinrp:refactor

Conversation

@calvinrp

@calvinrpcalvinrp commented Sep 21, 2024

Copy link
Copy Markdown
Contributor
  • adds read_to_end to AsyncRead trait
  • adds methods for Fields (used for Headers and Trailers)

Getting the trailers on http response is commented out. The current release of wasmtime panics incorrectly. Made a NOTE in the source code. Can uncomment after bytecodealliance/wasmtime#9208 is released.

Tested making outgoing requests with headers and reading the response.

@calvinrpcalvinrp changed the title RefactorUpdatesSep 21, 2024
@pchickey

Copy link
Copy Markdown
Contributor

@calvinrp I've done a ton of work on wstd since you posted this, fixing a handful of the issues you found here and even lifting some of this work directly into my PRs. Do you want to take another pass based on the latest main?

Please note there are still a lot of known bugs in wstd, I have to do a bunch of rework on the reactor/poller based on stuff I discovered in #26

@calvinrp

calvinrp commented Nov 26, 2024

Copy link
Copy Markdown
ContributorAuthor

@calvinrp I've done a ton of work on wstd since you posted this, fixing a handful of the issues you found here and even lifting some of this work directly into my PRs. Do you want to take another pass based on the latest main?

Please note there are still a lot of known bugs in wstd, I have to do a bunch of rework on the reactor/poller based on stuff I discovered in #26

@pchickey
Will do. I really want something like this crate to exist and be usable.

@calvinrp

Copy link
Copy Markdown
ContributorAuthor

Closing this PR. Thank you for all the updates that were merged and released in v0.5

Sign up for freeto 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.

2 participants

@calvinrp@pchickey