Skip to content

[stable-33.0] fix(asyncimageresponse): enhance check for received SVG response #9710

Merged
nilsding merged 5 commits intostable-33.0from
backport/9697/stable-33.0
Mar 26, 2026
Merged

[stable-33.0] fix(asyncimageresponse): enhance check for received SVG response #9710
nilsding merged 5 commits intostable-33.0from
backport/9697/stable-33.0

Conversation

@backportbot
Copy link
Copy Markdown

@backportbot backportbot Bot commented Mar 26, 2026

Backport of PR #9697

Signed-off-by: Jyrki Gadinger <nilsding@nilsding.org>
Signed-off-by: Jyrki Gadinger <nilsding@nilsding.org>
Some SVG files do not start with `<svg`, but with `<?xml` and could include
some additional metadata (e.g. a generator comment or licence header ...)

Signed-off-by: Jyrki Gadinger <nilsding@nilsding.org>
this may cause an infinite re-request loop if the layout does not match

Signed-off-by: Jyrki Gadinger <nilsding@nilsding.org>
Signed-off-by: Jyrki Gadinger <nilsding@nilsding.org>
@backportbot backportbot Bot requested a review from mgallien March 26, 2026 15:18
@backportbot backportbot Bot requested review from mgallien and nilsding as code owners March 26, 2026 15:18
@backportbot backportbot Bot added this to the 33.0.1 milestone Mar 26, 2026
@github-actions
Copy link
Copy Markdown

Artifact containing the AppImage: nextcloud-appimage-pr-9710.zip

Digest: sha256:caf22c7483208167b3fb21404cafb7ffc08ed9ea53749282a883029456754d28

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@nilsding nilsding merged commit 8e1de68 into stable-33.0 Mar 26, 2026
20 of 21 checks passed
@nilsding nilsding deleted the backport/9697/stable-33.0 branch March 26, 2026 18:14
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.

1 participant