-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
multi: add tower config defaults #7771
multi: add tower config defaults #7771
Conversation
0925c6e
to
b1ada8c
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.
Very nice, LGTM 🎉
In this commit, a `DefaultWtClientCfg` function is added which populates some default values for the `WtClient` struct. This is then used to populate the wtclient defaults in the main LND config struct.
8195b0f
to
d3d7657
Compare
In this commit, a `DefaultWatchtowerCfg` function is added which is used to construct a default `lncfg.Watchtower` struct. This is then used to populate the default watchtower config in the main LND config struct.
This commit removes the `PrivateTowerURIs` member from the `WtClient` config struct. This field has been deprecated since v0.8.0-beta and currently, LND would fail to start if the field was specified.
d3d7657
to
7484910
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.
LGTM 🎉
Concept Ack |
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.
LGTM🏖
Add
DefaultWtclientCfg
andDefaultWatchtowerConfig
functions that populatedefaults for the
lncfg.Wtclient
andlncfg.Watchtower
config structs. These arethen used in the main LND config struct so that the defaults are shown to a user in
the
lnd --help
command.This PR also removes the deprecated
PrivateTowerURIs
member from theWtclient
config struct since it has been deprecated since
v0.8.0-beta
and currently LND wouldfail to start if a user specified it.
Fixes #7769