Add default port as 8080 in options.js to match webpack-dev-server.js

• Operating System: Windows 10 x64
• Node Version: v11.4.0
• NPM Version: 6.4.1
• webpack Version: 4.29.6
• webpack-dev-server Version: 3.3.1

• This is a bug
• This is a modification request

Code

The minimal changes to webpack.config.js to introduce HMR are:

// webpack.config.js
// ....
devServer: {
    hot: true,
    port: 8080
}
// ...
plugins: [
    new HotModuleReplacementPlugin(),
    // other plugins...
]

Expected Behavior

The port option should default to 8080 (just like the server does).

Actual Behavior

The port option does not have a default, which results in the generated code trying to contact webpack-dev-server at the default port (for URLs) of 80.

Relevant webpack-dev-server code

In bin/webpack-dev-server.js

const DEFAULT_PORT = 8080;

In bin/options.js (missing a default key)

  port: {
    describe: 'The port',
    group: CONNECTION_GROUP,
  },

For Bugs; How can we reproduce the behavior?

Simply remove the port option above from a simple webpack-dev-server example project. There will be console errors on the loaded page, and if you check the generated bundle code, you’ll find that it tries to contact webpack-dev-server at http://localhost instead of http://localhost:8080

For Features; What is the motivation and/or use-case for the feature?

Having less required options makes webpack much more approachable to beginners looking to “just make it work”.

[alsotang]

In v3.8.2, when using with program style, you also need to specify the port in options

	new WebpackDevServer(compiler, {
		port: 8080, // do not ignore this
	}).listen(8080, 'localhost', function(err) {
		console.log("[webpack-dev-server]", "http://localhost:8080/webpack-dev-server/index.html");
	});

[alexander-akait]

I think it is bug:

1. we need findPort (https://github.com/webpack/webpack-dev-server/blob/master/bin/webpack-dev-server.js#L242) before run compilation https://github.com/webpack/webpack-dev-server/blob/master/bin/webpack-dev-server.js#L145 so you always have port
2. Add test on cli and node api usage