Skip to content

[wasm] Unwrap exception when calling entrypoint - #74235

Merged
maraf merged 2 commits into
dotnet:mainfrom
maraf:WasmEntrypointUnwrapException
Aug 20, 2022
Merged

[wasm] Unwrap exception when calling entrypoint#74235
maraf merged 2 commits into
dotnet:mainfrom
maraf:WasmEntrypointUnwrapException

Conversation

@maraf

Copy link
Copy Markdown
Member

Introduced in the new entrypoint API. TargetInvocationException should be unwrapped to get the real error message to the client.

@ghost

Copy link
Copy Markdown

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details

Introduced in the new entrypoint API. TargetInvocationException should be unwrapped to get the real error message to the client.

Author:maraf
Assignees:maraf
Labels:

arch-wasm, area-System.Runtime.InteropServices.JavaScript

Milestone:-

@maraf

Copy link
Copy Markdown
MemberAuthor

/azp run runtime-wasm

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@lewing

Copy link
Copy Markdown
Member

/backport to release/7.0-rc1

@github-actions

Copy link
Copy Markdown
Contributor

Started backporting to release/7.0-rc1: https://github.com/dotnet/runtime/actions/runs/2891028250

@jeffschwMSFTjeffschwMSFT left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@maraf
maraf merged commit 1d94433 into dotnet:mainAug 20, 2022
@maraf
maraf deleted the WasmEntrypointUnwrapException branch August 20, 2022 08:20
@ghostghost locked as resolved and limited conversation to collaborators Sep 19, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@maraf@lewing@jeffschwMSFT