Maybe a minor nit, but shouldn't be

           PartyInfo = (
               ? nonce : bstr / int,
               ? identity : bstr,
               ? other : bstr,
           )

better be done as

           PartyInfo = (
               nonce : bstr / int / nil,
               identity : bstr / nil,
               ? other : bstr,
           )

(This is used in an array context, so any non-final optionals create a
potential for mixup.)

While the attack surface of any confusion that can be created here is
rather small, I don't think we have to "save those bytes".

Grüße, Carsten

PS.: Yes, this could also be done as:


           PartyInfo = (
               nonce : bstr / int / nil,
               ? (identity : bstr / nil,
                  ? other : bstr),
           )

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

Reply via email to