test: update NG_ALLOWED_HOSTS to include port in e2e tests#33153
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates multiple end-to-end test files within the server-rendering suite to include the dynamic port in the NG_ALLOWED_HOSTS environment variable. By changing the value from 'localhost' to 'localhost:${port}', the tests now correctly account for the specific port assigned to the server instance. I have no feedback to provide.
alan-agius4
left a comment
alan-agius4
left a comment
There was a problem hiding this comment.
The is not right.
The change needs to be done in @angular/ssr, I’ll take care of this on Monday.
|
This pull request has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
2 participants
No description provided.