-
Notifications
You must be signed in to change notification settings - Fork 105
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
Add support for core lightning watchtower TEoS (The Eye of Satoshi) - server and client #543
base: master
Are you sure you want to change the base?
Conversation
98f9500
to
e0eae6d
Compare
bb319fe
to
42626b6
Compare
93ad7ba
to
509f3b8
Compare
Hello @jonasnick,
The However, if Sergi recommends using the master branch, I can prepare a package overlay at least for nix-bitcoin. For anyone interested in getting this merged, please test it and feel free to provide a review. I'll be glad to address any issues. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
I started testing this now with CLN as a backend and the teos daemon on a separate host with a pruned bitcoind.
Works great so far with minimal configuration.
Still running great on my side. I would love to see this in a release. |
I'm planning to cut a release sooner rather than later, which will include all the changes pointed out in the PR description AFAICT. Is there anything else that'd needed? |
7dd92e4
to
0dd5fdb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm running the watchtower and plugin modules from this PR and its working well.
It would be nice to have a service that automatically registers with the tower and renews the subscription when needed. But we can add that later.
Also I opened a PR to add a flake to TEOS.
0dd5fdb
to
bd3f0d9
Compare
# Conflicts: # modules/clightning-plugins/teos-watchtower-plugin.nix
This MR adds basic support for rust-teos - package, module and clightning watchtower plugin.
Fixes: #540
TODO(s):