algorand / algorand/java-algorand-sdk
TransactionId empty if SignedTransaction initialized by constructor @JsonCreator
- 主要語言
- Java
- 星號
- 73
- 分支
- 71
- 平均合併
- 41 分鐘
- 30 天內合併 PR
- 2
描述
## Problem
The SignedTransaction created by deserialization JSON (@JSONCreator) doesn't fill the attribute transactionId.
https://github.com/algorand/java-algorand-sdk/blob/develop/src/main/java/com/algorand/algosdk/transaction/SignedTransaction.java#L80
So with Atomic Composer if I put a TxnSigner as a external request that works with deserialization of SignedTransaction, It will not work cause the transactionId is emtpy.
https://github.com/algorand/java-algorand-sdk/blob/develop/src/main/java/com/algorand/algosdk/transaction/AtomicTransactionComposer.java#L188
## Solution
We could add the transactionId into the constructor through@JSONCreator or other options proposed by barnji:
instead of using the txid directly from signed transaction object, use signedtransaction.tx.getTxId() or w/e its called in the atc.
it'd be redundant hashing possibly but safer in the case of a enc/dec stxn
## Dependencies
No dependencies
## Urgency
Low Urgency, easy to implement a workaround
貢獻指南
這個儲存庫沒有索引到貢獻指南
評估
這個 Issue 還沒有評估資料。