From 835b431639c94489d92a79f6ce54731a0dd0e3e1 Mon Sep 17 00:00:00 2001 From: Lars Ingebrigtsen Date: Thu, 9 Jun 2022 17:11:54 +0200 Subject: [PATCH] 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 362dcf25b55..5b58c8ed860 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 -- 2.39.2