Skip to content

[stable33] Fix downloading files from public share - #3506

Merged
AndyScherzinger merged 3 commits into
stable33from
backport/3476/stable33
May 19, 2026
Merged

[stable33] Fix downloading files from public share#3506
AndyScherzinger merged 3 commits into
stable33from
backport/3476/stable33

Conversation

@backportbot

@backportbot backportbot Bot commented May 6, 2026

Copy link
Copy Markdown

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

danxuliu commented May 6, 2026

Copy link
Copy Markdown
Member

/compile /

@codecov

codecov Bot commented May 6, 2026

Copy link
Copy Markdown

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

@danxuliu danxuliu left a comment

Copy link
Copy Markdown
Member

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 🤔

@danxuliu

Copy link
Copy Markdown
Member

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

It is indeed unrelated. The snapshots show Your browser is not supported. Please upgrade to a newer version or a supported one 🤷

@AndyScherzinger
AndyScherzinger merged commit b29fb8f into stable33 May 19, 2026
48 of 54 checks passed
@AndyScherzinger
AndyScherzinger deleted the backport/3476/stable33 branch May 19, 2026 15:50
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.

3 participants