Skip to content

need to burn the nonce first to get to the correct ethflow address - #85

Open
kaze-cow wants to merge 2 commits into
mainfrom
fix/cannon
Open

need to burn the nonce first to get to the correct ethflow address#85
kaze-cow wants to merge 2 commits into
mainfrom
fix/cannon

Conversation

@kaze-cow

@kaze-cow kaze-cow commented Oct 22, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

it turns out the canonical ethflow deployment is at the 2nd nonce 1 rather than 0. So for a canonical deployment on a fresh chain, it is necessary to first burn the nonce.

local build simulation:

Initializing new package...
Name: cow-ethflow
Version: 1.0.4
Preset: main (default)
Chain ID: 13370
Private Source Code: false (source code will be included in the package)

Building the chain (ID 13370) via http://127.0.0.1:51687/...

Executing [import.dummyToken]...
  Resolving mintable-token:2.0@main (Chain ID: 13370) via local...
    Downloading ipfs://QmdGyg7vbD9arfNKtGB65KaEFPQgyRHMQB51LwcC2pt6Yi via /Users/kaze/code/cowprotocol/deployments/state/ipfs_cache

Executing [var.main]...
  Setting: cowSettlementPkg = cow-settlement:2.0.2
  Setting: wrappedNativeToken = 0xe60DB73Faa542e5b0FEFc2f31fe3f1B15Fc1C63D
  Setting: deployerAccount = 0x18AA26D5e4A18ef6E49c509D52a4f7d60242b10D

Executing [import.cow]...
  Resolving cow-settlement:2.0.2@main (Chain ID: 13370) via local...
    Downloading ipfs://QmbWADn2p79LJ1FPJ5hQQaZ54C1gAGjCckMbQn8EdZ2nEw via /Users/kaze/code/cowprotocol/deployments/state/ipfs_cache

Executing [var.settlementData]...
  Setting: cowSettlement = 0x9008D19f58AAbD9eD0D60971565AA8510560ab41

Executing [deploy.BurnedNonce]...
  ✔ Successfully deployed CoWSwapEthFlow
  Contract Address: 0x04501b9b1D52e67f6862d157E00D13419D2D6E95
  Transaction Hash: 0xa9a221fc164037f3aa1e255e1a9d257839c25242ced5e35f1b145efa7a365696
  Transaction Cost: 0.001813704217862139 ETH (1,418,687 gas)

Executing [deploy.EthFlow]...
  ✔ Successfully deployed CoWSwapEthFlow
  Contract Address: 0xbA3cB449bD2B4ADddBc894D8697F5170800EAdeC
  Transaction Hash: 0x6017a2b9cb2f6180a56f1cc4eeabc5a0501b7f178ed29c1c7ff62b0154b88738
  Transaction Cost: 0.001725208351489682 ETH (1,418,687 gas)

Attesting that 0xeb045d78d273107348b0300c01d29b7552d622ab deployed ipfs://QmYMv3JPDkWRCQ98oEZRHkJTWqZVDW7uxv4KuZATc6V4PE onchain...
💥 cow-ethflow:1.0.4@main built for Cannon (Chain ID: 13370)

Consideations

This doesn't work well for our current staging env which uses nonce 0, so probably the easiest way there is to transition ethflow to use nonce 1 there. Just requires deploying the new contract on every chain.

@kaze-cow kaze-cow self-assigned this Oct 22, 2026
@kaze-cow
kaze-cow requested a review from a team as a code owner October 22, 2026 01:26
@kaze-cow kaze-cow added the bug Something isn't working label Oct 22, 2026
@kaze-cow
have to remove gas limit override
4d4d9e2 
because robinhood chain was complaining about it for some reason
bh2smith
bh2smith reviewed Oct 27, 2026
View reviewed changes

@bh2smith bh2smith left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

cf mainnet deployment.
https://etherscan.io/address/0x04501b9b1D52e67f6862d157E00D13419D2D6E95#code

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

Reviewers

1 more reviewer
@bh2smith bh2smith bh2smith left review comments
Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

@kaze-cow kaze-cow

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@kaze-cow @bh2smith