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

docs: disambiguate CURLUPART_HOST is for host name (ie no port) #4424

Closed
wants to merge 1 commit into from

Conversation

@jay
Copy link
Member

commented Sep 25, 2019

Make it clear that it's for host name and not hostname:port (which could be implied by calling it host).

Another thing I added but only to set was "If it is a bracketed IPv6 numeric address it may contain a zone id (or you can use CURLUPART_ZONEID)." because right now we allow for example [fe80::20c:29ff:fe9c:409b%25foo] and it parses out the zone id.

@jay jay added the documentation label Sep 25, 2019
@bagder
bagder approved these changes Sep 26, 2019
docs/libcurl/curl_url_get.3 Show resolved Hide resolved
@bagder bagder closed this in 00b65e3 Sep 28, 2019
@bagder

This comment has been minimized.

Copy link
Member

commented Sep 28, 2019

Thanks!

@jay jay deleted the jay:clarify_CURLUPART_HOST branch Oct 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.