Skip to content

curl_url_set.md: expand the CURLU_NO_AUTHORITY description - #22515

Closed
bagder wants to merge 3 commits into
masterfrom
bagder/url-no-auth
Closed

bagder wants to merge 3 commits into
masterfrom
bagder/url-no-auth

Conversation

@bagder

@bagder bagder commented Aug 7, 2026

Copy link
Copy Markdown
Member

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Expands documentation for CURLU_NO_AUTHORITY, clarifying its behavior for hostless URLs using known schemes.

Changes:

  • Adds known-scheme examples.
  • Explains default parse-error behavior.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/libcurl/curl_url_set.md Outdated
@bagder
bagder marked this pull request as ready for review August 7, 2026 10:03
@testclutch

Copy link
Copy Markdown

Analysis of PR #22515 at 366227d2:

Test 2100 failed, but it has been 1.4% flaky lately, so it's probably NOT a fault of the PR. Note that this test has failed in 4 different CI jobs (the link just goes to one of them).

Test ../../tests/http/test_21_resolve.py::TestResolve::test_21_07_dnsd_a[DoH] failed, which has NOT been flaky recently, so there could be a real issue in this PR. Note that this test has failed in 2 different CI jobs (the link just goes to one of them).

Test ../../tests/http/test_21_resolve.py::TestResolve::test_21_08_dnsd_aaaa[DoH] failed, which has NOT been flaky recently, so there could be a real issue in this PR. Note that this test has failed in 2 different CI jobs (the link just goes to one of them).

Generated by Testclutch

@bagder bagder closed this in 2a5d6a5 Aug 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants