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

BOLT 1: Add a TLV stream extension to existing messages. #630

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
1 participant
@t-bast
Copy link
Collaborator

commented Jul 9, 2019

The spec already prepared a hook to add additional information to existing
messages (additional bytes at the end of a message must be ignored).

Since we're using TLV in many places, it would make sense to use that optional
additional space at the end of each message to allow an optional tlv stream.

We keep the "additional bytes must be ignored" rule for what happens after that
tlv stream, giving us extra flexibility to add another type of optional content
to existing messages later if needed.

@t-bast t-bast force-pushed the b01-trailing-byte-tlv-stream branch from f7b816e to a2b8bf5 Jul 9, 2019

BOLT 1: Add a TLV stream extension to existing messages.
The spec already prepared a hook to add additional information to existing
messages (additional bytes at the end of a message must be ignored).

Since we're using TLV in many places, it would make sense to use that optional
additional space at the end of each message to allow an optional tlv stream.

We keep the "additional bytes must be ignored" rule for what happens after that
tlv stream, giving us extra flexibility to add another type of optional content
to existing messages later if needed.

@t-bast t-bast force-pushed the b01-trailing-byte-tlv-stream branch from 0968bdc to 98c3640 Jul 15, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.