-
Notifications
You must be signed in to change notification settings - Fork 24
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
Use resources in wasi-http #58
Conversation
0a8b941
to
8584f46
Compare
8584f46
to
b154def
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thanks for doing this! Once feedback is addressed, feel free to merge or ping me to.
/// are ready to be consumed. | ||
subscribe: func() -> pollable; | ||
|
||
/// Retrieve reference to trailers, if they are ready. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like you're switching to doc-comments in a few places, but not all. It seems like we should be all-doc-comments or no-doc-comments (which is making me ask once again: why is this a valuable choice?!).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it would be worthwhile to make all comments doc comments: all the comments we write now are meant to document the api interface, instead of remain as notes for future readers.
5342d24
to
44de215
Compare
Co-authored-by: Luke Wagner <mail@lukewagner.name>
9aaf275
to
bc7b85f
Compare
Convert all pseudo-resources to full resources, and as such introduces some
borrow<..>
annotations.Additionally, this reworks body handling to go through an intermediate resource, rather than expose direct access to the underlying stream for the request/response resource. This intermediate resource allows us to better track when trailers are expected to be written or read.