search

RustCrypto / formats

Cryptography-related format encoders/decoders: DER, PEM, PKCS, PKIX
235 stars 125 forks source link

tls_codec: implement `U24` #1284

Closed tnytown closed 8 months ago

tnytown commented 8 months ago

Resolves #1277.

tnytown commented 8 months ago

CC @franziskuskiefer :)

tnytown commented 8 months ago

I quickly checked RFC 9000 but it doesn't seem to allow u24 for variable length encodings but maybe double check that we shouldn't add it for variable length encodings too.

@franziskuskiefer I'm not too familiar with the variable length encodings defined by RFC 9000. I also did a quick read over https://datatracker.ietf.org/doc/html/rfc9000#name-variable-length-integer-enc, and I agree that it doesn't seem to allow for 3-byte encoded values :)

Is there anything else I can do to get this ready to merge?