Skip to content

lnwire: Support upfront shutdown script feature#449

Closed
stevenroose wants to merge 1 commit into
lightningnetwork:masterfrom
stevenroose:upfront-shutdown-script
Closed

lnwire: Support upfront shutdown script feature#449
stevenroose wants to merge 1 commit into
lightningnetwork:masterfrom
stevenroose:upfront-shutdown-script

Conversation

@stevenroose
Copy link
Copy Markdown
Contributor

I noticed that this was part of the RFC, but not yet implemented here.

@stevenroose stevenroose force-pushed the upfront-shutdown-script branch from b3c8f28 to 5bd5d11 Compare December 3, 2017 01:42
@Roasbeef
Copy link
Copy Markdown
Member

Roasbeef commented Dec 9, 2017

As is, this PR is incomplete as it doesn't actually implement the logic to use the option if supported by the other party, or ensure that both sides commit to the same delivery script.

@stevenroose
Copy link
Copy Markdown
Contributor Author

I suppose you mean s/complete/incomplete? I know, it's an lnwire update, though. It doesn't make lnd use the feature flag yet, so it just makes it understand peers using the feature flag.

@stevenroose stevenroose force-pushed the upfront-shutdown-script branch from 5bd5d11 to 30807f0 Compare December 20, 2017 18:49
@meshcollider meshcollider added the wire protocol Encoding of messages for the communication between nodes label Mar 18, 2018
@stevenroose stevenroose force-pushed the upfront-shutdown-script branch from 30807f0 to 3a68a82 Compare April 5, 2018 12:55
@Roasbeef Roasbeef added this to the 0.5 milestone May 2, 2018
@Roasbeef Roasbeef added the P3 might get fixed, nice to have label Jul 10, 2018
@Roasbeef Roasbeef removed this from the 0.5 milestone Jul 11, 2018
@Roasbeef Roasbeef added this to the 0.6 milestone Dec 4, 2018
@Roasbeef
Copy link
Copy Markdown
Member

Roasbeef commented Dec 4, 2018

Needs a rebase!

@stevenroose stevenroose force-pushed the upfront-shutdown-script branch 2 times, most recently from 534968e to b4ba2b1 Compare December 5, 2018 00:10
@stevenroose
Copy link
Copy Markdown
Contributor Author

I tried rebasing, but I forgot about the original logic so I can't reliably review my own work :) So I trust your reviewing judgement!

@Roasbeef Roasbeef removed this from the 0.6 milestone Jan 16, 2019
@Roasbeef Roasbeef added this to the 0.9 milestone Oct 4, 2019
@carlaKC
Copy link
Copy Markdown
Collaborator

carlaKC commented Oct 31, 2019

Replaced with #3655, this commit is included in the new PR 😄

@carlaKC carlaKC closed this Oct 31, 2019
@joostjager joostjager removed the v0.9.0 label Nov 1, 2019
@joostjager joostjager removed this from the 0.9.0 milestone Nov 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P3 might get fixed, nice to have wire protocol Encoding of messages for the communication between nodes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants