Option #2 seems fine to me.

Russ


On Oct 19, 2015, at 12:28 PM, Eric Rescorla wrote:

> Folks,
> 
> https://github.com/tlswg/tls13-spec/issues/278
> 
> The additional data field presently includes the version:
> 
>       additional_data = seq_num + TLSPlaintext.record_version
> 
> However, TLSPlaintext.record_version is now always {3, 1}, so
> this is redundant. There seem to be two primary options here:
> 
>      1. Don't MAC the version at all.
>      2. MAC the negotiated version (which should be clear at
>         this point).
> 
> I could go either way on this (slightly leaning towards #2) but
> the current thing seems silly.
> 
> -Ekr
> 
> _______________________________________________
> TLS mailing list
> TLS@ietf.org
> https://www.ietf.org/mailman/listinfo/tls

_______________________________________________
TLS mailing list
TLS@ietf.org
https://www.ietf.org/mailman/listinfo/tls

Reply via email to