Skip to content

Commit 4c5573b

Browse files
authored
Merge pull request #5955 from BitGo/coin-3813-fix-failed-to-explain-transaction
fix(sdk-coin-apt): prevent build() step during rebuildTransaction()
2 parents afcb198 + 2b887aa commit 4c5573b

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

modules/sdk-coin-apt/src/apt.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -192,7 +192,7 @@ export class Apt extends BaseCoin {
192192
const txBuilderFactory = this.getTxBuilderFactory();
193193
try {
194194
const txBuilder = txBuilderFactory.from(txHex);
195-
return await txBuilder.build();
195+
return txBuilder.getRebuiltTransaction();
196196
} catch {
197197
throw new Error('Failed to rebuild transaction');
198198
}

modules/sdk-coin-apt/src/lib/transactionBuilder/transactionBuilder.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,10 @@ export abstract class TransactionBuilder extends BaseTransactionBuilder {
2828
*/
2929
protected abstract get transactionType(): TransactionType;
3030

31+
getRebuiltTransaction(): Transaction {
32+
return this._transaction;
33+
}
34+
3135
getSequenceNumber(): number {
3236
return this.transaction.sequenceNumber;
3337
}

0 commit comments

Comments
 (0)