Hi Luuk,

On 8/3/23 00:54, Paolo Lucente wrote:

Per section:

Sec 3:
- For indexed TLVs, the text is not explicit about whether the length
   field includes the 2 bytes for the index field, or describes the
   length of only the value.
   I don't see a benefit of one over the other, as long as the text is
   clear about it.

Section 3 does refer to Section 4.4 of RFC7854 where the length field is defined as "The length of the following Information field, in bytes.". Do you feel this is under defined?

You were referring to indexed TLVs specifically. You are right this was under defined and i received a similar comment today from another implementor.

Added text at this propo: https://github.com/paololucente/draft-ietf-grow-bmp-tlv/commit/54bf1efb4f1b1d62085df50e18c21734c2495ccd

Paolo

_______________________________________________
GROW mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/grow

Reply via email to