• Mark Tyneway's avatar
    l2geth: sequencer fee buffer · c612a903
    Mark Tyneway authored
    The fees are currently calculated as a sum of the L1 fee and the L2 fee
    where the L1 fee is the approximate cost of the batch submission of the
    transaction (L1 gas price * L1 gas used) and the L2 fee is the
    approximate cost of the execution on L2 taking into account congestion
    (L2 gas price * L2 gas limit).
    
    When either the L1 or L2 gas price is volatile, it can result in the
    quote that the user receives from `eth_estimateGas` to be rejected
    as the fee that the Sequencer is expecting has gone up.
    
    This PR adds logic to set a buffer in either direction of the current
    price that will allow the sequencer to still accept transactions within.
    
    Two new config options are added:
    
    - `--rollup.feethresholddown` or `ROLLUP_FEE_THRESHOLD_DOWN`
    - `--rollup.feethresholdup` or `ROLLUP_FEE_THRESHOLD_UP`
    
    Note that these config options are only useful for when running
    in Sequencer mode, they are not useful for replicas/verifiers.
    This is because the Sequencer is the only write node in the network.
    
    These config options are interpreted as floating point numbers and are
    multiplied against the current fee that the sequencer is expecting.
    To allow for a downward buffer of 10% and an upward buffer of 100%,
    use the options:
    
    - `ROLLUP_FEE_THRESHOLD_DOWN=0.9`
    - `ROLLUP_FEE_THRESHOLD_UP=2`
    
    This will allow for slight fee volatility downwards and prevent users
    from excessively overpaying on fees accidentally.
    
    This is a UX and profit tradeoff for the sequencer and can be exploited
    by bots. If bots are consistently taking advantage of this, the max
    threshold down will have to be calibrated to what the normal fee is
    today.
    
    Both config options are sanity checked in the `SyncService` constructor
    and will result in errors if they are bad. The threshold down must
    be less than 1 and the threshold up must be greater than 1.
    c612a903
Name
Last commit
Last update
.changeset Loading commit data...
.github Loading commit data...
.husky Loading commit data...
.vscode Loading commit data...
examples Loading commit data...
go Loading commit data...
integration-tests Loading commit data...
l2geth Loading commit data...
ops Loading commit data...
packages Loading commit data...
patches Loading commit data...
specs Loading commit data...
.dockerignore Loading commit data...
.editorconfig Loading commit data...
.eslintrc.js Loading commit data...
.gitattributes Loading commit data...
.gitignore Loading commit data...
.gitmodules Loading commit data...
.nvmrc Loading commit data...
.prettierrc.js Loading commit data...
.prettierrc.json Loading commit data...
CONTRIBUTING.md Loading commit data...
LICENSE Loading commit data...
README.md Loading commit data...
lerna.json Loading commit data...
package.json Loading commit data...
tsconfig.build.json Loading commit data...
tsconfig.json Loading commit data...
yarn.lock Loading commit data...