Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

Bug Fix for https://github.com/joyent/node/issues/562 #1078

Closed
wants to merge 4 commits into from
Closed

Bug Fix for https://github.com/joyent/node/issues/562 #1078

wants to merge 4 commits into from

Commits on May 20, 2011

  1. Fixing a bug in url.parse: nodejs#562 

    The file:// protocol *always* has a hostname; it's frequently abbreviated as
an empty string, which represents 'localhost' implicitly.

According to RFC 1738 (http://tools.ietf.org/html/rfc1738):

A file URL takes the form:

   file://<host>/<path>

where <host> is the fully qualified domain name of the system on
which the <path> is accessible...

As a special case, <host> can be the string "localhost" or the empty
string; this is interpreted as `the machine from which the URL is
being interpreted'.
    @ryanpetrello
    ryanpetrello committed May 20, 2011
    Configuration menu
    Copy the full SHA
    2c96e2d View commit details
    Browse the repository at this point in the history

  2. Improving a unit test for url.parse.

    @ryanpetrello
    ryanpetrello committed May 20, 2011
    Configuration menu
    Copy the full SHA
    34b29b8 View commit details
    Browse the repository at this point in the history

  3. Revert "Improving a unit test for url.parse." 

    This reverts commit 34b29b8.
    @ryanpetrello
    ryanpetrello committed May 20, 2011
    Configuration menu
    Copy the full SHA
    3a4a029 View commit details
    Browse the repository at this point in the history

  4. Improved unit tests for url.parse's handling of file://

    @ryanpetrello
    ryanpetrello committed May 20, 2011
    Configuration menu
    Copy the full SHA
    e4cdf83 View commit details
    Browse the repository at this point in the history