Skip to content

IPv6 address in URL #3218

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.

Sign up for GitHub

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

Jump to bottom
Closed
tpaukrt opened this issue Nov 2, 2018 · 1 comment
Closed

IPv6 address in URL #3218

tpaukrt opened this issue Nov 2, 2018 · 1 comment
Labels
regression URL

Comments

@tpaukrt
Copy link

tpaukrt commented Nov 2, 2018

I did this

curl http://[fd00:a41::50]

I did not expected the following

curl: (3) URL using bad/illegal format or missing URL

curl/libcurl version

curl 7.62.0 (arm-unknown-linux-gnu) libcurl/7.62.0 OpenSSL/1.0.2o zlib/1.2.11
Release-Date: 2018-10-31
Protocols: ftp ftps http https
Features: IPv6 Largefile NTLM SSL libz TLS-SRP UnixSockets HTTPS-proxy
@bagder
Copy link
Member

bagder commented Nov 2, 2018

Clearly a regression from the URL parser overhaul. I'm on it. A work-around that seems to avoid this bug is to add the port number:

curl 'http://[fd00:a41::50]:80/'

bagder added a commit that referenced this issue Nov 2, 2018
@bagder
URL: fix IPv6 numeral address parser

Verified

This commit was signed with the committer’s verified signature.
bagder Daniel Stenberg
GPG key ID: 5CC908FDB71E12C2
Verified
Learn about vigilant mode
7ffef9d 
Regression from 46e1640. Extended test 1560 to verify.

Reported-by: tpaukrt on github
Fixes #3218
@bagder bagder mentioned this issue Nov 2, 2018
Closed
@bagder bagder closed this as completed in b280948 Nov 2, 2018
delroth added a commit to delroth/nixpkgs that referenced this issue Nov 22, 2018
@delroth
curl: cherry-pick upstream patch for ipv6 url parsing

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
Learn about vigilant mode
ed08ff4 
Upstream bug: curl/curl#3218.

This causes nixos/tests/ipv6.nix to fix since the last staging merge.
@delroth delroth mentioned this issue Nov 22, 2018
Merged
9 tasks
FRidh pushed a commit to NixOS/nixpkgs that referenced this issue Nov 22, 2018
@delroth @FRidh
curl: cherry-pick upstream patch for ipv6 url parsing

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
Learn about vigilant mode
90720d0 
Upstream bug: curl/curl#3218.

This causes nixos/tests/ipv6.nix to fix since the last staging merge.
vcunat pushed a commit to NixOS/nixpkgs that referenced this issue Nov 25, 2018
@delroth @vcunat
curl: cherry-pick upstream patch for ipv6 url parsing

Partially verified

This commit is signed with the committer’s verified signature.
vcunat’s contribution has been verified via GPG key.
We cannot verify signatures from co-authors, and some of the co-authors attributed to this commit require their commits to be signed.
a7fd431 
Upstream bug: curl/curl#3218.

This causes nixos/tests/ipv6.nix to fix since the last staging merge.

(cherry picked from commit 90720d0)
In the end I decided to pick it directly without staging,
so that the various security fixes get to stable* channels fast.
@tonystz tonystz mentioned this issue Nov 28, 2018
Closed
@lock lock bot locked as resolved and limited conversation to collaborators Jan 31, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
regression URL
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bagder @tpaukrt