[bisq-network/bisq] Prevent trading when not connected to the Bitcoin network (#5325)

jmacxx notifications at github.com
Wed Mar 17 22:04:46 CET 2021


It is currently possible to create and take offers while not connected to the bitcoin network.

This should have been fixed by https://github.com/bisq-network/bisq/pull/4764 but that PR only considered the scenario where chainheight from the DAO was stale.  

The fix is to also handle the other part of the equation, when the **chainheight of bitcoin network peers is stale**.  



---

Instances of this problem have been experienced with offers that have transactions not found by blockexplorers, as described in https://github.com/bisq-network/bisq/issues/5151 and https://github.com/bisq-network/bisq/issues/3305.  


---


**To reproduce the issue:**

- Start Bisq, specifying a bitcoin node address that is not running a node.  (For regtest see `DEFAULT_REGTEST_HOST`)
- Bisq will start up and display "Connecting to Bitcoin network" in the footer bar.
- Create offers, take offers.  No error messages are displayed.


**To verify the fix:**

- Start Bisq, specifying a bitcoin node address that is not running a node.  (For regtest see `DEFAULT_REGTEST_HOST`)
- Bisq will start up and display "Connecting to Bitcoin network" in the footer bar.
- Attempt to create offers, take offers.  You will be informed as that Bisq is not in sync with the network.

![image](https://user-images.githubusercontent.com/47253594/111535603-4aa1c980-8737-11eb-8b9f-c4ac4b06eaf8.png)


Network Settings:

![image](https://user-images.githubusercontent.com/47253594/111537770-d9174a80-8739-11eb-8b9e-40f938f33487.png)

You can view, comment on, or merge this pull request online at:

  https://github.com/bisq-network/bisq/pull/5325

-- Commit Summary --

  * handle case where bitcoind chain height is stale

-- File Changes --

    M core/src/main/java/bisq/core/btc/setup/WalletsSetup.java (2)
    M core/src/main/resources/i18n/displayStrings.properties (2)

-- Patch Links --

https://github.com/bisq-network/bisq/pull/5325.patch
https://github.com/bisq-network/bisq/pull/5325.diff

-- 
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/5325
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20210317/65c39ac0/attachment-0001.htm>


More information about the bisq-github mailing list