Add `removePort` option

In the Docs, you have an example that has a port on a url and then when you wrap the url with “normalizeUrl” the port is no longer included:

When I run this same example locally, the Port :80/ is still included - which is different behavior from what your documentation example above shows:

How would I remove the :80/ port portion of a url with your package ?

I am pretty sure there is a bug because when I did another test the behavior is not consistent. As you can see in the image below, the results are not consistent just by changing a url from having http too https and also I see that just changing the actual number for the port in the url makes the behavior different too:

[loynoir]

@ianizaguirre

I think removePort should be rename to unsafeRemovePort, or current naming style forceRemovePort.

Because https://example.com:80 and https://example.com is NOT SAME AT ALL, they MAY just happen to LOOK same in specific cases.

• https://example.com:80 Secure Connection Failed
• https://example.com Example Domain

This suggestion may prevent confusion with the safe remove port, like remove :80 from http and remove :443 from https.

[ianizaguirre]

@sindresorhus

Pull request has been created: https://github.com/sindresorhus/normalize-url/pull/174