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

Improve submit_multiple_and_wait_verification return value #1393

Open
avilagaston9 opened this issue Nov 8, 2024 · 0 comments
Open

Improve submit_multiple_and_wait_verification return value #1393

avilagaston9 opened this issue Nov 8, 2024 · 0 comments
Labels
sdk issues within aligned-sdk

Comments

@avilagaston9
Copy link
Collaborator

In #1325, the output of submit_multiple_and_wait_verification was changed to a Vec<Result<_>> with:

  • Ok() meaning that the proof was included in a batch.
  • Err() meaning that the proof was not included due to an error.

But we are missing information about the verification result. If a timeout occurs while awaiting verification, an additional Err() is appended to the end of the returned array, which is somewhat unusual and does not provide information about the proof that triggers the error.

@avilagaston9 avilagaston9 added the sdk issues within aligned-sdk label Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sdk issues within aligned-sdk
Projects
None yet
Development

No branches or pull requests

1 participant