search

tlswg / tls13-spec

TLS 1.3 Specification
563 stars 159 forks source link

Additional data #1162

Closed ekr closed 6 years ago

ekr commented 6 years ago

Previous PR plus some cleanup.

cjpatton commented 6 years ago

Looks good to me, for what it's worth :)

seanturner commented 6 years ago

ekr: please merge this so we can get a new version out for the IESG.

ekr commented 6 years ago

@kaduk: can you be more precise about the appendix D text? is the issue that we say "ignored" but then use it as additional_data?

kaduk commented 6 years ago 
The value of
   TLSCiphertext.legacy_record_version MAY be ignored, or MAY be
   validated to match the fixed constant value.
ekr commented 6 years ago

@kaduk : PTAL