Skip to content

Add option to not create new macaroons - #127

Closed
orbitalturtle wants to merge 3 commits into
lightninglabs:lnd-14from
orbitalturtle:macaroon-stateless
Closed

orbitalturtle wants to merge 3 commits into
lightninglabs:lnd-14from
orbitalturtle:macaroon-stateless

Conversation

@orbitalturtle

@orbitalturtle orbitalturtle commented Jul 19, 2021 •

Copy link
Copy Markdown
Contributor

When using the Lightning Terminal in stateless remote mode (described here: lightninglabs/lightning-terminal#160), Faraday shouldn't create a new macaroon. This is a small change to stop macaroons from being created when macaroon data is passed to Faraday directly.

@orbitalturtle
orbitalturtle changed the base branch from master to lnd-14 September 26, 2021 06:18
…rminal

Export AllPermissions slice so that we can retrieve the permissions
required to run Faraday's operations. We need these permissions in
Lightning Terminal in order to bake a super macaroon that allows
access to Pool.
@orbitalturtle orbitalturtle changed the title When a macaroon string is passed in, don't create new macaroons Add option to not create new macaroons Sep 27, 2021
@guggero
guggero force-pushed the lnd-14 branch 2 times, most recently from 1de7153 to 2ec82ff Compare October 26, 2021 10:30
@guggero
guggero force-pushed the lnd-14 branch 2 times, most recently from a19b518 to 9f9e8a8 Compare November 3, 2021 18:24
@guggero
guggero deleted the branch lightninglabs:lnd-14 November 4, 2021 10:32
@guggero guggero closed this Nov 4, 2021
@guggero

guggero commented Nov 4, 2021

Copy link
Copy Markdown
Contributor

Same here, can be rebased onto master now. Sorry for the indirect/unintended close.

@guggero

guggero commented Nov 4, 2021

Copy link
Copy Markdown
Contributor

I'm going to take over those PRs so they can hopefully make it into the lnd 0.14.0 release of LiT. I hope you don't mind!

@guggero guggero mentioned this pull request Nov 4, 2021
2 tasks
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.

2 participants