url.relative (to complement path.relative)

[junosuarez]

The core url module currently has url.resolve() to compute absolute URLs from relative segments (which is complementary with path.resolve()). but it is missing the complementary .relative() function to compute relative URLs from two absolute URLs.

Many programs simply use path.relative() when dealing with URLs (e.g.: https://github.com/substack/node-browserify/blob/ce3deecf7be27373c0131286bda86533ec83ffd1/index.js#L692 ) but on Windows systems, path.relative() uses the path.sep character, resulting in relative URLs like ..\\..\\a\\b instead of ../../a/b.

The algorithm for computing relative URLs is (AFAIK) the same, but the separator character should always be a / as specified in RFC 3986 Section 3.3.

Opening this for discussion here prior to writing a PR.

[Qard]

+1 from me. I see no reason not to have it.

[rvagg]

I see no reason not to have it.

Not exactly a great rule to guide additions to core. I guess this points to the need for clearer guiding principles; something like "minimalism and quality API" would sum it up for me.

[Qard]

Well, I don't mean we should just accept anything that's not blatantly against the spirit of core. Minimalism is certainly good.

I just agree that the consistency with the path module is nice too, and it seems both generally useful and relatively easy to support.

[Fishrock123]

Seems like something that users would expect. ±0

[junosuarez]

Agree with @rvagg that there should be a guiding principle, and I generally agree with "keep core small, push things into userland modules." In that spirit, I published url-relative.

Another principle to balance is "principle of least surprise" in terms of providing consistency across core apis. I knew about and use path.relative() fairly often, and was surprised not to set an equivalent function for dealing with urls. Since the url module is at stability 3, I figured this issue was worth raising.

[rvagg]

ftr I'm not a -1 on this, but I'm going to label it semver-minor so if this gets solid sponsorship from a collaborator and no -1s then it can be merged but we first need to figure out a good model for doing minor version bumps and handling that with git

[Fishrock123]

@jden maybe you could send a PR for it?

[stevenvachon]

There's also relateurl which does its best to keep the url as short as possible, which is pretty well the only reason to use a relative url.

[sam-github]

-1

core url library isn't the be-all of url manipulations, its not even spec conformant! See nodejs/node-convergence-archive#49 (EDIT: pointed to correct issue 49)

url has minimal set of url features required to implement node itself, I don't think we should add to it, leave it to user-land.