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

cURL 8.1.0 URL using bad/illegal format or missing URL for URL starting with digit #11129

Closed
PkAIR opened this issue May 17, 2023 · 1 comment
Assignees

Comments

@PkAIR
Copy link

PkAIR commented May 17, 2023

With latest release of cURL 8.1.0 now it impossible to use it for get/put requests with URLs starting with digits.

I did this

curl http://10.xxx.xxx.xxx/composer/packages.json

I expected the following

File was downloaded but got URL using bad/illegal format or missing URL

curl/libcurl version

cURL 8.1.0

operating system

Latest Alpine docker image

@PkAIR PkAIR changed the title cURL 8.1.0 URL using bad/illegal format or missing URL for URL starting with digit cURL 8.1.0 URL using bad/illegal format or missing URL for URL starting with digit May 17, 2023
@bagder bagder added the URL label May 17, 2023
@bagder bagder self-assigned this May 17, 2023
@bagder
Copy link
Member

bagder commented May 17, 2023

My apologies.

This regression is considered important enough to warrant a follow-up patch release. 8.1.1 is now due to ship on May 23, 2023. Stay tuned. We will also merge other bugfixes that are ready to include in this release.

bagder added a commit that referenced this issue May 17, 2023
It can only be an IPv4 address if the last part is all digits, otherwise
it is a host name.

Extended test 1560 accordingly.

Reported-by: Pavel Kalyugin
Fixes #11129
@bagder bagder pinned this issue May 18, 2023
bagder added a commit that referenced this issue May 18, 2023
By providing the URL API error message into the error message.

Ref: #11129
bagder added a commit that referenced this issue May 18, 2023
It can only be an IPv4 address if the last part is all digits, otherwise
it is a host name. Even slightly wrong IPv4 will now be passed through
as a host name.

Regression from 17a15d8 shipped in 8.1.0

Extended test 1560 accordingly.

Reported-by: Pavel Kalyugin
Fixes #11129
Closes #11131
bagder added a commit that referenced this issue May 18, 2023
It can only be an IPv4 address if all parts are all digits and no more than
four parts, otherwise it is a host name. Even slightly wrong IPv4 will now be
passed through as a host name.

Regression from 17a15d8 shipped in 8.1.0

Extended test 1560 accordingly.

Reported-by: Pavel Kalyugin
Fixes #11129
Closes #11131
bagder added a commit that referenced this issue May 18, 2023
By providing the URL API error message into the error message.

Ref: #11129
Closes #11137
bagder added a commit that referenced this issue May 18, 2023
It can only be an IPv4 address if all parts are all digits and no more than
four parts, otherwise it is a host name. Even slightly wrong IPv4 will now be
passed through as a host name.

Regression from 17a15d8 shipped in 8.1.0

Extended test 1560 accordingly.

Reported-by: Pavel Kalyugin
Fixes #11129
Closes #11131
@bagder bagder closed this as completed in 92772e6 May 19, 2023
@bagder bagder unpinned this issue May 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants