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

Add max_early_data_size to TicketEarlyDataInfo #674

Closed
wants to merge 1 commit into from
Closed

Add max_early_data_size to TicketEarlyDataInfo #674

wants to merge 1 commit into from

Conversation

FiloSottile
Copy link
Contributor

Email to list coming.

@ekr
Copy link
Contributor

ekr commented Oct 7, 2016

@FiloSottile this needs a rebase.

correction: Github just was freaking out

@FiloSottile
Copy link
Contributor Author

@ekr done, the damned GitHub editor doesn't detect conflicts. This is a bit ridiculous.

@ekr
Copy link
Contributor

ekr commented Oct 7, 2016

@FiloSottile Don't forget to email the list.

@FiloSottile
Copy link
Contributor Author

} TicketEarlyDataInfo;
~~~~

This extension indicates that the ticket may be used to send 0-RTT data
({{early-data-indication}})). It contains one value:
({{early-data-indication}})). It contains two values:
Copy link
Contributor

Choose a reason for hiding this comment

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

(editorial nit: do we really need to have the number here? Could it just be "Its contents are:"?)


max_early_data_size
: The maximum amount of 0-RTT data that the client is allowed to send when using
this ticket, in bytes. Only Application Data payload is counted.
Copy link
Contributor

Choose a reason for hiding this comment

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

A serer receiving more than this many bytes of 0-RTT data will abort the connection with a XYZ alert.

@ekr ekr modified the milestone: -17 Oct 8, 2016
@ekr
Copy link
Contributor

ekr commented Oct 10, 2016

Merged by hand

@ekr ekr closed this Oct 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants