bug#18825: eww does not handle file:/file/name properly

[Ivan Shmakov]

Package:  emacs
Severity: minor
Tags:     patch

        The condition currently employed in ‘eww’ for file: scheme URIs
        is a bit more restrictive than necessary:

   156  ;;;###autoload
   157  (defun eww (url)
   158    "Fetch URL and render the page.
   159  If the input doesn't look like an URL or a domain name, the
   160  word(s) will be searched for via `eww-search-prefix'."
   161    (interactive "sEnter URL or keywords: ")
   162    (cond ((string-match-p "\\`file://" url))
   163          ((string-match-p "\\`ftp://"; url)
   164           (user-error "FTP is not supported."))

        While RFC 1738 does not allow for no-authority file: URIs,
        RFC 3986 makes the authority part optional /in general,/ and
        recognizing file:/name as synonymous to file:///name looks like
        a common extension over the RFC 1738 syntax.

        I thus suggest the condition to be relaxed to only include a
        single slash after ‘file:’.

-- 
FSF associate member #7257  http://boycottsystemd.org/  … 3013 B6A0 230E 334A

--- eww.el
+++ eww.el
@@ -159,7 +159,7 @@ eww (url)
 If the input doesn't look like an URL or a domain name, the
 word(s) will be searched for via `eww-search-prefix'."
   (interactive "sEnter URL or keywords: ")
-  (cond ((string-match-p "\\`file://" url))
+  (cond ((string-match-p "\\`file:/" url))
         ((string-match-p "\\`ftp://"; url)
          (user-error "FTP is not supported."))
         (t