Skip to content

Don't use broadcast_tx_commit in bridge-withdrawer #1283

@itamarreif

Description

@itamarreif

The bridge-withdrawer currently uses broadcast_tx_commit to submit transactions to the sequencer. We used this instead of broadcast_tx_sync in order to make sure the transaction containing a batch of withdrawals is finalized on the sequencer before the next batch is processed.

The CometBFT RPC docs warn against using broadcast_tx_commit in production.

Instead, we should use broadcast_tx_sync and then check for the transaction finalizing with a backoff before processing the next batch.

Metadata

Metadata

Assignees

Labels

bridge-withdrawercode touching the bridge-withdrawer servicebridgingbugSomething isn't working

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions