search

revault / revaultd

The "wallet" daemon for participants under the Revault architecture
BSD 3-Clause "New" or "Revised" License
49 stars 21 forks source link

bitcoind: sanity check descriptors were imported #416

Open darosior opened 2 years ago

darosior commented 2 years ago

At startup, if a watchonly wallet isn't present we'll create and import the descriptors. If one is, we'll just assume the descriptors were successfully imported in it. It would be more robust to sanity check that descriptors were indeed imported: https://github.com/revault/revaultd/blob/e72146ecbf6c1fb8e498aeb0bccbb3f82a6126c7/src/bitcoind/poller.rs#L1715-L1717

darosior commented 2 years ago

lol i had a 2 years old issue for this already: https://github.com/revault/revaultd/issues/35. Keeping this one as it's more detailed.