Skip to content

Improved fix for TLS1.3 to TLS1.2 client downgrade#7626

Merged
JacobBarthelmeh merged 4 commits intowolfSSL:masterfrom
lealem47:parseServerHello
Jun 6, 2024
Merged

Improved fix for TLS1.3 to TLS1.2 client downgrade#7626
JacobBarthelmeh merged 4 commits intowolfSSL:masterfrom
lealem47:parseServerHello

Conversation

@lealem47
Copy link
Copy Markdown
Contributor

@lealem47 lealem47 commented Jun 6, 2024

Description

An iteration on the fix in #7619

Fixes zd#18074

Testing

Reproducer in ticket

Checklist

  • added tests
  • updated/added doxygen
  • updated appropriate READMEs
  • Updated manual and documentation

@lealem47 lealem47 self-assigned this Jun 6, 2024
@lealem47 lealem47 assigned wolfSSL-Bot and unassigned lealem47 Jun 6, 2024
Comment thread src/tls13.c
Comment thread src/tls13.c Outdated
@lealem47 lealem47 requested a review from JacobBarthelmeh June 6, 2024 22:11
@lealem47 lealem47 assigned wolfSSL-Bot and unassigned lealem47 Jun 6, 2024
Copy link
Copy Markdown
Contributor

@JacobBarthelmeh JacobBarthelmeh left a comment

Choose a reason for hiding this comment

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

Thanks! I'd like another reviewer on the change before it's merged.

@JacobBarthelmeh
Copy link
Copy Markdown
Contributor

Retest this please Jenkins

@JacobBarthelmeh JacobBarthelmeh merged commit d09f955 into wolfSSL:master Jun 6, 2024
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.

4 participants