Include contract constructor args when estimating intrinsic gas (q9f#111

)
mculp · Jul 28, 2022 · 9f3d34e · 9f3d34e
1 parent 55aea1a
commit 9f3d34e
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/lib/eth/client.rb b/lib/eth/client.rb
@@ -181,11 +181,11 @@ def deploy_and_wait(contract, *args, **kwargs)
     def deploy(contract, *args, **kwargs)
       raise ArgumentError, "Cannot deploy contract without source or binary!" if contract.bin.nil?
       raise ArgumentError, "Missing contract constructor params!" if contract.constructor_inputs.length != args.length
-      gas_limit = Tx.estimate_intrinsic_gas(contract.bin) + Tx::CREATE_GAS
       data = contract.bin
       unless args.empty?
         data += encode_constructor_params(contract, args)
       end
+      gas_limit = Tx.estimate_intrinsic_gas(data) + Tx::CREATE_GAS
       params = {
         value: 0,
         gas_limit: gas_limit,