dagurval / electrum-cash-protocol

Electrum Cash protocol reference
MIT License
9 stars 6 forks source link

Clarify the caveat of the missing `hosts` key from features #4

Closed cculianu closed 4 years ago

cculianu commented 4 years ago

Added language to explain that if the hosts key is missing, other servers will not peer with the server.

Both ElectrumX/ElectronX and Fulcrum require this key to exist otherwise they will not peer with the server.

Stated another way: The absence of this key is an indication that the server does not wish to peer.

So, I added language to clarify that to the spec.

dagurval commented 4 years ago

The absence of this key is an indication that the server does not wish to peer.

Nice, thanks for documenting that hidden implied protocol feature.

cculianu commented 4 years ago

Maybe I should add that to the language in that paragraph? What do you think? The way it's written it sounds like a "punishment" that the key is missing -- then it's also an "opportunity" to not be too public as a server... what do you think?

dagurval commented 4 years ago

Agreed, that would be even better

cculianu commented 4 years ago

Ok, I'll do that now.