Skip to content

fix(files_sharing): Drop trailing '?' from public download redirect URL#59867

Merged
susnux merged 1 commit intomasterfrom
fix/public-share-download-redirect-trailing-question-mark
Apr 23, 2026
Merged

fix(files_sharing): Drop trailing '?' from public download redirect URL#59867
susnux merged 1 commit intomasterfrom
fix/public-share-download-redirect-trailing-question-mark

Conversation

@nfebe
Copy link
Copy Markdown
Contributor

@nfebe nfebe commented Apr 23, 2026

Public share download links now redirect to the public DAV endpoint cleanly when no additional query parameters are present. Previously the redirect always appended a trailing '?' to the target URL, which caused an Internal Server Error for clients that called the legacy download endpoint without query parameters (e.g. the Thunderbird FileLink extension). Links shared before this change are resolvable again.

Public share download links now redirect to the public DAV endpoint
cleanly when no additional query parameters are present. Previously the
redirect always appended a trailing '?' to the target URL, which caused
an Internal Server Error for clients that called the legacy download
endpoint without query parameters (e.g. the Thunderbird FileLink
extension). Links shared before this change are resolvable again.

Signed-off-by: nfebe <fenn25.fn@gmail.com>
@nfebe nfebe requested a review from a team as a code owner April 23, 2026 04:59
@nfebe nfebe requested review from Altahrim, ArtificialOwl, CarlSchwan and icewind1991 and removed request for a team April 23, 2026 05:00
@nfebe
Copy link
Copy Markdown
Contributor Author

nfebe commented Apr 23, 2026

/backport to stable33

@nfebe
Copy link
Copy Markdown
Contributor Author

nfebe commented Apr 23, 2026

/backport to stable32

@nfebe
Copy link
Copy Markdown
Contributor Author

nfebe commented Apr 23, 2026

/backport to stable31

@nfebe
Copy link
Copy Markdown
Contributor Author

nfebe commented Apr 23, 2026

/backport to stable33

@nfebe
Copy link
Copy Markdown
Contributor Author

nfebe commented Apr 23, 2026

/backport to stable32

@nfebe
Copy link
Copy Markdown
Contributor Author

nfebe commented Apr 23, 2026

/backport to stable31

@tcitworld
Copy link
Copy Markdown
Member

Could you drop the unrelated changes and only keep your commit?

@miaulalala
Copy link
Copy Markdown
Contributor

Could you drop the unrelated changes and only keep your commit?

Base branch issues 🙈 should be fine now

@susnux susnux merged commit 0761693 into master Apr 23, 2026
202 of 204 checks passed
@susnux susnux deleted the fix/public-share-download-redirect-trailing-question-mark branch April 23, 2026 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants