feat: use server address and fallback to 0.0.0.0 (location.hostname) in createDomain #2869
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
For Bugs and Features; did you add new tests?
Covered by existing tests.
Motivation / Use-Case
createDomain
is used in two places:createSocketUrl
: by setting to0.0.0.0
, it uses thelocation.hostname
by default, like the port (fix(cli): use webpack-cli@4 #2845 (comment), fix: use location.port as the client port by default #2850, feat(client): allow sock host to use browser location's host #2859)showStatus
: this allows settingoptions.host = undefined
and letlisten()
handle the address binding.This is required for webpack/webpack-cli#2141.
Breaking Changes
Yes, the client uses the hostname of the current location (
location.hostname
), by default. To get existing behavior, set theclient.host
with the hostname you'd like to set to, e.g.:Additional Info