From e50339d44bbb1b89bcc7054666f1dbcfd59d175a Mon Sep 17 00:00:00 2001 From: Olaoluwa Osuntokun Date: Thu, 13 Jun 2019 13:54:33 -0700 Subject: [PATCH] lnwallet/btcwallet: also include raw tx hex for unconf txns In this commit, we patch a small bug in the newly added raw tx hex field for ListTransactions. We now ensure that we also set the raw tx hex field for unconfirmed transactions. --- lnwallet/btcwallet/btcwallet.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/lnwallet/btcwallet/btcwallet.go b/lnwallet/btcwallet/btcwallet.go index 522aac6d..a3d262d2 100644 --- a/lnwallet/btcwallet/btcwallet.go +++ b/lnwallet/btcwallet/btcwallet.go @@ -550,6 +550,7 @@ func minedTransactionsToDetails( func unminedTransactionsToDetail( summary base.TransactionSummary, ) (*lnwallet.TransactionDetail, error) { + wireTx := &wire.MsgTx{} txReader := bytes.NewReader(summary.Transaction) @@ -561,6 +562,7 @@ func unminedTransactionsToDetail( Hash: *summary.Hash, TotalFees: int64(summary.Fee), Timestamp: summary.Timestamp, + RawTx: summary.Transaction, } balanceDelta, err := extractBalanceDelta(summary, wireTx) @@ -738,8 +740,8 @@ func (b *BtcWallet) SubscribeTransactions() (lnwallet.TransactionSubscription, e return txClient, nil } -// IsSynced returns a boolean indicating if from the PoV of the wallet, -// it has fully synced to the current best block in the main chain. +// IsSynced returns a boolean indicating if from the PoV of the wallet, it has +// fully synced to the current best block in the main chain. // // This is a part of the WalletController interface. func (b *BtcWallet) IsSynced() (bool, int64, error) {