Re: [GROW] Working Group Last Call (WGLC) for draft-ietf-grow-bmp-peer-up (start 22/Jan/2024 end 6/Feb/2024)

Surely Maxence, thanks for your comment. I will update the draft with 
this note.

Paolo

On 25/1/24 07:53, Maxence Younsi wrote:
> Dear all,
> 
> 
> I support the adoption of draft-ietf-grow-bmp-peer-up.
> 
> A small detail I think could be changed is that the definition of a 
> string in Section 2 should specify that the length in the TLV's Length 
> field is in bytes and not a number of character ("whose length *in 
> bytes* is given by [...]" ). It is already specified in Section 3.3:
> 
>     *  Information Length (2 bytes): The length of the following
>        Information field, in bytes.
> 
> but not in Section 2:
> 
> 2.  String Definition
> 
>     A string TLV is a free-form sequence of UTF-8 characters whose length
>     is given by the TLV's Length field.  There is no requirement to
>     terminate the string with a null (or any other particular) character
>     -- the Length field gives its termination.
> 
> 
> 
> Kind regards,
> 
> Maxence.
> 
> 
> _______________________________________________
> GROW mailing list
> GROW@ietf.org
> https://www.ietf.org/mailman/listinfo/grow