Subject: bug#18825: closed (Re: bug#18825: eww does not handle file:/file/name properly)

From: [email-masked] (GNU bug Tracking System)
Date: Sun, 02 Nov 2014 21:09:02 +0000

Your bug report

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

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
18825: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=18825
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#18825: eww does not handle file:/file/name properly
From: Ted Zlatanov <[email-masked]>
To: Ivan Shmakov <[email-masked]>
Cc: [email-masked]
Date: Sun, 02 Nov 2014 16:08:24 -0500
List-Id: <debbugs-submit.debbugs.gnu.org>

On Sat, 25 Oct 2014 12:17:50 +0000 Ivan Shmakov <[email-masked]> wrote: 

IS> -  (cond ((string-match-p "\\`file://" url))
IS> +  (cond ((string-match-p "\\`file:/" url))

Thanks for the patch; applied in your name.  Closing.

Ted

Subject: eww does not handle file:/file/name properly
From: Ivan Shmakov <[email-masked]>
To: [email-masked]
Date: Sat, 25 Oct 2014 12:17:50 +0000
List-Id: <debbugs-submit.debbugs.gnu.org>

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