lightning-onion icon indicating copy to clipboard operation
lightning-onion copied to clipboard

multi: Route blinding

Open ellemouton opened this issue 2 years ago • 5 comments

This PR adds the changes that higher layers will need in order to take advantage of route blinding as defined in the spec proposal

Note that callers of this code will need to know how to extract the next_blinding_point (if the caller is the introduction node in the route) and next_blinding_override points from the onion packet payloads.

ellemouton avatar Jun 30 '22 20:06 ellemouton

Also a meta thing that I noticed while using this PR. When this is merged I think that it would make sense to make two tags: one on the blinded route commit, and one on the final merge/full PR.

Reason being that code that imports this change and lnd requires lnd to be upgraded to use the new arg in ProcessOnionPacket (which requires lnd to update its dependency as well). Adding an extra tag to easily allow code to just point to the blinded paths stuff could be useful in avoiding modhell/ requiring lnd to update its dep prematurely. Just a note for future-selves.

carlaKC avatar Jul 05 '22 11:07 carlaKC

making a note here for myself to update the PR to point to the latest state of the spec PR & its test vectors

ellemouton avatar Aug 04 '22 08:08 ellemouton

ah ok, looks like the updated test vectors still pass with the existing code. I think the main changes were related to the tlv fields/types which does not affect this pr since pr assumes the tlvs are already encoded. Will double check this later though

ellemouton avatar Aug 04 '22 08:08 ellemouton

@ellemouton, remember to re-request review from reviewers when ready

lightninglabs-deploy avatar Aug 11 '22 09:08 lightninglabs-deploy

!lightninglabs-deploy mute

ellemouton avatar Aug 11 '22 09:08 ellemouton

Just needs rebase on ca23184 - when I import this to LND now some tests fail.

Rebase-beg when you have a minute!

carlaKC avatar Jan 25 '23 15:01 carlaKC

done! cc @carlaKC

ellemouton avatar Jan 26 '23 07:01 ellemouton