Skip to content

[stable33] fix(zip): Fix warning when downloading Zip file#59383

Merged
susnux merged 1 commit intostable33from
backport/59372/stable33
Apr 4, 2026
Merged

[stable33] fix(zip): Fix warning when downloading Zip file#59383
susnux merged 1 commit intostable33from
backport/59372/stable33

Conversation

@backportbot
Copy link
Copy Markdown

@backportbot backportbot bot commented Apr 1, 2026

Backport of PR #59372

@backportbot backportbot bot requested a review from a team as a code owner April 1, 2026 12:52
@backportbot backportbot bot requested review from ArtificialOwl, CarlSchwan, icewind1991, leftybournes, salmart-dev, skjnldsv and susnux and removed request for a team, skjnldsv and susnux April 1, 2026 12:52
@backportbot backportbot bot added 3. to review Waiting for reviews technical debt 🧱 🤔🚀 performance 🚀 labels Apr 1, 2026
@backportbot backportbot bot added this to the Nextcloud 33.0.2 milestone Apr 1, 2026
The HEAD request, create a GET subrequest which is not compatible with
the ZIP plugin since the ZIP plugin is directly streaming the content to
php://output, so we were sending the content in a HEAD request and
creating the ZIP twice and this was creating various warning in logs
too.

Signed-off-by: Carl Schwan <carlschwan@kde.org>
@susnux susnux force-pushed the backport/59372/stable33 branch from 792bc66 to d438db4 Compare April 3, 2026 23:37
@susnux susnux merged commit 3c90a8e into stable33 Apr 4, 2026
181 of 185 checks passed
@susnux susnux deleted the backport/59372/stable33 branch April 4, 2026 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews performance 🚀 technical debt 🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants