Skip to content

Subtype TLV fixes!#1

Open
niftynei wants to merge 10 commits intorustyrussell:guilt/testsfrom
niftynei:nifty/subtype-tlv-fixes
Open

Subtype TLV fixes!#1
niftynei wants to merge 10 commits intorustyrussell:guilt/testsfrom
niftynei:nifty/subtype-tlv-fixes

Conversation

@niftynei
Copy link

@niftynei niftynei commented Aug 3, 2019

Patches for TLV's and subtypes (WIP!)

rustyrussell and others added 10 commits July 24, 2019 11:32
I initially used JSON, but it was not human-writable.  The simple
DSL is much more human-writable, and readable.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Works with current c-lightning master branch, as long as it's built with
--enable-developer.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Very simple successful open.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Does a simple dust HTLC and a non-dust HTLC.  Both fail.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
This helps for testing that we handle it correctly.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Preliminary parsing for subtypes. Probably need something on the
unpacking side also.
rustyrussell added a commit that referenced this pull request May 18, 2020
`u8` isn't a type; see BOLT #1 "Fundamental Types".

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
rustyrussell added a commit that referenced this pull request Aug 3, 2020
* Rename all the 'varint' to 'bigsize'.

Having both is confusing; we chose the name bigsize, so use it
explicitly.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

* BOLT 7: use `byte` instead of `u8`.

`u8` isn't a type; see BOLT #1 "Fundamental Types".

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>

* BOLT 1: promote bigsize to a Fundamental Type.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
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