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

feat: Turn on the iOS error handling approach for wasm #191

Closed
0xFloyd opened this issue Sep 6, 2024 · 3 comments
Closed

feat: Turn on the iOS error handling approach for wasm #191

0xFloyd opened this issue Sep 6, 2024 · 3 comments
Assignees

Comments

@0xFloyd
Copy link

0xFloyd commented Sep 6, 2024

Turn on the iOS error handling approach for wasm so that errors/panics can be caught and the extension can process the error and decide what to do next; retry etc

image

@mattes
Copy link
Contributor

mattes commented Sep 6, 2024

Turning this into panic_unwind does not compile.

-Z build-std=panic_abort,std

Just leaving here for future reference.

@mattes
Copy link
Contributor

mattes commented Sep 10, 2024

Unfortunately wasm arch doesn't support panic_unwind. I tried a few things including enabling some LLVM build features.

The only option I see is overwriting console.error:

const rescueConsoleError = console.error;

console.error = function(...args) {
  // capture panic logs
}

// call prover here ...

console.error = rescueConsoleError;

Javascript LFG :D

@0xFloyd
Copy link
Author

0xFloyd commented Sep 13, 2024

closed by 077e2d5

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

No branches or pull requests

2 participants