Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

BM-664: Handle "already known" error during fulfillment calls #422

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

willpote
Copy link
Contributor

@willpote willpote commented Mar 19, 2025

This code addresses:

ERROR broker::submitter: Failed to submit proofs for batch 7649: Error(decoding err, missing data, code: -32603 msg: already known

which leads the broker to treat it as a failure and try again, getting

Failed to submit proofs for batch 7649: Error(BoundlessMarket Err: RequestIsFulfilled(RequestIsFulfilled { requestId: 4777893710576536890458989649656896615860034287083700324942 })

Note there doesn't seem to be an easy way to handle this error generally, each type of transaction will require special treatment. This PR starts with the known error that we've hit.

@github-actions github-actions bot changed the title Layer for handling "already known" error BM-664: Layer for handling "already known" error Mar 19, 2025
@willpote willpote changed the title BM-664: Layer for handling "already known" error BM-664: Handle "already known" error during fulfillment calls Mar 20, 2025
@willpote willpote marked this pull request as ready for review March 21, 2025 13:00
Copy link

github-actions bot commented Mar 27, 2025

🚀 Documentation Preview

Deployment URL: https://boundless-documentation-5fjdsget2-risczero.vercel.app

Updated at: 2025-03-31 14:37:23 UTC

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant