[Webkit-unassigned] [Bug 106728] [KURL] 'https:' gets adjusted to 'https:/'

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jan 16 13:18:49 PST 2013


https://bugs.webkit.org/show_bug.cgi?id=106728





--- Comment #6 from Benjamin Poulain <benjamin at webkit.org>  2013-01-16 13:20:35 PST ---
(In reply to comment #5)
> (In reply to comment #4)
> > The bug is correct ("https:" is an invalid URL, and we should not attempt to canonicalize it), but the patch is wrong.
> 
> Could you please elaborate on 'https:' being an invalid URL?
> 
> I'm using RFC 3986 as a base, and that states:
> 
> "The scheme and path components are required, though the path may be
> empty (no characters).  When authority is present, the path must
> either be empty or begin with a slash ("/") character."
> http://tools.ietf.org/html/rfc3986#page-16
> 
> Basically, 'https:' consists of only a scheme and an empty path.
> Am I missing something?

Also read http://tools.ietf.org/html/rfc3986#section-6.2.3
And HTTP normalization.

RFC 3986 far from being the whole story, especially for the Web. You have subsequent spec defining error handling and normalization.

The following WhatWG spec is an attempt to clean things out (work in progress): http://url.spec.whatwg.org
(I actually see the "standard URL" parts has been removed since the last time I read it, so "https:" may be a valid URL nowadays...)

> > If you are interested in improving URL parsing, I suggest you to look into WTFURL.
> 
> I've looked around a bit about WTFURL, I find it interesting. It seems though it's not currently used by any port, so I'd like to fix this problem in the scope of the old KURL implementation first.

Because of the current mess in URL parsing, I have little interest in the current KURL. We really need to have a common implementation for all ports. That is why I suggested WTFURL.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the webkit-unassigned mailing list