-
Notifications
You must be signed in to change notification settings - Fork 47
multi: use lndclient MacaroonService #336
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
Conversation
fbdc158
to
fbcd696
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.
Yayy, deleted code 🎉
LGTM apart from the database password migration issue.
b7f4372
to
8dfa0d2
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, nits only!
8dfa0d2
to
0a99f9b
Compare
Since the code for creating and using a macaroon service is the same for multiple projects (pool, loop, litd etc), the code has been unified in lndclient. So this commit removes the macaroon service code and instead uses the lndclient code.
Rename createDefaultMacaroonFile to withMacaroonService since this is now a more appropriate name.
0a99f9b
to
c4d43e7
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.
Thanks for this @ellemouton! 🎉
…l-itests mod+itest: update to latest Pool version and add itest
Since the code for creating and using a macaroon service is the same for
multiple projects (pool, loop, litd etc), the code has been unified in
lndclient. So this commit removes the macaroon service code and instead
uses the lndclient code.
Depends on lightninglabs/lndclient#86