emacs-diffs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

master 835b431639: Make browse-url-encode-url encode even less


From: Lars Ingebrigtsen
Subject: master 835b431639: Make browse-url-encode-url encode even less
Date: Thu, 9 Jun 2022 11:12:33 -0400 (EDT)

branch: master
commit 835b431639c94489d92a79f6ce54731a0dd0e3e1
Author: Lars Ingebrigtsen <larsi@gnus.org>
Commit: Lars Ingebrigtsen <larsi@gnus.org>

    Make browse-url-encode-url encode even less
    
    * lisp/net/browse-url.el (browse-url-encode-url): Don't encode
    dollar signs, because that's out of spec (bug#55873).
---
 lisp/net/browse-url.el | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/lisp/net/browse-url.el b/lisp/net/browse-url.el
index 362dcf25b5..5b58c8ed86 100644
--- a/lisp/net/browse-url.el
+++ b/lisp/net/browse-url.el
@@ -635,9 +635,10 @@ CHARS is a regexp that matches a character."
 The annoying characters are those that can mislead a web browser
 regarding its parameter treatment."
   ;; FIXME: Is there an actual example of a web browser getting
-  ;; confused?  (This used to encode commas, but at least Firefox
-  ;; handles commas correctly and doesn't accept encoded commas.)
-  (browse-url-url-encode-chars url "[\"()$ ]"))
+  ;; confused?  (This used to encode commas and dollar signs, but at
+  ;; least Firefox handles commas correctly and doesn't accept those
+  ;; encoded.)
+  (browse-url-url-encode-chars url "[\"() ]"))
 
 ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
 ;; URL input



reply via email to

[Prev in Thread] Current Thread [Next in Thread]