Skip to content

docs: clarify CURLOPT_MAXFILESIZE and CURLOPT_MAXFILESIZE_LARGE - #13372

Closed
the-blank-x wants to merge 1 commit into
curl:masterfrom
the-blank-x:clarify-maxfilesize
Closed

docs: clarify CURLOPT_MAXFILESIZE and CURLOPT_MAXFILESIZE_LARGE#13372
the-blank-x wants to merge 1 commit into
curl:masterfrom
the-blank-x:clarify-maxfilesize

Conversation

@the-blank-x

Copy link
Copy Markdown
Contributor

The bounds of the size parameter were not specified, and nor was it specified how to disable the maximum file size check.

The documentation also incorrectly stated that CURLOPT_MAXFILESIZE always returns CURLE_OK and that CURLOPT_MAXFILESIZE_LARGE only returns CURLE_OK or CURLE_UNKNOWN_OPTION.

It also did not mention what the default value is, which is zero. This commit updates the documentation to make note of all these things.

The bounds of the size parameter were not specified, and nor was it
specified how to disable the maximum file size check.

The documentation also incorrectly stated that CURLOPT_MAXFILESIZE
always returns CURLE_OK and that CURLOPT_MAXFILESIZE_LARGE only returns
CURLE_OK or CURLE_UNKNOWN_OPTION.

It also did not mention what the default value is, which is zero. This
commit updates the documentation to make note of all these things.

@bagder bagder left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent!

@bagder bagder closed this in e1f1ec0 Apr 15, 2024
@bagder

bagder commented Apr 15, 2024

Copy link
Copy Markdown
Member

Thanks!

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