search

ietf-wg-idr / draft-ietf-idr-5g-edge-service-metadata

Editing for the 5G Service Metadata
0 stars 2 forks source link

TLV fields need to discuss lengths #35

Closed jhaas-pfrc closed 1 month ago

jhaas-pfrc commented 1 month ago

The document now consistently uses TLV format throughout the whole document. The length fields for a given TLV don't describe their expected lengths. This is necessary for validation checks.

Most TLVs are fixed lengths. For example, the Site-Prefere-Index TLV has 5 bytes of value. This means that the length documentation should say "should be 5".

Update the length descriptions throughout the document.

lindadunbar commented 1 month ago

That is easy. Will add the following statement for all the Length field in the next Push:

For the Site-Preference-Index (or X) Sub-Type, the length should be set to 5 (or Y)

jhaas-pfrc commented 1 month ago

This appears to be in the incoming commit.