Commit 7db47df5 authored by ben-chain's avatar ben-chain Committed by GitHub

fix: increase MAX_ROLLUP_TX_SIZE (#367)

* bump MAX_ROLLUP_TX_SIZE

* fix test gasLimit

* lint
parent 43de7fc4
...@@ -38,7 +38,7 @@ contract OVM_CanonicalTransactionChain is iOVM_CanonicalTransactionChain, Lib_Ad ...@@ -38,7 +38,7 @@ contract OVM_CanonicalTransactionChain is iOVM_CanonicalTransactionChain, Lib_Ad
// L2 tx gas-related // L2 tx gas-related
uint256 constant public MIN_ROLLUP_TX_GAS = 100000; uint256 constant public MIN_ROLLUP_TX_GAS = 100000;
uint256 constant public MAX_ROLLUP_TX_SIZE = 10000; uint256 constant public MAX_ROLLUP_TX_SIZE = 50000;
uint256 constant public L2_GAS_DISCOUNT_DIVISOR = 32; uint256 constant public L2_GAS_DISCOUNT_DIVISOR = 32;
// Encoding-related (all in bytes) // Encoding-related (all in bytes)
......
...@@ -188,7 +188,9 @@ describe('OVM_CanonicalTransactionChain', () => { ...@@ -188,7 +188,9 @@ describe('OVM_CanonicalTransactionChain', () => {
const data = '0x' + '12'.repeat(MAX_ROLLUP_TX_SIZE + 1) const data = '0x' + '12'.repeat(MAX_ROLLUP_TX_SIZE + 1)
await expect( await expect(
OVM_CanonicalTransactionChain.enqueue(target, gasLimit, data) OVM_CanonicalTransactionChain.enqueue(target, gasLimit, data, {
gasLimit: 40000000,
})
).to.be.revertedWith( ).to.be.revertedWith(
'Transaction data size exceeds maximum for rollup transaction.' 'Transaction data size exceeds maximum for rollup transaction.'
) )
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment