GNU bug report logs - #55873
28.1; browse-url-encode-url escaped dollar symbol

Previous Next

Full log

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: urugang <urugang <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 28.1; browse-url-encode-url escaped dollar symbol
Date: Thu, 09 Jun 2022 23:04:37 +0800

(browse-url
"https://www.javadoc.io/doc/com.lightbend.akka/akka-stream-alpakka-amqp_2.13/latest/akka/stream/alpakka/amqp/javadsl/AmqpSource$.html")

It opens url
"https://www.javadoc.io/doc/com.lightbend.akka/akka-stream-alpakka-amqp_2.13/latest/akka/stream/alpakka/amqp/javadsl/AmqpSource%24.html"
which escaped dollor symbol "$" to "%24".
firefox/chrome/safari all recognize dollor symbol ("$") and escaped char ("%24")
as different things.

I found browse-url-url-encode-chars replace "$" as "%24", which is not
correct.

(defun browse-url-encode-url (url)
 (browse-url-url-encode-chars url "[\"()$ ]"))
(defun browse-url-url-encode-chars (text chars)
  (replace-regexp-in-string chars
                            (lambda (s)
                              (format "%%%X" (string-to-char s)))
                            text))

So I think we should remove escape encode of dollar symbol ("$") as below.

(defun browse-url-encode-url (url)
 (browse-url-url-encode-chars url "[\"() ]"))

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.