Skip to content

Comments

docs: add documentation for SSR security and host validation, including details on allowedHosts configuration#67091

Open
alan-agius4 wants to merge 4 commits intoangular:mainfrom
alan-agius4:allowed-host-docs
Open

docs: add documentation for SSR security and host validation, including details on allowedHosts configuration#67091
alan-agius4 wants to merge 4 commits intoangular:mainfrom
alan-agius4:allowed-host-docs

Conversation

@alan-agius4
Copy link
Contributor

This document adds more information about allowedHost option

@alan-agius4 alan-agius4 added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release labels Feb 17, 2026
@angular-robot angular-robot bot added the area: docs Related to the documentation label Feb 17, 2026
@ngbot ngbot bot added this to the Backlog milestone Feb 17, 2026
@github-actions
Copy link

github-actions bot commented Feb 17, 2026

Deployed adev-preview for 7e1f58c to: https://ng-dev-previews-fw--pr-angular-angular-67091-adev-prev-dw1q5uvs.web.app

Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt.

@alan-agius4 alan-agius4 force-pushed the allowed-host-docs branch 2 times, most recently from 034b94e to d632787 Compare February 17, 2026 10:11
@alan-agius4 alan-agius4 requested a review from dgp1130 February 17, 2026 10:43
@alan-agius4 alan-agius4 force-pushed the allowed-host-docs branch 2 times, most recently from efc7281 to 3b3ebd1 Compare February 17, 2026 14:14
Copy link
Contributor

@dgp1130 dgp1130 left a comment

Choose a reason for hiding this comment

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

Should we mention something in https://angular.dev/tools/cli/deployment?

I wonder if this change might be somewhat breaking to the general deployment story (ex. Firebase) if you can no longer ng new && ./deploy since you need to manually configure allowedHosts before deploying.

@alan-agius4 alan-agius4 force-pushed the allowed-host-docs branch 2 times, most recently from 1f5dc02 to df74527 Compare February 19, 2026 11:40
@alan-agius4 alan-agius4 requested a review from dgp1130 February 19, 2026 13:16
@angular-robot angular-robot bot added detected: feature PR contains a feature commit area: docs-infra Angular.dev application and infrastructure labels Feb 19, 2026
@alan-agius4 alan-agius4 force-pushed the allowed-host-docs branch 2 times, most recently from f956ba5 to 7431d1f Compare February 19, 2026 13:51
@pullapprove pullapprove bot requested a review from josephperrott February 19, 2026 14:09
@alan-agius4 alan-agius4 force-pushed the allowed-host-docs branch 5 times, most recently from 44b11b0 to a9a0300 Compare February 19, 2026 14:49
Copy link
Member

@josephperrott josephperrott left a comment

Choose a reason for hiding this comment

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

LGTM

@pullapprove pullapprove bot requested a review from josephperrott February 19, 2026 18:14
Copy link
Member

@josephperrott josephperrott left a comment

Choose a reason for hiding this comment

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

LGTM

Reviewed-for: fw-security

…ng details on `allowedHosts` configuration

This document adds more information about `allowedHost` option
…fix in shell code blocks.

The dollar sign is not always required.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: review The PR is still awaiting reviews from at least one requested reviewer adev: preview area: docs Related to the documentation area: docs-infra Angular.dev application and infrastructure detected: feature PR contains a feature commit target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants