Skip to content
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

static channel backup is insufficient #44

Open
neocarmack opened this issue Sep 22, 2021 · 1 comment
Open

static channel backup is insufficient #44

neocarmack opened this issue Sep 22, 2021 · 1 comment
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@neocarmack
Copy link
Member

This is from an auditor:

As to https://omnilaboratory.github.io/obd/#/safety-guidelines
Since you are using LND as base instead of C-Lightning v(^^) users need to ensure that their storage is reliable.
C-Lightning includes a hook that allows storage to be replicated by any mechanism, see >>https://lightning.readthedocs.io/BACKUP.html for the options users have in practice.

To my knowledge, LND has not corresponding hook.
Static Channel Backups may be insufficient; if Omni requires the LND-derived daemon to keep track of additional metadata to >>ensure correctness, then the data in the Static Channel Backups may be insufficient (depends on how you structure that data).
And you may need to modify the SCB mechanism to include the extra data required by Omni.

My suggestion is to urge your users (or at least the ones with significant amounts of stored value), in the safety guidelines, to >>use some kind of RAID system, such as ZFS (mirror or any RAID-Z), mdadm (RAID1 mode), or BTRFS (raid1, raid1c3, or >>raid1c4) and to monitor the storage continuously so that any storage failure is detected.

This is in addition to any SCB they might regularly make (and you need to augment the SCBs with Omni-specific data as well).
See the above-linked BACKUP.html file for C-Lightning for how I suggest you present the safety guidelines.

@neocarmack
Copy link
Member Author

Although omnibolt is not based on lnd, using RAID system for safe storage is good advise. We will add it to document.

@neocarmack neocarmack self-assigned this Sep 22, 2021
@neocarmack neocarmack added the documentation Improvements or additions to documentation label Sep 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

1 participant