Skip to content

[stable31] fix(dav): allow uploading of files with long filenames - #52302

Merged
provokateurin merged 2 commits into
stable31from
backport/51491/stable31
Apr 22, 2025
Merged

[stable31] fix(dav): allow uploading of files with long filenames#52302
provokateurin merged 2 commits into
stable31from
backport/51491/stable31

Conversation

@backportbot

@backportbotbackportbotBot commented Apr 22, 2025

Copy link
Copy Markdown

Backport of PR #51491

@susnux@backportbot
refactor(dav): simplify length header handling
15726db
Reduce nesting and drop duplicated sections.
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux@backportbot
fix(dav): allow uploading of files with long filenames
d3fe082
A filename must be less or equal 255 characters, but when adding the
`.part` and `.ocfiletransfer` extensions we might overflow this limit.
So we should also use filename hashes for uploading when the file has a
long filename, similar like when we are uploading to the user storage
directly.
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@backportbotbackportbotBot added this to the Nextcloud 31.0.5 milestone Apr 22, 2025
@provokateurin
provokateurin merged commit a3fcd57 into stable31Apr 22, 2025
@provokateurin
provokateurin deleted the backport/51491/stable31 branch April 22, 2025 12:52
@blizzzblizzz mentioned this pull request May 5, 2025
Merged
10 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@susnuxsusnuxsusnux approved these changes
@provokateurinprovokateurinprovokateurin approved these changes
@come-nccome-ncAwaiting requested review from come-nc
@skjnldsvskjnldsvAwaiting requested review from skjnldsv

Assignees

@susnuxsusnux

Labels

3. to reviewWaiting for reviewsbugfeature: davfeature: files

Projects

None yet

Milestone

Nextcloud 31.0.5

Development

Successfully merging this pull request may close these issues.

2 participants

@susnux@provokateurin