Skip to content

Isolate withdrawal liquidity lookup failures - #9

Open
hellojason3 wants to merge 2 commits into
multi_chainfrom
fix/relayer-isolate-invalid-withdrawal
Open

Isolate withdrawal liquidity lookup failures#9
hellojason3 wants to merge 2 commits into
multi_chainfrom
fix/relayer-isolate-invalid-withdrawal

Conversation

@hellojason3

Copy link
Copy Markdown
Collaborator

Summary

  • keep an ERC20 liquidity lookup failure scoped to the affected withdrawal
  • continue validating and settling the remaining withdrawals in the batch
  • retain the failed withdrawal in durable pending state for retry and audit

Root cause

A token address from a different destination chain caused balanceOf decoding to fail. The error used ?, so one malformed withdrawal aborted the entire submit_batch call and prevented unrelated pending withdrawals from being processed.

Verification

  • cargo fmt --package psy_relayer_cli -- --check
  • cargo test -p psy_relayer_cli --bin psy_relayer_cli (121 passed)
  • git diff --check

The historical malformed withdrawal was quarantined operationally before this change; this PR prevents the same failure class from blocking otherwise valid work.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@hellojason3