Skip to content

[stable31] fix(nginx): fastcgi_request_buffering must be on#13603

Merged
AndyScherzinger merged 1 commit intostable31from
backport/13598/stable31
Sep 4, 2025
Merged

[stable31] fix(nginx): fastcgi_request_buffering must be on#13603
AndyScherzinger merged 1 commit intostable31from
backport/13598/stable31

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Sep 4, 2025

Backport of PR #13598

* resolves: #9574

Some clients like DAVx5 use chunked transfer encoding.
PHP-FPM does not support this[1] because it requires to comply with the
CGI standard - 20+ years old - which enforces a Content-Length header.

So we need to buffer the whole request until passing it to PHP-FPM.

Without this 0-byte files would be created on Nextcloud as PHP-FPM will
not only set content-length to 0 but also does not pass the body data
stream.

[1] https://bugs.php.net/bug.php?id=51191 and php/php-src#7509

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@AndyScherzinger AndyScherzinger merged commit 54ae10b into stable31 Sep 4, 2025
11 of 12 checks passed
@AndyScherzinger AndyScherzinger deleted the backport/13598/stable31 branch September 4, 2025 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants