Skip to content

fix: load Image/IFrame sources when disabled (CP: 25.0)#24364

Merged
caalador merged 1 commit into
25.0from
cherry/cherrypick-24346-to-25.0
May 19, 2026
Merged

fix: load Image/IFrame sources when disabled (CP: 25.0)#24364
caalador merged 1 commit into
25.0from
cherry/cherrypick-24346-to-25.0

Conversation

@mcollovati
Copy link
Copy Markdown
Collaborator

This PR cherry-picks changes from the original PR #24346 to branch 25.0.


When an Image or IFrame backed by a DownloadHandler lives inside a disabled component, the browser receives a 403 and the resource never loads.

Image.setSrc(DownloadHandler) and IFrame.setSrc(DownloadHandler) now allow the resource to be served regardless of the owner's enabled state, since these sources are fetched passively as part of rendering rather than as a user action.

Fixes #22772

When an `Image` or `IFrame` backed by a `DownloadHandler` lives inside a
disabled component, the browser receives a 403 and the resource never
loads.

`Image.setSrc(DownloadHandler)` and `IFrame.setSrc(DownloadHandler)` now
allow the resource to be served regardless of the owner's enabled state,
since these sources are fetched passively as part of rendering rather
than as a user action.

Fixes #22772
@sonarqubecloud
Copy link
Copy Markdown

@mcollovati mcollovati marked this pull request as ready for review May 18, 2026 13:58
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 18, 2026

Test Results

1 322 files  + 2  1 322 suites  +2   1h 18m 58s ⏱️ - 3m 32s
9 432 tests +10  9 365 ✅ +10  67 💤 ±0  0 ❌ ±0 
9 883 runs  + 3  9 808 ✅ + 3  75 💤 ±0  0 ❌ ±0 

Results for commit 1231bc1. ± Comparison against base commit 61bfa84.

♻️ This comment has been updated with latest results.

@caalador caalador merged commit 54a12a0 into 25.0 May 19, 2026
48 of 50 checks passed
@caalador caalador deleted the cherry/cherrypick-24346-to-25.0 branch May 19, 2026 04:11
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.

2 participants