Issue metadata
Sign in to add a comment
|
blob should be added to the list of standard scheme |
||||||||||||||||||||||
Issue descriptionThe definition of GURL::IsStandard points to https://tools.ietf.org/html/rfc3986#section-3 which defines the generic URI syntax. It seems that blob confirms this definition. It has the same format as filesystem URLs, and filesystem is listed in kStandardURLSchemes in url_util.cc. We should add blob to StandardURLSchemes.
,
Jan 24 2017
,
Jan 24 2017
Please confirm with nick@ before landing a change here, since he did a lot of security work on blob URLs recently.
,
Jan 24 2017
nick: Are you also familiar with filesystem URLs? I'm wondering why filesystem is marked as SCHEME_WITHOUT_AUTHORITY in kStandardURLSchemes, given that filesystem URLs always have an associated origin (test page: http://meacer.com/filesystem.html).
,
Jan 25 2017
I'm fairly familiar with both blob and filesystem. blobs URLS have inner origins (which GURL, outside of the url::Origin code, doesn't really recognize), and filesystem URLs have inner URLs (which GURL does parse into an inner_url() member). My understanding of rfc3986 -- which may well be incorrect -- was that these inner origins / inner urls weren't authorities, but, they had authorities inside of them (since they embed standard schemes). Or put differently: you can't parse the authority out of a blob or filesystem URLs the same way you can an http, https, chrome, or chrome-extension URL. I'm generally in support of efforts to have GURL do more canonicalization of blob URLs. I think SCHEME_WITHOUT_AUTHORITY is probably appropriate for blobs, but it's not 100% clear to me quite what SCHEME_WITHOUT_AUTHORITY actually means.
,
Nov 10 2017
,
Feb 18 2018
,
Aug 23
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by mea...@chromium.org
, Jan 24 2017