Skip to content

[stable34] fix(appstore): do not wrap app store screenshots in the proxy twice - #65990

Open
backportbot[bot] wants to merge 1 commit into
stable34from
backport/64771/stable34
Open

backportbot[bot] wants to merge 1 commit into
stable34from
backport/64771/stable34

Conversation

@backportbot

@backportbot backportbot Bot commented Oct 28, 2026

Copy link
Copy Markdown

Backport of PR #64771

The app store now serves screenshot URLs already pointing at
usercontent.apps.nextcloud.com. Wrapping them again names a file the
proxy never stores the image under, so no app shows its screenshot.

Pass such URLs through unchanged, and encode the others with the URL-safe
alphabet the proxy stores its files under.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Martin Rüegg <martin.rueegg@metaworx.ch>
@backportbot
backportbot Bot requested a review from a team as a code owner October 28, 2026 20:56
@backportbot backportbot Bot added bug 3. to review Waiting for reviews feature: apps management php Pull requests that update Php code feedback-requested labels Oct 28, 2026
@backportbot
backportbot Bot removed the request for review from a team October 28, 2026 20:56
@backportbot backportbot Bot added the community pull requests from community label Oct 28, 2026
@backportbot backportbot Bot added this to the Nextcloud 34.0.6 milestone Oct 28, 2026
@backportbot
backportbot Bot requested a review from sorbaugh October 28, 2026 20:56

This branch has not been deployed

No deployments
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 community pull requests from community feature: apps management feedback-requested php Pull requests that update Php code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant