<p>This follows after a discussion with <a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=1145132" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/oscarguindzberg">@oscarguindzberg</a> about his finding of the tx broadcast bug.<br>
It will require more time to get the proper bugfix for the underlying problem (tx confidence handling in btcj). The issues only happens at BSQ txs as there the complete methods is called before the final tx is created. We discussed different intermediate fixes but decided to not add any risk and just reduce the timeout so that it does not cause issues on the trade protocol as well to not force a too long delay at proposal or blind vote publishings.<br>
This happen only if you are connected to more then 1 btc peer) not on local regtest or dao_regtest....</p>
<p>The tx broadcast itself has no bug, it is just that we don't hear back because the confidence object has changed and the success handler depends on that. So changing to a rather short time of 5 seconds is justified. The trade protocol has 1 minute timeout and to lose in the trade fee tx publishing too much time on that would risk that the trade protocol cause an timeout error. This would be only the case when BSQ is used as trade fee.</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/bisq/pull/2657#issuecomment-480407803">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AkpZtspFxaj6relxiGmt9zPmsSMgMsq4ks5vd66OgaJpZM4cfo7a">mute the thread</a>.<img src="https://github.com/notifications/beacon/AkpZtiD06DRKdFmsx4e1g_DsSZOT1HV6ks5vd66OgaJpZM4cfo7a.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/bisq","title":"bisq-network/bisq","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/bisq"}},"updates":{"snippets":[{"icon":"PERSON","message":"@ManfredKarrer in #2657: This follows after a discussion with @oscarguindzberg about his finding of the tx broadcast bug.\r\nIt will require more time to get the proper bugfix for the underlying problem (tx confidence handling in btcj). The issues only happens at BSQ txs as there the complete methods is called before the final tx is created. We discussed different intermediate fixes but decided to not add any risk and just reduce the timeout so that it does not cause issues on the trade protocol as well to not force a too long delay at proposal or blind vote publishings.\r\nThis happen only if you are connected to more then 1 btc peer) not on local regtest or dao_regtest....\r\n\r\nThe tx broadcast itself has no bug, it is just that we don't hear back because the confidence object has changed and the success handler depends on that. So changing to a rather short time of 5 seconds is justified. The trade protocol has 1 minute timeout and to lose in the trade fee tx publishing too much time on that would risk that the trade protocol cause an timeout error. This would be only the case when BSQ is used as trade fee."}],"action":{"name":"View Pull Request","url":"https://github.com/bisq-network/bisq/pull/2657#issuecomment-480407803"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/bisq-network/bisq/pull/2657#issuecomment-480407803",
"url": "https://github.com/bisq-network/bisq/pull/2657#issuecomment-480407803",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>