Skip to content

fix: SVG image not displayed on Links portlet - EXO-88864 - #849

Merged
hakermi merged 1 commit into
developfrom
dev-30072026
Jul 30, 2026
Merged

fix: SVG image not displayed on Links portlet - EXO-88864#849
hakermi merged 1 commit into
developfrom
dev-30072026

Conversation

@hakermi

@hakermi hakermi commented Jul 30, 2026

Copy link
Copy Markdown
Member

Prior to this change, link icons were always stored and served as "image/png", regardless of the uploaded file type. An SVG icon was therefore served with a wrong content type and couldn't be rendered by the browser, so no image appeared on the link item.

The icon is now stored with the real mime type and file name from the uploaded resource, and served with that content type (falling back to image/png when unknown).

Prior to this change, link icons were always stored and served as
"image/png", regardless of the uploaded file type. An SVG icon was
therefore served with a wrong content type and couldn't be rendered by
the browser, so no image appeared on the link item.

The icon is now stored with the real mime type and file name from the
uploaded resource, and served with that content type (falling back to
image/png when unknown).
@hakermi
hakermi requested review from Jihed525 and ahamdi July 30, 2026 13:56
@hakermi
hakermi enabled auto-merge (squash) July 30, 2026 13:56
@sonarqubecloud

Copy link
Copy Markdown

@hakermi
hakermi merged commit c8ae01c into develop Jul 30, 2026
10 checks passed
@hakermi
hakermi deleted the dev-30072026 branch July 30, 2026 14:06
Jihed525 pushed a commit that referenced this pull request Jul 31, 2026
Prior to this change, link icons were always stored and served as
"image/png", regardless of the uploaded file type. An SVG icon was
therefore served with a wrong content type and couldn't be rendered by
the browser, so no image appeared on the link item.

The icon is now stored with the real mime type and file name from the
uploaded resource, and served with that content type (falling back to
image/png when unknown).

(cherry picked from commit c8ae01c)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants