[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#18825: eww does not handle file:/file/name properly
From: |
Ivan Shmakov |
Subject: |
bug#18825: eww does not handle file:/file/name properly |
Date: |
Sat, 25 Oct 2014 12:17:50 +0000 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) |
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
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- bug#18825: eww does not handle file:/file/name properly,
Ivan Shmakov <=