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

Lightning Specification Meeting 2022/08/15 #1017

Closed
14 of 25 tasks
t-bast opened this issue Aug 12, 2022 · 7 comments
Closed
14 of 25 tasks

Lightning Specification Meeting 2022/08/15 #1017

t-bast opened this issue Aug 12, 2022 · 7 comments

Comments

@t-bast
Copy link
Collaborator

t-bast commented Aug 12, 2022

The meeting will take place on Monday 2022/08/15 at 8pm UTC (5:30am Adelaide time) on Libera Chat IRC #lightning-dev. It is open to the public.

A video link is available for higher bandwidth communication: https://meet.jit.si/Lightning-Spec-Meeting

Pull Request Review

Waiting for interop

Long Term Updates

Backlog

The following are topics that we should discuss at some point, so if we have time to discuss them great, otherwise they slip to the next meeting.

@t-bast t-bast pinned this issue Aug 12, 2022
@Roasbeef
Copy link
Collaborator

Co-op close commentary: #1016 (comment)

@Roasbeef
Copy link
Collaborator

Route blinding commentary:

  • Originally: mug all errors, turn it into a different error (very opaque)
  • More recent ideas: ideally we can propagate well formed errors that are useful (mpp timeout, etc). Important that the error needs to be wrapped in the blinded key rather than your actual key
  • Now wrangling the wording in the spec, with the test vectors updated as well.
  • CL has updated code, works w/ the test vectors. Now working on doing an e2e test w/ blinded paths for payments
  • Future requests: want a persistent fake identity for the final hop as well? Something to be done in the future potentially.
  • Blinded payments now also compulsory for BOLT 12, you want to also supply one even if you don't need it. Gives you sort of mandatory payment secrets

@Roasbeef
Copy link
Collaborator

Offers commentary:

  • Right now you ignore certain numbered fields that store signatures, or are included in signatures
  • Some want to have two distinct streams: one stream for the things hashed, and another stream for the sigs
  • Might want to add some more distinct layers, some feel like it can be inefficient

@Roasbeef
Copy link
Collaborator

Roasbeef commented Aug 15, 2022

web sockets commentary (#891):

  • Host name merged in now. Turns out web sockets must always be HTTPs for browsers, so you need a cert, which means you need a host name.
  • Now seems we want a new explicit type, to indicate it's web sockets only, with maybe also a bit to set if TLS is required or not. So this would sort of copy over the DNS field information.
  • CL supports websockets w/o TLS (can't be accessed from a browser), cuz ppl use JS in weird places

@Roasbeef
Copy link
Collaborator

Roasbeef commented Aug 15, 2022

dual funding (#851):

  • CL uses anchor outputs always, eclair has implemented but doesn't always have anchors
  • related threads re splicing and pinning stuff above

splicing:

  • rule 3 for RBF is bad, messes up your bid dynamics
  • there're full rbf ppl now, but that just means you don't need to signal
  • RBF maximalists want to just destroy the whole thing and replace if you bid higher or not
  • new version of package RBF w/ a new tx version: policy: nVersion=3 and Package RBF bitcoin/bitcoin#25038

edit: eclair has already fully implemented dual funding

@niftynei
Copy link
Collaborator

CL uses anchor outputs always, eclair starting to implement but doesn't always have anchors

eclair has implemented fwiw! "Dual funding PRs are being merged to master (#2247, #2273, #2274, #2275)"

@t-bast
Copy link
Collaborator Author

t-bast commented Aug 16, 2022

Thanks for the summary @Roasbeef, much appreciated!

@t-bast t-bast unpinned this issue Aug 29, 2022
@t-bast t-bast closed this as completed Aug 29, 2022
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

No branches or pull requests

3 participants