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

Repair EAP-FAST session resumption #250

Closed
wants to merge 1 commit into from
Closed

Conversation

ekasper
Copy link
Contributor

@ekasper ekasper commented Mar 27, 2015

EAP-FAST session resumption relies on handshake message lookahead
to determine server intentions. Commits
980bc1e
and
7b3ba50
removed the lookahead so broke session resumption.

This change partially reverts the commits and brings the lookahead back
in reduced capacity for TLS + EAP-FAST only. Since EAP-FAST does not
support regular session tickets, the lookahead now only checks for a
Finished message, and only if EAP-FAST is in play.

Regular handshakes are unaffected by this change.

EAP-FAST session resumption relies on handshake message lookahead
to determine server intentions. Commits
980bc1e
and
7b3ba50
removed the lookahead so broke session resumption.

This change partially reverts the commits and brings the lookahead back
in reduced capacity for TLS + EAP-FAST only. Since EAP-FAST does not
support regular session tickets, the lookahead now only checks for a
Finished message.

Regular handshakes are unaffected by this change.
@richsalz
Copy link
Contributor

+1
@ekasper did you forget you had this? :)

@ekasper
Copy link
Contributor Author

ekasper commented Jan 21, 2016

This was committed but I forgot to update the github branch. Closing.

@ekasper ekasper closed this Jan 21, 2016
openssl-machine pushed a commit to openssl/web that referenced this pull request Jul 28, 2021
For some reason, these files were excempt from automatic generation.
I cannot see a reason why we did this, so we restore the automatic
generation as originally planned.

Reviewed-by: Paul Dale <pauli@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from openssl/openssl#250)
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.

None yet

2 participants