]> git.eshelyaron.com Git - emacs.git/commitdiff
Make browse-url-encode-url encode even less
authorLars Ingebrigtsen <larsi@gnus.org>
Thu, 9 Jun 2022 15:11:54 +0000 (17:11 +0200)
committerLars Ingebrigtsen <larsi@gnus.org>
Thu, 9 Jun 2022 15:11:54 +0000 (17:11 +0200)
* 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

index 362dcf25b557782a6742a938eead687bacda5501..5b58c8ed86037e9d8c4558c20a47f193149174ec 100644 (file)
@@ -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