[bisq-network/bisq] Prevent dust outputs from being created during withdraw from wallet (#4093)

James Cox notifications at github.com
Tue Mar 31 02:20:15 UTC 2020


@jmacxx commented on this pull request.



> @@ -330,8 +331,9 @@ private void onWithdraw() {
                     feeEstimationTransaction = walletService.getFeeEstimationTransactionForMultipleAddresses(fromAddresses, sendersAmount);
                 }
                 checkNotNull(feeEstimationTransaction, "feeEstimationTransaction must not be null");
-                Coin fee = feeEstimationTransaction.getFee();
-                sendersAmount = feeExcluded ? amountAsCoin.add(fee) : amountAsCoin;
+                Coin dust = getDust(feeEstimationTransaction);
+                Coin fee = feeEstimationTransaction.getFee().add(dust);
+                sendersAmount = feeExcluded ? amountAsCoin.add(fee) : amountAsCoin.add(dust);

@m52go @sqrrm your changes have been implemented.

![image](https://user-images.githubusercontent.com/47253594/77980063-eee2f400-72cb-11ea-8864-5c583f6664cf.png)


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/4093#discussion_r400603441
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20200330/79b1431d/attachment.html>


More information about the bisq-github mailing list