fix(da): fix gas price for da gas estimation to be UO gas price

alchemyplatform · Oct 29, 2024 · b0d629d · b0d629d
1 parent d0500a1
commit b0d629d
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 15 deletions.
diff --git a/crates/builder/src/bundle_proposer.rs b/crates/builder/src/bundle_proposer.rs
@@ -394,7 +394,7 @@ where
             da_gas_oracle.calc_da_gas_sync(
                 &op.da_gas_data,
                 da_block_data,
-                required_op_fees.max_fee_per_gas,
+                op.uo.gas_price(base_fee),
             )
         } else {
             match self
@@ -403,7 +403,7 @@ where
                 .calc_da_gas(
                     op.uo.clone().into(),
                     block_hash.into(),
-                    required_op_fees.max_fee_per_gas,
+                    op.uo.gas_price(base_fee),
                 )
                 .await
             {

diff --git a/crates/pool/src/mempool/pool.rs b/crates/pool/src/mempool/pool.rs
@@ -266,7 +266,7 @@ where
                 let required_da_gas = da_gas_oracle.calc_da_gas_sync(
                     &op.po.da_gas_data,
                     block_da_data,
-                    candidate_gas_price,
+                    op.uo().gas_price(base_fee),
                 );
 
                 let required_pvg = op.uo().required_pre_verification_gas(

diff --git a/crates/provider/src/alloy/da/optimism.rs b/crates/provider/src/alloy/da/optimism.rs
@@ -65,6 +65,10 @@ where
         block: BlockHashOrNumber,
         gas_price: u128,
     ) -> ProviderResult<(u128, DAGasUOData, DAGasBlockData)> {
+        if gas_price == 0 {
+            Err(anyhow::anyhow!("gas price cannot be zero"))?;
+        }
+
         let l1_fee: u128 = self
             .oracle
             .getL1Fee(data)

diff --git a/crates/sim/src/gas/gas.rs b/crates/sim/src/gas/gas.rs
@@ -11,9 +11,9 @@
 // You should have received a copy of the GNU General Public License along with Rundler.
 // If not, see https://www.gnu.org/licenses/.
 
-use std::{cmp, fmt::Debug};
+use std::fmt::Debug;
 
-use anyhow::{bail, Context};
+use anyhow::Context;
 #[cfg(feature = "test-utils")]
 use mockall::automock;
 use rundler_provider::{BlockHashOrNumber, DAGasProvider, EvmProvider};
@@ -68,17 +68,8 @@ pub async fn calc_required_pre_verification_gas<UO: UserOperation, E: DAGasProvi
     base_fee: u128,
 ) -> anyhow::Result<(u128, DAGasUOData)> {
     let (da_gas, uo_data) = if chain_spec.da_pre_verification_gas {
-        let gas_price = cmp::min(
-            base_fee + op.max_priority_fee_per_gas(),
-            op.max_fee_per_gas(),
-        );
-
-        if gas_price == 0 {
-            bail!("Gas price cannot be zero")
-        }
-
         let (da_gas, uo_data, _) = entry_point
-            .calc_da_gas(op.clone(), block, gas_price)
+            .calc_da_gas(op.clone(), block, op.gas_price(base_fee))
             .await?;
         (da_gas, uo_data)
     } else {