Skip to content

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

Merged
mgechev merged 1 commit into
angular:masterfrom
clydin:server-custom-path
Mar 29, 2019
Merged

fix(@angular-devkit/build-angular): ensure correct dev server path with public host option#13838
mgechev merged 1 commit into
angular:masterfrom
clydin:server-custom-path

Conversation

@clydin

@clydin clydin commented Mar 6, 2019

Copy link
Copy Markdown
Member

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
Comment thread packages/angular_devkit/build_angular/src/dev-server/index.ts Outdated
@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
Copy Markdown

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