-
Notifications
You must be signed in to change notification settings - Fork 29.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove ":" in Url.protocol? #8406
Labels
url
Issues and PRs related to the legacy built-in url module.
Comments
Including the
/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.) |
@Trott, cool! then I should create an issue for python 😎 |
This was referenced Mar 9, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Question: The protocol of an url
http://a.b.c
should behttp
orhttp:
?Observation: Given an url
http://a.b.c
, the following table shows the different parsing results ofprotocol
among Node, Python and Java:http:
http
http
Note:
Current parsing logic: https://github.com/nodejs/node/blob/master/lib/url.js#L39
An example:
The text was updated successfully, but these errors were encountered: