Do not override the supplied fields with dummy data

eth-infinitism · Oct 15, 2024 · 67349f8 · 67349f8
1 parent 7e72b06
commit 67349f8
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/packages/sdk/src/PreVerificationGasCalculator.ts b/packages/sdk/src/PreVerificationGasCalculator.ts
@@ -108,9 +108,9 @@ export class PreVerificationGasCalculator {
 
   _fillUserOpWithDummyData (userOp: Partial<UserOperation>): UserOperation {
     const filledUserOp: UserOperation = Object.assign({}, userOp) as UserOperation
-    filledUserOp.preVerificationGas = 21000 // dummy value
-    filledUserOp.signature = hexlify(Buffer.alloc(this.config.estimationSignatureSize, 0xff))
-    filledUserOp.paymasterData = hexlify(Buffer.alloc(this.config.estimationPaymasterDataSize, 0xff))
+    filledUserOp.preVerificationGas = filledUserOp.preVerificationGas ?? 21000
+    filledUserOp.signature = filledUserOp.signature ?? hexlify(Buffer.alloc(this.config.estimationSignatureSize, 0xff))
+    filledUserOp.paymasterData = filledUserOp.paymasterData ?? hexlify(Buffer.alloc(this.config.estimationPaymasterDataSize, 0xff))
     return filledUserOp
   }
 }