Skip to content

[Backport 21.2.X] fix(core): reject dynamic script host elements - #69569

Merged
alxhub merged 1 commit into
angular:21.2.xfrom
SkyZeroZx:backport-69551-to-21.2.x
Jul 1, 2026
Merged

[Backport 21.2.X] fix(core): reject dynamic script host elements#69569
alxhub merged 1 commit into
angular:21.2.xfrom
SkyZeroZx:backport-69551-to-21.2.x

Conversation

@SkyZeroZx

Copy link
Copy Markdown
Contributor

Backport of #69551

The previous fix for GHSA-692r-grfm-v8x7 was incomplete because it rejected script tags only when locating an explicit host element. Dynamic component instantiation can also infer the host element from the component selector.

Move the script-host rejection to the point where ComponentFactory has resolved the host element for either path, so createComponent rejects script hosts consistently.

(cherry picked from commit 135f375)
@pullapprove
pullapprove Bot requested a review from atscott June 29, 2026 15:45
@angular-robot angular-robot Bot added the area: core Issues related to the framework runtime label Jun 29, 2026
@ngbot ngbot Bot added this to the Backlog milestone Jun 29, 2026
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker target: lts This PR is targeting a version currently in long-term support labels Jun 30, 2026
@alan-agius4
alan-agius4 removed the request for review from atscott June 30, 2026 07:28
@alxhub
alxhub merged commit 5a693ba into angular:21.2.x Jul 1, 2026
25 checks passed
@alxhub

alxhub commented Jul 1, 2026

Copy link
Copy Markdown
Member

This PR was merged into the repository. The changes were merged into the following branches:

@angular-automatic-lock-bot

Copy link
Copy Markdown

This pull request has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Aug 1, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: core Issues related to the framework runtime PullApprove: disable target: lts This PR is targeting a version currently in long-term support

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants