Skip to content

fix: load Image/IFrame sources when disabled (#24346) (CP: 25.1)#24363

Merged
caalador merged 1 commit into
25.1from
cherry-pick-24346-to-25.1-1779111590875
May 19, 2026
Merged

fix: load Image/IFrame sources when disabled (#24346) (CP: 25.1)#24363
caalador merged 1 commit into
25.1from
cherry-pick-24346-to-25.1-1779111590875

Conversation

@vaadin-bot
Copy link
Copy Markdown
Collaborator

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

Original PR description

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

@github-actions
Copy link
Copy Markdown

Test Results

 1 393 files  + 2   1 393 suites  +2   1h 29m 34s ⏱️ +53s
10 045 tests +10   9 975 ✅ +10  70 💤 ±0  0 ❌ ±0 
10 518 runs  +10  10 439 ✅ +10  79 💤 ±0  0 ❌ ±0 

Results for commit 782e787. ± Comparison against base commit 4a2c5f4.

@caalador caalador merged commit d15523b into 25.1 May 19, 2026
29 checks passed
@caalador caalador deleted the cherry-pick-24346-to-25.1-1779111590875 branch May 19, 2026 04:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants