Skip to content

[Backport 21.2.X ] fix(@angular/build): serialize server manifest asset paths - #34532

Merged
alan-agius4 merged 2 commits into
angular:21.2.xfrom
SkyZeroZx:fix/server-manifest-serialization-21.2.x
Oct 26, 2026
Merged

alan-agius4 merged 2 commits into
angular:21.2.xfrom
SkyZeroZx:fix/server-manifest-serialization-21.2.x

Conversation

@SkyZeroZx

Copy link
Copy Markdown
Contributor

backport of #34493

Serialize the values embedded in the generated server manifests with JSON.stringify, so a prerendered route such as ssg/customer's-choice no longer produces an invalid manifest.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the Angular server application manifest generation to use JSON.stringify instead of direct string interpolation for asset paths, hashes, base hrefs, and entry points. This ensures that paths containing JavaScript string delimiters, such as single quotes, are correctly serialized and do not cause syntax errors in the generated executable manifest. Unit and end-to-end tests have been added to verify this behavior. As there are no review comments, no additional feedback is provided.

@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 Oct 26, 2026
@angular-robot
angular-robot Bot requested a review from alan-agius4 October 26, 2026 15:20
@alan-agius4
alan-agius4 merged commit 8ad2547 into angular:21.2.x Oct 26, 2026
59 of 62 checks passed
@alan-agius4

Copy link
Copy Markdown
Collaborator

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

@SkyZeroZx
SkyZeroZx deleted the fix/server-manifest-serialization-21.2.x branch October 26, 2026 19:05
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: @angular/build 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.

2 participants