Skip to content

[21.2.x] fix(core): harden TransferState restoration against DOM clobbering#69081

Merged
atscott merged 1 commit into
angular:21.2.xfrom
JeanMeche:212-transfer-state-clobering
Jun 3, 2026
Merged

[21.2.x] fix(core): harden TransferState restoration against DOM clobbering#69081
atscott merged 1 commit into
angular:21.2.xfrom
JeanMeche:212-transfer-state-clobering

Conversation

@JeanMeche
Copy link
Copy Markdown
Member

Reject non-script elements when reading the SSR transfer state payload by id. This prevents attacker-controlled elements with a clobbered id from spoofing hydration state.

Backport of #69064

@angular-robot angular-robot Bot added the area: core Issues related to the framework runtime label Jun 2, 2026
@ngbot ngbot Bot added this to the Backlog milestone Jun 2, 2026
@JeanMeche JeanMeche requested a review from alan-agius4 June 2, 2026 11:52
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Jun 2, 2026
@JeanMeche JeanMeche added target: lts This PR is targeting a version currently in long-term support and removed target: lts This PR is targeting a version currently in long-term support labels Jun 2, 2026
Reject non-script elements when reading the SSR transfer state payload by id.
This prevents attacker-controlled elements with a clobbered id from spoofing
hydration state.
@JeanMeche JeanMeche force-pushed the 212-transfer-state-clobering branch from d9ac549 to 13ef820 Compare June 2, 2026 12:04
@atscott atscott added target: lts This PR is targeting a version currently in long-term support and removed target: patch This PR is targeted for the next patch release labels Jun 3, 2026
@atscott atscott merged commit 1523061 into angular:21.2.x Jun 3, 2026
20 checks passed
@atscott
Copy link
Copy Markdown
Contributor

atscott commented Jun 3, 2026

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: core Issues related to the framework runtime 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