<p><a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=23560607" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/sqrrm">@sqrrm</a> Ah ok, that explains why we don't encounted any issues as all BSQ txs are broadcasted by the BSQ wallet and cloned at that step. Also the trade fee txs are cloning the tx for the btc wallet.<br>
So the only use cases are invalid ones (managing to send BTC from Bisq to the BSQ wallet outside the UI).</p>
<p>I agree we should fix that even if it is not a real risk atm. But would prefer to make it at a later point to not add risk now and focus on other more concrete issues.</p>
<p><a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=23560607" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/sqrrm">@sqrrm</a> Is it ok to close that PR?</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/bitcoinj/pull/18#issuecomment-451755463">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AkpZtuzHDmtOguDRGe5WCSClAP8qhMInks5vAiatgaJpZM4Zgibw">mute the thread</a>.<img src="https://github.com/notifications/beacon/AkpZthDVd8ZR9DrUje00065c55-ob3rDks5vAiatgaJpZM4Zgibw.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/bisq-network/bitcoinj","title":"bisq-network/bitcoinj","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/bisq-network/bitcoinj"}},"updates":{"snippets":[{"icon":"PERSON","message":"@ManfredKarrer in #18: @sqrrm Ah ok, that explains why we don't encounted any issues as all BSQ txs are broadcasted by the BSQ wallet and cloned at that step. Also the trade fee txs are cloning the tx for the btc wallet.\r\nSo the only use cases are invalid ones (managing to send BTC from Bisq to the BSQ wallet outside the UI). \r\n\r\nI agree we should fix that even if it is not a real risk atm. But would prefer to make it at a later point to not add risk now and focus on other more concrete issues. \r\n\r\n@sqrrm Is it ok to close that PR?"}],"action":{"name":"View Pull Request","url":"https://github.com/bisq-network/bitcoinj/pull/18#issuecomment-451755463"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/bisq-network/bitcoinj/pull/18#issuecomment-451755463",
"url": "https://github.com/bisq-network/bitcoinj/pull/18#issuecomment-451755463",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>