Page MenuHomeFreeBSD

libfetch: parse scheme://domain:/ correctly
ClosedPublic

Authored by khng on Mar 25 2024, 5:01 AM.
Tags
None
Referenced Files
F157812352: D44493.id.diff
Mon, May 25, 11:29 AM
F157771886: D44493.id136201.diff
Mon, May 25, 12:22 AM
Unknown Object (File)
Fri, May 22, 11:44 AM
Unknown Object (File)
Thu, May 21, 10:30 AM
Unknown Object (File)
Tue, May 19, 5:54 PM
Unknown Object (File)
Tue, May 12, 1:19 AM
Unknown Object (File)
Tue, May 12, 12:46 AM
Unknown Object (File)
Tue, May 12, 12:23 AM
Subscribers

Details

Summary

This improves URL-parsing compability with cURL, and unbreaks parsing of
similar kinds of URLs after commit 8d9de5b10a24.

Sponsored by: Juniper Networks, Inc.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

khng requested review of this revision.Mar 25 2024, 5:01 AM

Can you please provide an example of a URL which is parsed incorrectly without this patch?

@des something like https://example.com:/ used to work before https://reviews.freebsd.org/rG8d9de5b10a24bd2d79ed99f139c0ac28c09b15ca. Meanwhile, curl -L https://example.com:/ gives the HTML page just as curl -L https://example.com/, and is true for Firefox and Chromium-based browsers as well.

This revision is now accepted and ready to land.Mar 25 2024, 5:49 PM
This revision was automatically updated to reflect the committed changes.