Remove ":" in Url.protocol?

[maxis1718]

Question: The protocol of an url http://a.b.c should be http or http:?

Observation: Given an url http://a.b.c, the following table shows the different parsing results of protocol among Node, Python and Java:

Node	Python	Java
http:	http	http

Note:

Current parsing logic: https://github.com/nodejs/node/blob/master/lib/url.js#L39

An example:

import URL from 'url';
let parsedUrl = URL.parse('http://a.b.c')

Url {
  protocol: 'http:',
  ...
}

[Trott]

Including the : complies with the WHATWG URL Standard:

The protocol attribute’s getter must return context object url’s scheme, followed by ":".

/cc @jasnell

@maxis1718 You may also be interested in the currently-being-worked-on URL parser that is intended to be fully WHATWG URL compliant. (The current one pre-dates the spec and is not fully compliant.)

[maxis1718]

@Trott, cool! then I should create an issue for python 😎