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

Clarify labels example in -tls Appendix A.1. #4403

Merged
merged 1 commit into from Nov 20, 2020
Merged

Clarify labels example in -tls Appendix A.1. #4403

merged 1 commit into from Nov 20, 2020

Conversation

PaulDance
Copy link
Contributor

Hello again!

We are currently in an effort of implementing QUIC in the Scapy packet manipulation library. While doing so, we've had a few moments of doubts as to what was the intention or simply the true meaning behind various parts of the specification. One of them was the first sentence of the appendix A.1. of the QUIC-TLS draft: it lead us to think that the results listed below were the output of the HKDF-Expand-label function instead of the labels built during its execution and then given to HKDF-Expand to obtain the final result.

This PR thus proposes a small change in order to remove any possible ambiguity from that first sentence. If you have any remark or suggestion regarding typos, grammar, wording or additional clarifications required, please let us know.

Hope it helps.
Cheers,
Paul.

Remove any possible ambiguity from the first sentence of the A.1.
sub-section.

Signed-off-by: Paul Mabileau <paulmabileau@hotmail.fr>
@martinthomson martinthomson added -tls editorial An issue that does not affect the design of the protocol; does not require consensus. labels Nov 20, 2020
Copy link
Member

@martinthomson martinthomson left a comment

Choose a reason for hiding this comment

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

Thanks Paul. I am not able to fix the build error here, because I only have a phone right now, but this text is good.

@martinthomson
Copy link
Member

Ok, that is another tools.ietf.org failure.

@martinthomson martinthomson merged commit 55ec7c0 into quicwg:master Nov 20, 2020
@PaulDance PaulDance deleted the clarify-labels-example branch November 20, 2020 23:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-tls editorial An issue that does not affect the design of the protocol; does not require consensus.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants