<h3>Description</h3>
<p>The user first posted on the #general channel on Keybase on February 9th 2020 at 16h52m and both me and <a class="user-mention" data-hovercard-type="user" data-hovercard-url="/users/leo816/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/leo816">@leo816</a> responded within the minute.<br>
They claimed that their taker fee transaction couldn't be found on a bock explorer but that they had already sent the fiat to the other trader.</p>
<p>After talking with them I learned that they sent more than requested by Bisq causing an invalid trade.</p>
<p><strong>Current status</strong>: offer taker, btc buyer, sent fiat to offer maker, btc seller, without having a valid trade and therefore a valid multisig binding them.</p>
<h4>Version</h4>
<p>Bisq v0.9.5</p>
<h3>Steps to reproduce</h3>
<p>Take an offer and instead of the value requested in the Bisq software, send that plus a under the dust value to that address. So if it says that you should fund the address X with 0.012993 btc, send instead  0.012995 btc. Adding 200 sats.</p>
<h3>Expected behaviour</h3>
<p>A not "expected", but ideal behaviour would be: if amountSent to address X is superior to the amountRequired and the diffrence between amountSent and amountRequired is equal or inferior to the dust limit, then this difference should be added to the mining fees of the transaction. If the difference is superior, a third output should be created sending this amount to the taker.</p>
<h3>Actual behaviour</h3>
<p>The trading fee tx for the taker is created and splits the inputs into predetermined output amounts:<br>
output 1 - trading fee<br>
output 2 - security deposit + trade amount (if btc seller) that will be sent to the multisig<br>
Normally those would be the only outputs, but in this case an extra 200 sats are unaccounted for and instead Bisq creates a third output sending 200 sats to one of the taker's addresses. This creates an invalid transaction that can't be broadcast to the network due to the dust output.</p>
<p>The Bisq v1.2.5, where I ran the test, recognizes immediately. The user was using Bisq v.0.9.5</p>
<h3>Screenshots</h3>
<p>Image 01 - immediately after making the transaction from an external wallet to take the offer</p>
<p><a target="_blank" rel="noopener noreferrer" href="https://user-images.githubusercontent.com/44140017/74194740-cb97b300-4c51-11ea-96c1-2ba7b26b2a5e.png"><img src="https://user-images.githubusercontent.com/44140017/74194740-cb97b300-4c51-11ea-96c1-2ba7b26b2a5e.png" alt="bisq001" style="max-width:100%;"></a></p>
<p>Image 02 - after closing the first pop-up</p>
<p><a target="_blank" rel="noopener noreferrer" href="https://user-images.githubusercontent.com/44140017/74194744-ccc8e000-4c51-11ea-8ac7-b39ef2410d82.png"><img src="https://user-images.githubusercontent.com/44140017/74194744-ccc8e000-4c51-11ea-8ac7-b39ef2410d82.png" alt="bisq002" style="max-width:100%;"></a></p>
<p>Image 03 - after closing the second pop-up if we go to PORTFOLIO > OPEN TRADES we see this<br>
<a target="_blank" rel="noopener noreferrer" href="https://user-images.githubusercontent.com/44140017/74194746-ccc8e000-4c51-11ea-9ebe-c8df78de0f82.png"><img src="https://user-images.githubusercontent.com/44140017/74194746-ccc8e000-4c51-11ea-9ebe-c8df78de0f82.png" alt="bisq003" style="max-width:100%;"></a></p>
<p>Image 04 - after restarting the software and going to PORTFOLIO > OPEN TRADES we are presented with this<br>
<a target="_blank" rel="noopener noreferrer" href="https://user-images.githubusercontent.com/44140017/74194747-cd617680-4c51-11ea-9a8d-5b31bf65ce06.png"><img src="https://user-images.githubusercontent.com/44140017/74194747-cd617680-4c51-11ea-9a8d-5b31bf65ce06.png" alt="bisq004" style="max-width:100%;"></a></p>
<h4>Device or machine</h4>
<p>MAC OS</p>
<h4>Additional info</h4>
<p>I tried to run tests on the Regtest but my knowledge was limited. If I try to make a transaction inside Bitcoin Core with a dust output I'll get an error. But following the steps above, Bisq is able to create it and relay it through the network and finish the trade successfully. On my Bisq wallet I'll have a valid dust output.</p>
<p>I wasn't able to properly monitor the maker side of things due to low time and high fees but I'm looking forward to it to see if there are ways in which we can rectify mistakes like this.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/bisq-network/support/issues/337?email_source=notifications&email_token=AJFFTNUL2OYP2E5ZUHI6D2TRCHISFA5CNFSM4KSWY4W2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IMMWBQQ">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNVFQZ4PGTYTDYWXD2LRCHISFANCNFSM4KSWY4WQ">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AJFFTNQGNUVU725RFHS4HZLRCHISFA5CNFSM4KSWY4W2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IMMWBQQ.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/bisq-network/support/issues/337?email_source=notifications\u0026email_token=AJFFTNUL2OYP2E5ZUHI6D2TRCHISFA5CNFSM4KSWY4W2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IMMWBQQ",
"url": "https://github.com/bisq-network/support/issues/337?email_source=notifications\u0026email_token=AJFFTNUL2OYP2E5ZUHI6D2TRCHISFA5CNFSM4KSWY4W2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IMMWBQQ",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>