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

doc: use meaningless port number in CURLOPT_LOCALPORT example #3513

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
2 participants
@jmarshall
Copy link
Contributor

commented Jan 31, 2019

A tiny documentation tweak to hopefully help avoid the next misinterpretation like on the mailing list today: “Bug: Ignored port setting”.

Use an ephemeral port number here; previously the example had 8080 which could be confusing as the common web server port number might be misinterpreted as suggesting this option affects the remote port.

doc: use meaningless port number in CURLOPT_LOCALPORT example
Use an ephemeral port number here; previously the example had 8080
which could be confusing as the common web server port number might
be misinterpreted as suggesting this option affects the remote port.
@bagder

bagder approved these changes Jan 31, 2019

Copy link
Member

left a comment

Good point!

@bagder

This comment has been minimized.

Copy link
Member

commented Jan 31, 2019

Thanks!

@bagder bagder closed this in 427fa51 Jan 31, 2019

@jmarshall jmarshall deleted the jmarshall:localport-doc branch Jan 31, 2019

@lock lock bot locked as resolved and limited conversation to collaborators May 1, 2019

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.