Skip to content

[stable33] Fix downloading files from public share#3506

Open
backportbot[bot] wants to merge 3 commits into
stable33from
backport/3476/stable33
Open

[stable33] Fix downloading files from public share#3506
backportbot[bot] wants to merge 3 commits into
stable33from
backport/3476/stable33

Conversation

@backportbot
Copy link
Copy Markdown

@backportbot backportbot Bot commented May 6, 2026

Backport of #3476

@backportbot backportbot Bot requested review from artonge, danxuliu and susnux May 6, 2026 14:42
@backportbot backportbot Bot added bug Something isn't working 3. to review Waiting for reviews regression Regression of a previous working feature labels May 6, 2026
@backportbot backportbot Bot added this to the Nextcloud 33.0.4 milestone May 6, 2026
danxuliu added 2 commits May 7, 2026 01:34
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
The "public.php/dav" endpoint from server is not compatible with the
tokens used for public albums by the Photos app, but they work fine with
the "remote.php/dav" endpoint. By default @nextcloud/files/dav uses
"public.php/dav" for remote URLs in public pages, so they need to be
explicitly replaced with "remote.php/dav" (which is the one already used
for non public pages, so it is fine to use it unconditionally).

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
@danxuliu danxuliu force-pushed the backport/3476/stable33 branch from ff650c0 to c7f6e1f Compare May 6, 2026 23:53
@danxuliu
Copy link
Copy Markdown
Member

danxuliu commented May 6, 2026

/compile /

@codecov
Copy link
Copy Markdown

codecov Bot commented May 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@danxuliu danxuliu marked this pull request as ready for review May 7, 2026 00:41
Copy link
Copy Markdown
Member

@danxuliu danxuliu left a comment

Choose a reason for hiding this comment

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

Tested and works 👍

Cypress is failing hard, although it should™ be unrelated 🤔

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 bug Something isn't working regression Regression of a previous working feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants