Skip to content

fix(@angular/ssr): allow all hosts in common engine rendering options to prevent validation errors#33180

Merged
alan-agius4 merged 1 commit into
angular:21.2.xfrom
alan-agius4:21-ci-fix
May 13, 2026
Merged

fix(@angular/ssr): allow all hosts in common engine rendering options to prevent validation errors#33180
alan-agius4 merged 1 commit into
angular:21.2.xfrom
alan-agius4:21-ci-fix

Conversation

@alan-agius4
Copy link
Copy Markdown
Collaborator

This is needed due to the angular/angular#68593

Copy link
Copy Markdown

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

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 CommonEngine in the Angular SSR package to include allowedHosts: ['*'] within the commonRenderingOptions, noting that validation is handled earlier in the render process. No review comments were provided for this change, and I have no feedback to provide.

@alan-agius4 alan-agius4 added target: patch This PR is targeted for the next patch release action: merge The PR is ready for merge by the caretaker labels May 13, 2026
@alan-agius4 alan-agius4 merged commit eafe1a7 into angular:21.2.x May 13, 2026
37 checks passed
@alan-agius4
Copy link
Copy Markdown
Collaborator Author

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

@alan-agius4 alan-agius4 deleted the 21-ci-fix branch May 13, 2026 08:33
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/ssr target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants