search

revault / practical-revault

Version 0 specifications for a Revault deployment
Creative Commons Attribution 4.0 International
33 stars 9 forks source link

messages: remove watchtower opinion polling #65

Closed darosior closed 3 years ago

darosior commented 3 years ago

This is sooooooo much cleaner without this complexity. Whose idea was it already?

darosior commented 3 years ago

Lol guys i dropped half the spending process doc.. Arg we should have catched it! :)

darosior commented 3 years ago

delay: yea, it would make sense to make it configurable. I think a fraction of the CSV delay would be a sensible default (2%? With a 12 blocks CSV it's (12 - 1) / 10 = 2.2 seconds, with -1 accounting for the worst case event of a block being found just after our poll).

re fallback coordinator: definitely.

‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐ Le lundi, mars 15, 2021 2:06 PM, Kevin Loaec @.***> a écrit :

@kloaec commented on this pull request.

Sorry for the late ack, but yes, much cleaner.

We need some kind of "delay" between asking the coordinator and triggering the cancel if no or wrong reply. Short but not stupidly short. Also might be a config option?

Also a fallback coordinator really makes sense here, if first one isn't responding (especially for external WT like the Revault corp ones.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or unsubscribe.