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

asyn-ares: initial HTTPS resolve support #16039

Closed
wants to merge 1 commit into from
Closed

Conversation

bagder
Copy link
Member

@bagder bagder commented Jan 17, 2025

Gets the ALPN list the same way DoH does. Needs c-ares 1.28.0 or later.

@bagder bagder added the name lookup DNS and related tech label Jan 17, 2025
@testclutch
Copy link

Analysis of PR #16039 at 93344895:

Test http/test_14_auth.py::TestAuth::test_14_03_digest_put_auth[h3] failed, which has NOT been flaky recently, so there could be a real issue in this PR.

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

Generated by Testclutch

@bagder bagder marked this pull request as ready for review January 18, 2025 21:47
Gets the ALPN list the same way DoH does. Needs c-ares 1.28.0 or later.

Thanks-to: Brad House

Closes #16039
@bagder bagder force-pushed the bagder/cares-HTTPS branch from 9334489 to 4a341dc Compare January 18, 2025 21:54
@bagder bagder closed this in 8368249 Jan 19, 2025
@bagder bagder deleted the bagder/cares-HTTPS branch January 19, 2025 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
name lookup DNS and related tech
Development

Successfully merging this pull request may close these issues.

2 participants