-
-
Notifications
You must be signed in to change notification settings - Fork 9.9k
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
High level overview of QUIC Implementation #18406
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It isn't entirely obvious which way some of the arrows in the diagram at going. Some separation of the end points of multiple lines would be used here.
e.g. the "TLS Handshake Record Layer" is great, "Path and Conn Demux" is difficult. It isn't possible to tell if it is a one way or two way arrow between "Timer and Event Queue" and "ACK Handling and Loss Detector"?
Done. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Datagram BIO arrows (at the top) are indistinct but intelligible.
Good enough for I guess.
Should OTC approve this (at least by consensus?) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This pull request is ready to merge |
OTC approved it by consensus on the last meeting. Merged to master branch. Thank you for the reviews. |
Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com> Reviewed-by: Paul Dale <pauli@openssl.org> (Merged from #18406)
Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com> Reviewed-by: Paul Dale <pauli@openssl.org> (Merged from openssl#18406)
This provides a high level overview of OpenSSL QUIC Implementation building blocks.
Checklist