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

Add last per_hop padding #564

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@OrfeasLitos
Copy link
Contributor

OrfeasLitos commented Feb 2, 2019

Also clarify that payload := per_hop

* `outgoing_cltv_value`: set to the final expiry specified by the recipient
* `amt_to_forward`: set to the final amount specified by the recipient
* `padding`: set to 20 zero bytes

This comment has been minimized.

@rustyrussell

rustyrussell Feb 4, 2019

Collaborator

padding is only 12 bytes, though?

The short_channel_id field is left unspecified, but c-lightning sets it to 0.

This comment has been minimized.

@OrfeasLitos

OrfeasLitos Feb 4, 2019

Author Contributor

Fixed it

Add last per_hop padding
Also clarify that payload := `per_hop`

@OrfeasLitos OrfeasLitos force-pushed the OrfeasLitos:add-receiver-padding branch from 54d224f to 7b5648a Feb 4, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment