Skip to content

[stable32] fix: 🐛 Update pdf save to use correct dav path and credentials#1439

Merged
danxuliu merged 5 commits into
stable32from
backport/1414/stable32
May 18, 2026
Merged

[stable32] fix: 🐛 Update pdf save to use correct dav path and credentials#1439
danxuliu merged 5 commits into
stable32from
backport/1414/stable32

Conversation

@backportbot
Copy link
Copy Markdown

@backportbot backportbot Bot commented May 12, 2026

Backport of #1414

moodyjmz and others added 4 commits May 12, 2026 18:20
Use /public.php/dav/files/SHARING_TOKEN instead of the legacy
/public.php/webdav endpoint for public share uploads.

Signed-off-by: James Manuel <[email protected]>
This should not happen, but if it does, it should throw to avoid
silently resolving to "/files/undefined" and hiding the error.

Signed-off-by: Ferdinand Thiessen <[email protected]>
Signed-off-by: Daniel Calviño Sánchez <[email protected]>
Inline getUserRoot into getRootPath so both public and private
paths are resolved in one place.

Signed-off-by: James Manuel <[email protected]>
requesttoken is for CSRF protection and is only meaningful when
there is an authenticated session. Public share uploads authenticate
via the token in the URL path, so the header is not needed there.

Signed-off-by: James Manuel <[email protected]>
@danxuliu danxuliu force-pushed the backport/1414/stable32 branch from 95322e5 to d2662ae Compare May 12, 2026 16:21
@danxuliu
Copy link
Copy Markdown
Member

/compile /

Signed-off-by: nextcloud-command <[email protected]>
@danxuliu danxuliu marked this pull request as ready for review May 18, 2026 17:45
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 👍

@danxuliu danxuliu merged commit 75a5b28 into stable32 May 18, 2026
38 checks passed
@danxuliu danxuliu deleted the backport/1414/stable32 branch May 18, 2026 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants