[bisq-network/bisq] [DRAFT] Create new ping API method (#4502)

chimp1984 notifications at github.com
Wed Sep 9 02:56:34 UTC 2020


>  This may be a simple solution to the 'p2p not ready' problem affecting daemons.

I doubt that this is a feasible approach. There are several state in the p2p and wallet which need be treated differently depending on use case.
P2p:
- getInitialDataResponse -> can show offers but is not listening to network as no hidden service published yet.
- HS published -> ok not we are part of the network
- UpdatedDataResonse -> process the maybe missed data in the duration between initiaal response and HS published

Wallet:
- initialize -> wallet is set up
- sufficient peers connected -> it is safe to broadcast txs
- block download complete -> all synced, not needed to use the app but some txs might not show confirmed or we might miss balance if we received funds

So you see this complexity cannot be simulated with a single method. I would not suggest to use ping for more then simple connectivity tests. 

-- 
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/4502#issuecomment-689268246
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20200908/c998f415/attachment.html>


More information about the bisq-github mailing list