[Teep] Re: draft-ietf-teep-protocol-24 telechat Artart review

[Hannes Tschofenig <[e-mail]>]

Thank you, Darrel, for the review.

We addressed your feedback as follows:

1. Section 13.5 / Table 6 range typo
Confirmed and corrected. The range is now 23-1023 (not 2-255) in the 
TEEP CBOR Label Registry table.

2. Section 13.6 / registry value vs CDDL representation
Good catch. After thinking about this a bit more I decided to remove the 
ciphersuite registry since it is not really necessary for 
interoperability. The values in the array are actually taken from an 
existing registry.

3. Section 4.6 / token text in Error message
Corrected the sentence to refer to the message the Error is in response 
to (instead of “Success”).

Thanks again for carefully reading the draft.

Ciao
Hannes


Am 22.02.2026 um 00:07 schrieb Darrel Miller via Datatracker:
> Document: draft-ietf-teep-protocol
> Title: Trusted Execution Environment Provisioning (TEEP) Protocol
> Reviewer: Darrel Miller
> Review result: Ready with Nits
>
> This is an ART area review.
>
> ## NITs
>
> Table 6 in section 13.5 has the following line:
>
>> 2-255 (Reserved for future use)
> I suspect that was intended to be 24-255.
>
> 13.6 assigns registry values 0 and 1 to the two cipher suites, but the CDDL in
> Section 8.1 defines them as arrays of operations ([[18, -19]] and [[18,-9]]),
> not as integer values. Are the integer values in the registry used anywhere on
> the wire, or are they only for registry identification?
>
> 4.6, token field description says: "It MUST match the value of the token
> parameter in the message the Success is in response to". This seems like it
> should say "Error" instead of "Success."
>
> That's all from me,
>
> Darrel Miller
>
>
>
> _______________________________________________
> TEEP mailing list -- [e-mail]
> To unsubscribe send an email to [e-mail]