Incorrect output when using environment PORT

🐛 bug report

When setting the port using the environment variable PORT; parcel incorrectly reports that the port could not be used.

🎛 Configuration (.babelrc, package.json, cli command)

Command line:

set PORT=3000
node --max-old-space-size=8192 node_modules/parcel-bundler/bin/cli.js serve -d build/serve --no-autoinstall --cache-dir build/serve_cache src/*.html

Server running at http://localhost:3000 - configured port 3000 could not be used.

🤔 Expected Behavior

Expects parcel to report:

Server running at http://localhost:3000

😯 Current Behavior

Parcel reports:

Server running at http://localhost:3000 - configured port 3000 could not be used.

💁 Possible Solution

In src/Server.js

 let addon =
        server.address().port !== port
          ? `- ${logger.chalk.yellow(
              `configured port ${port} could not be used.`
            )}`
          : '';

server.address().port is a number port could be a string or a number; when specifying an environment PORT on Windows, it seems that port is a string

Adding a Number.parseInt to the port seems to fix the problem:

 let addon =
        server.address().port !== Number.parse(port)
          ? `- ${logger.chalk.yellow(
              `configured port ${port} could not be used.`
            )}`
          : '';

🌍 Your Environment

Software	Version(s)
Parcel	1.12.3
Operating System	Windows 10

[callumgare]

@ppqn This was annoying me so I’ve submitted a pull request 😃

[JwanKhalaf]

This was the fix for me: https://stackoverflow.com/a/58457861/613605