revault / practical-revault

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

Two-round watchtower opinion polling for fund managers #38

Closed darosior closed 3 years ago

darosior commented 3 years ago

This introduces a way to poll watchtowers without a fully signed spend transaction to:

This also includes two drive-by fixups needed for this.

Fixes https://github.com/re-vault/practical-revault/issues/4

darosior commented 3 years ago

Update diagrams flow Add the unsigned spend tx to the opinion

darosior commented 3 years ago

@JSwambo plz sir me wanna merge :(

darosior commented 3 years ago

How actually i didn't edit it since last time! Please disregard will do soon

darosior commented 3 years ago

Done, this is now ready for review

darosior commented 3 years ago

Ok, updated the first nits, the pubkey inclusion deserves its own pr imo

darosior commented 3 years ago

As discussed yesterday, this introduces a ""reliance"" on the sync server for authenticating the managers. I think it's fine, so was for reference.