Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

handling of KeyUpdate in other epochs are specified in RFC 8446, do not override #4412

Merged
merged 3 commits into from
Dec 8, 2020
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion draft-ietf-quic-tls.md
Original file line number Diff line number Diff line change
Expand Up @@ -1424,7 +1424,8 @@ without needing to receive the first packet that triggered the change. An
endpoint that notices a changed Key Phase bit updates keys and decrypts the
packet that contains the changed value.

This mechanism replaces the TLS KeyUpdate message. Endpoints MUST NOT send a
This mechanism replaces the key update mechanism of TLS, which relies on
KeyUpdate messages sent using 1-RTT encryption keys. Endpoints MUST NOT send a
TLS KeyUpdate message. Endpoints MUST treat the receipt of a TLS KeyUpdate
message in a 1-RTT packet as a connection error of type 0x10a, equivalent to a
fatal TLS alert of unexpected_message (see {{tls-errors}}).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IIUC, KeyUpdate is forbidden by the the TLS spec, right? Can we add a sentence here that states taht?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PTAL at b33c487. With the change, I think the text now aligns better with what we state for EOED (section 8.3).

martinthomson marked this conversation as resolved.
Show resolved Hide resolved
Expand Down