Skip to content
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

proposal: Added Funding Timeout Recovery proposal (KEYOLO variant) #866

Closed
wants to merge 1 commit into from

Conversation

cdecker
Copy link
Collaborator

@cdecker cdecker commented Apr 26, 2021

This is the counter-proposal to #854, using the more flexible, but
also potentially more dangerous direct private key exchange, rather
than sending over a sighash_none signature for a close transaction.

It was proposed due to the observation that, while not yet required by
todays spec, all implementations decided to use hardened derivation
for the channel funding_privkey, making this a safe option.

Please continue discussing tradeoffs on #854 so we can keep them in
one coherent place, wording recommendations on this PR are still
welcome.

The two proposals are mutually exclusive, so we need to hammer out the
differences in the PRs :-)

Closes #854

This is the counter-proposal to #854, using the more flexible, but
also potentially more dangerous direct private key exchange, rather
than sending over a `sighash_none` signature for a close transaction.

It was proposed due to the observation that, while not yet required by
todays spec, all implementations decided to use hardened derivation
for the channel `funding_privkey`, making this a safe option.
@niftynei
Copy link
Collaborator

niftynei commented Jun 7, 2021

<BlueMatt> didnt we discuss this last time or two times ago and decide against the keyolo variant (after having been weakly in favor the week before)?
<BlueMatt> i had thought there was rough consensus against keyolo

Closing this as per June 7, 2020 IRC meeting in #lightning-dev (libera)

@niftynei niftynei closed this Jun 7, 2021
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