Fix keyless transaction simulation (#419)

* Fix keyless txn simulation * add changlog statement
aptos-labs · Jun 5, 2024 · 6763145 · 6763145
1 parent b6abcf4
commit 6763145
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,8 @@ All notable changes to the Aptos TypeScript SDK will be captured in this file. T
 
 # Unreleased
 
+- [`Fix`] Keyless transaction simulation now reports gas correctly
+
 # 1.18.0 (2024-06-03)
 
 - Adds Keyless Account support

diff --git a/src/transactions/transactionBuilder/transactionBuilder.ts b/src/transactions/transactionBuilder/transactionBuilder.ts
@@ -13,6 +13,7 @@ import {
   AnyPublicKey,
   AnySignature,
   KeylessPublicKey,
+  KeylessSignature,
   Secp256k1PublicKey,
   Secp256k1Signature,
 } from "../../core/crypto";
@@ -465,9 +466,7 @@ export function getAuthenticatorForSimulation(publicKey: PublicKey) {
       return new AccountAuthenticatorSingleKey(publicKey, new AnySignature(new Secp256k1Signature(new Uint8Array(64))));
     }
     if (publicKey.publicKey instanceof KeylessPublicKey) {
-      // TODO: Replace with the line below one simulation works properly for Keyless
-      // return new AccountAuthenticatorSingleKey(publicKey, new AnySignature(KeylessSignature.getSimulationSignature()));
-      return new AccountAuthenticatorSingleKey(publicKey, new AnySignature(new Ed25519Signature(new Uint8Array(64))));
+      return new AccountAuthenticatorSingleKey(publicKey, new AnySignature(KeylessSignature.getSimulationSignature()));
     }
   }