Skip to content

docs: clarify that curl often uses the argument as-is#17829

Closed
bagder wants to merge 2 commits intomasterfrom
bagder/arguments-as-is
Closed

docs: clarify that curl often uses the argument as-is#17829
bagder wants to merge 2 commits intomasterfrom
bagder/arguments-as-is

Conversation

@bagder
Copy link
Copy Markdown
Member

@bagder bagder commented Jul 6, 2025

Often without any specific filtering done, so whatever the user passes on is used verbatim. Sometimes this is good, sometimes it causes unexpected outcomes.

@bagder
Copy link
Copy Markdown
Member Author

bagder commented Jul 6, 2025

Maybe we should rather mention this caveat in a generic fashion for options in general instead of repeating this many times. Especially since this repetition makes it easy to forget to do it for a specific option and if we do it for 11 options but not for some specific other option, it might trick someone into believing those other options are different.

@bagder
Copy link
Copy Markdown
Member Author

bagder commented Jul 6, 2025

That is what I did now.

@bagder bagder marked this pull request as ready for review July 6, 2025 10:34
@bagder bagder requested a review from dfandrich July 6, 2025 10:34
@dfandrich
Copy link
Copy Markdown
Contributor

This looks fine to me. But there should be an equivalent for curl_easy_setopt() as well.

@bagder bagder force-pushed the bagder/arguments-as-is branch from 65ddfcd to 22187a0 Compare July 8, 2025 08:13
@bagder bagder changed the title docs/cmdline-opts: clarify that curl often uses the argument as-is docs: clarify that curl often uses the argument as-is Jul 8, 2025
@bagder bagder closed this in 897633e Jul 9, 2025
@bagder bagder deleted the bagder/arguments-as-is branch July 9, 2025 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants