-
Notifications
You must be signed in to change notification settings - Fork 44
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support different xud networks #781
Comments
And i believe we need to additionally separate per coin. bitcoin testnet, litecoin testnet, different eth testnets |
Suggestions:
If we’ll allow separation per currency, then xud nodes on the same network might use different underlying blockchains coins, no? I was thinking to allow to configure only the xud network:
The respective Let me know what you think. |
Actually a global
Sounds good to me! Maybe ropsten for eth testnet since this seems to be preferred by the raiden team. What do you think? @sangaman Others:
Why do we need that? Would you store all in the same node table and then just e.g. read the
Sounds good!
That doesn't sound like a very nice UX to me. Configurable yes, but not empty. There should always be some of our nodes in there. And we'll need to do some BD to also get nodes from other entities in there.
Sounds about right, but see answer below.
I'd prefer |
Yes, or use different tables. Main point is to have a separated p2p discovery for each network.
Yes we can make it similar to that.
For
Sounds fine. |
Definitely publicly available and used to p2p bootstrap other nodes. We might even have some bots maintaining (real orders) on there.
Ok, both fine! |
This expands the support for separate XU networks to differentiate between mainnet, testnet, simnet, and regtest. - Each node is now associated with a specific XU network - Mainnet & testnet seed nodes are empty for now Closes #781. BREAKING CHANGE: nodes database table & separate databases by network
At the moment
network
config property is default totestnet
, and its only usage is to derive thelnd
macaroon file path.This property usage need to be extended to support different
xud
networks (mainnet
/testnet
/simnet
/regnet
), where peers from different networks shouldn't be connected.mainnet
andtestnet
can refer to the underlying blockchains public networks, whilesimnet
can refer to thexud
controlled network. I'm not sure aboutregnet
.Each network will have a distinct value to be used in the wire protocol (following the values in bitcoin), both to identify the packet/peer origin network and as the socket stream delimiter (same as in bitcoin):
The text was updated successfully, but these errors were encountered: