Skip to content

Commit 7d6a86b

Browse files
authored
fix: change min gas fees for Ronin and Ink (#7222)
Co-authored-by: Danil Nemirovsky <[email protected]>
1 parent c3b860b commit 7d6a86b

File tree

4 files changed

+33
-5
lines changed

4 files changed

+33
-5
lines changed

rust/main/config/mainnet_config.json

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6007,7 +6007,12 @@
60076007
"index": {
60086008
"from": 860343
60096009
},
6010-
"gnosisSafeTransactionServiceUrl": "https://safe-transaction-ink.safe.global/"
6010+
"gnosisSafeTransactionServiceUrl": "https://safe-transaction-ink.safe.global/",
6011+
"transactionOverrides": {
6012+
"minGasPrice": 1,
6013+
"minFeePerGas": 1,
6014+
"minPriorityFeePerGas": 1
6015+
}
60116016
},
60126017
"soneium": {
60136018
"blockExplorers": [
@@ -6928,7 +6933,12 @@
69286933
"index": {
69296934
"from": 42656210
69306935
},
6931-
"gnosisSafeTransactionServiceUrl": "https://safe-transaction-ronin.safe.onchainden.com"
6936+
"gnosisSafeTransactionServiceUrl": "https://safe-transaction-ronin.safe.onchainden.com",
6937+
"transactionOverrides": {
6938+
"minGasPrice": 20000000000,
6939+
"minFeePerGas": 20000000000,
6940+
"minPriorityFeePerGas": 20000000000
6941+
}
69326942
},
69336943
"sophon": {
69346944
"blockExplorers": [

rust/main/lander/src/adapter/chains/ethereum/adapter.rs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -177,6 +177,10 @@ impl EthereumAdapter {
177177

178178
if gas_price == &tx_gas_price {
179179
// If new gas price is the same as the old one, no point in resubmitting
180+
info!(
181+
?tx,
182+
"not resubmitting transaction since new gas price is the same as the old one"
183+
);
180184
return Err(LanderError::TxAlreadyExists);
181185
}
182186

rust/main/lander/src/transaction/types.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -81,10 +81,10 @@ impl TransactionStatus {
8181
return TransactionStatus::Finalized;
8282
} else if *included_count > 0 {
8383
return TransactionStatus::Included;
84-
} else if *pending_count > 0 {
85-
return TransactionStatus::PendingInclusion;
8684
} else if *mempool_count > 0 {
8785
return TransactionStatus::Mempool;
86+
} else if *pending_count > 0 {
87+
return TransactionStatus::PendingInclusion;
8888
} else if !status_counts.is_empty() {
8989
// if the hashmap is not empty, it must mean that the hashes were dropped,
9090
// because the hashmap is populated only if the status query was successful
@@ -191,7 +191,6 @@ mod tests {
191191

192192
let statuses = vec![
193193
Ok(TransactionStatus::Dropped(DropReason::DroppedByChain)),
194-
Ok(TransactionStatus::Mempool),
195194
Ok(TransactionStatus::PendingInclusion),
196195
Err(LanderError::NetworkError("Network error".to_string())),
197196
];
@@ -208,6 +207,7 @@ mod tests {
208207
let statuses = vec![
209208
Err(LanderError::NetworkError("Network error".to_string())),
210209
Ok(TransactionStatus::Mempool),
210+
Ok(TransactionStatus::PendingInclusion),
211211
Ok(TransactionStatus::Dropped(DropReason::DroppedByChain)),
212212
];
213213

typescript/infra/config/environments/mainnet3/chains.ts

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,20 @@ export const chainMetadataOverrides: ChainMap<Partial<ChainMetadata>> = {
5757
minPriorityFeePerGas: 1 * 10 ** 9, // 1 gwei
5858
},
5959
},
60+
ronin: {
61+
transactionOverrides: {
62+
minGasPrice: 20 * 10 ** 9, // 20 gwei
63+
minFeePerGas: 20 * 10 ** 9, // 20 gwei
64+
minPriorityFeePerGas: 20 * 10 ** 9, // 20 gwei
65+
},
66+
},
67+
ink: {
68+
transactionOverrides: {
69+
minGasPrice: 1, // 1 wei
70+
minFeePerGas: 1, // 1 wei
71+
minPriorityFeePerGas: 1, // 1 wei
72+
},
73+
},
6074
// Deploy-only overrides, set when deploying contracts
6175
// chilizmainnet: {
6276
// transactionOverrides: {

0 commit comments

Comments
 (0)