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

Bolt4: clarify packet forwarding blinding #1012

Conversation

johncantrell97
Copy link

Currently says the blinding is done using the processing node's public key which is not correct. Instead of trying to explain how blinding works again, just link to the section that describes it.

Currently says the blinding is done using the processing node's public key which is not correct.  Instead of trying to explain how blinding works again, just link to the section that describes it.
@johncantrell97 johncantrell97 changed the title clarify packet forwarding blinding Bolt4: clarify packet forwarding blinding Jul 26, 2022
@@ -679,8 +679,7 @@ A special `hmac` value of 32 `0x00`-bytes indicates that the currently processin

If the HMAC does not indicate route termination, and if the next hop is a peer of the
processing node; then the new packet is assembled. Packet assembly is accomplished
by blinding the ephemeral key with the processing node's public key, along with the
shared secret, and by serializing the `hop_payloads`.
by (blinding the ephemeral key)[#blinding-ephemeral-keys], and by serializing the `hop_payloads`.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't correctly display the link, this should work:

Suggested change
by (blinding the ephemeral key)[#blinding-ephemeral-keys], and by serializing the `hop_payloads`.
by [blinding the ephemeral key](#blinding-ephemeral-keys), and by serializing the `hop_payloads`.

@t-bast
Copy link
Collaborator

t-bast commented Sep 18, 2024

I believe this is better since #1181 and #1182, so closing this conflicting PR. Feel free to re-open if more clarification is needed.

@t-bast t-bast closed this Sep 18, 2024
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