Skip to content

Conversation

ellemouton
Copy link
Member

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

Copy link
Contributor

@guggero guggero left a 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.

@ellemouton ellemouton force-pushed the macaroonService branch 3 times, most recently from b7f4372 to 8dfa0d2 Compare January 14, 2022 12:43
Copy link
Contributor

@guggero guggero left a 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!

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.
Copy link
Contributor

@positiveblue positiveblue left a 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! 🎉

@guggero guggero merged commit bc8537a into lightninglabs:master Jan 18, 2022
positiveblue pushed a commit to positiveblue/pool that referenced this pull request Oct 11, 2022
…l-itests

mod+itest: update to latest Pool version and add itest
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants