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

openssl: fix ECH logic #15814

Closed
wants to merge 1 commit into from
Closed

openssl: fix ECH logic #15814

wants to merge 1 commit into from

Conversation

bagder
Copy link
Member

@bagder bagder commented Dec 23, 2024

  • on error, bail out proper
  • remove unused #define

- on error, bail out proper
- remove unused #define
@bagder bagder added the TLS label Dec 23, 2024
@testclutch
Copy link

Analysis of PR #15814 at dc194b3d:

Test 1009 failed, which has NOT been flaky recently, so there could be a real issue in this PR.

Test http/test_02_download.py::TestDownload::test_02_31_parallel_upgrade failed, which has NOT been flaky recently, so there could be a real issue in this PR.

Test 498 failed, which has NOT been flaky recently, so there could be a real issue in this PR.

Generated by Testclutch

@bagder bagder closed this in 6090018 Dec 23, 2024
@bagder bagder deleted the bagder/ech-fix branch December 23, 2024 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants