Skip to content

fix(@angular-devkit/build-angular): ensure correct dev server path with public host option #13838

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 29, 2019

Conversation

clydin
Copy link
Member

@clydin clydin commented Mar 6, 2019

If a custom path is provided the webpack dev server client drops the sockjs-node segment. This adds it back so that behavior is consistent when using a custom URL path.

Fixes #13627

@clydin clydin force-pushed the server-custom-path branch from 179bfb2 to a12c0c7 Compare March 7, 2019 00:12
@clydin clydin added the target: major This PR is targeted for the next major release label Mar 7, 2019
@clydin clydin added target: rc This PR is targeted for the next release-candidate and removed target: major This PR is targeted for the next major release labels Mar 19, 2019
@clydin clydin changed the base branch from master to 7.3.x March 19, 2019 16:22
@clydin clydin changed the base branch from 7.3.x to master March 19, 2019 16:22
@clydin clydin added target: patch This PR is targeted for the next patch release and removed target: rc This PR is targeted for the next release-candidate labels Mar 19, 2019
…th public host option

If a custom path is provided the webpack dev server client drops the sockjs-node segment. This adds it back so that behavior is consistent when using a custom URL path.

Fixes angular#13627
@clydin clydin force-pushed the server-custom-path branch from a12c0c7 to 3f1ee2c Compare March 29, 2019 22:03
@mgechev mgechev merged commit 36d78de into angular:master Mar 29, 2019
@clydin clydin deleted the server-custom-path branch March 29, 2019 23:39
@clydin clydin added target: major This PR is targeted for the next major release and removed target: patch This PR is targeted for the next patch release labels Mar 29, 2019
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
target: major This PR is targeted for the next major release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wrong SockJS Host validation with Reverse Proxy
4 participants