Skip to content

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

Merged
edward-ly merged 2 commits into
stable34from
backport/64771/stable34
Nov 2, 2026
Merged

edward-ly merged 2 commits into
stable34from
backport/64771/stable34

Conversation

@backportbot

@backportbot backportbot Bot commented Oct 28, 2026

Copy link
Copy Markdown

Backport of PR #64771

@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
@edward-ly
edward-ly force-pushed the backport/64771/stable34 branch from a82776b to f353adb Compare October 29, 2026 16:45
@edward-ly
edward-ly enabled auto-merge October 29, 2026 16:52
martin-rueegg and others added 2 commits November 2, 2026 09:40
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>
Signed-off-by: Edward Ly <contact@edward.ly>
@edward-ly
edward-ly force-pushed the backport/64771/stable34 branch from 9a7a6bc to ba3e14e Compare November 2, 2026 17:40
@edward-ly
edward-ly merged commit 088d612 into stable34 Nov 2, 2026
162 of 164 checks passed
@edward-ly
edward-ly deleted the backport/64771/stable34 branch November 2, 2026 18:21
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.

3 participants