elliottt opened PR #7434 from elliottt:trevor/wasi-http-errors
to bytecodealliance:main
:
Start integrating the more descriptive error variant present in https://github.com/WebAssembly/wasi-http/pull/52.
<!--
Please make sure you include the following information:
If this work has been discussed elsewhere, please include a link to that
conversation. If it was discussed in an issue, just mention "issue #...".Explain why this change is needed. If the details are in an issue already,
this can be brief.Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.htmlPlease ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->
elliottt edited PR #7434:
Start integrating the more descriptive error variant present in https://github.com/WebAssembly/wasi-http/pull/52.
Fixes #7248
<!--
Please make sure you include the following information:
If this work has been discussed elsewhere, please include a link to that
conversation. If it was discussed in an issue, just mention "issue #...".Explain why this change is needed. If the details are in an issue already,
this can be brief.Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.htmlPlease ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->
elliottt updated PR #7434.
elliottt updated PR #7434.
elliottt updated PR #7434.
elliottt submitted PR review.
elliottt created PR review comment:
This was preventing
downcast::<Error>()
in test cases.
elliottt edited PR #7434:
Integrate the more descriptive error variant present in https://github.com/WebAssembly/wasi-http/pull/52. This PR attempts to port the existing errors we raise directly to the new variant.
Fixes #7248
<!--
Please make sure you include the following information:
If this work has been discussed elsewhere, please include a link to that
conversation. If it was discussed in an issue, just mention "issue #...".Explain why this change is needed. If the details are in an issue already,
this can be brief.Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.htmlPlease ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->
elliottt updated PR #7434.
elliottt updated PR #7434.
elliottt updated PR #7434.
elliottt updated PR #7434.
elliottt updated PR #7434.
elliottt updated PR #7434.
elliottt updated PR #7434.
elliottt updated PR #7434.
elliottt updated PR #7434.
elliottt has marked PR #7434 as ready for review.
elliottt requested fitzgen for a review on PR #7434.
elliottt requested wasmtime-core-reviewers for a review on PR #7434.
elliottt updated PR #7434.
elliottt updated PR #7434.
elliottt requested pchickey for a review on PR #7434.
elliottt updated PR #7434.
elliottt updated PR #7434.
pchickey submitted PR review.
elliottt updated PR #7434.
elliottt updated PR #7434.
elliottt has enabled auto merge for PR #7434.
elliottt updated PR #7434.
elliottt updated PR #7434.
elliottt updated PR #7434.
elliottt merged PR #7434.
Last updated: Nov 22 2024 at 17:03 UTC