Skip to content

URL-SYNTAX: document more URL parsing details#21841

Closed
bagder wants to merge 2 commits into
masterfrom
bagder/url-syntax
Closed

URL-SYNTAX: document more URL parsing details#21841
bagder wants to merge 2 commits into
masterfrom
bagder/url-syntax

Conversation

@bagder
Copy link
Copy Markdown
Member

@bagder bagder commented Jun 2, 2026

  • IPv4 numerical address
  • IPv6 numerical address + zone id mention
  • No IPvFuture support
  • Some path parsing details

- IPv4 numerical address
- IPv6 numerical address + zone id mention
- No IPvFuture support
- Some path parsing details
@bagder bagder marked this pull request as ready for review June 2, 2026 08:59
@bagder bagder requested a review from Copilot June 2, 2026 09:02
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Expands docs/URL-SYNTAX.md with additional details about how libcurl parses URLs, covering numerical IPv4/IPv6 address handling, lack of IPvFuture support, and path normalization. Also applies minor line-wrapping reflow to two existing paragraphs.

Changes:

  • Adds new subsections describing numerical IPv4 normalization, IPv6 zone-id handling, and unsupported IPvFuture format.
  • Adds a new top-level "Path" section noting /./, /../ removal and percent-encoding normalization to uppercase.
  • Notes that libcurl rejects hostnames with more than one trailing dot, and minor reflow of two existing paragraphs.

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

Comment thread docs/URL-SYNTAX.md Outdated
language

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@bagder bagder closed this in 277db54 Jun 2, 2026
@bagder bagder deleted the bagder/url-syntax branch June 2, 2026 09:14
outcast36 pushed a commit to greearb/curl that referenced this pull request Jun 3, 2026
- IPv4 numerical address
- IPv6 numerical address + zone id mention
- No IPvFuture support
- Some path parsing details

Closes curl#21841
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.

2 participants