From: Carsten Dominik <carsten.dominik@gmail.com>
To: Geert Kloosterman <g.j.kloosterman@gmail.com>
Cc: emacs-orgmode@gnu.org
Subject: Re: Bug: org-make-link-string incorrect with URL containing escapes [6.34a]
Date: Sat, 16 Jan 2010 01:11:40 +0100 [thread overview]
Message-ID: <02C88D47-85C1-4E21-B53E-3128FF7B0FB4@gmail.com> (raw)
In-Reply-To: <1263562863.5459.129.camel@localhost.localdomain>
Hi Geert,
wow, this was awesome! This is the best kind of bug report I can
imagine.
Thank you, your patch has been applied.
- Carsten
On Jan 15, 2010, at 2:41 PM, Geert Kloosterman wrote:
> Emacs : GNU Emacs 23.1.1 (i386-redhat-linux-gnu, GTK+ Version 2.18.3)
> of 2009-12-02 on x86-7.fedora.phx.redhat.com
> Package: Org-mode version 6.34a
>
> Hi all,
>
> When an org link is created from an URL containing a hex escape
> `org-make-link-string' creates a link that ends up corrupted the
> moment
> it is followed (e.g. using `org-open-at-point').
>
> I've traced this back to `org-link-escape' and `org-link-unescape'.
> The
> following shows how the hex code "%2B" is converted to a "+" after an
> escaping round trip:
>
> (org-link-unescape (org-link-escape "http://some.host.com/form?&id=blah%2Bblah
> "))
> ==>
> "http://some.host.com/form?&id=blah+blah"
>
> In my case this small change ended up in a broken URL.
>
> Additionally, when the URL-escape happens to be in lower case (or
> otherwise not present in `org-link-escape-chars') we end up with an
> error:
>
> (org-link-unescape (org-link-escape "http://some.host.com/form?&id=blah%2bblah
> "))
> ==>
> Debugger entered--Lisp error: (wrong-type-argument characterp nil)
> char-to-string(nil)
> ...
>
> When `org-url-encoding-use-url-hexify' is set to `t' we do get a
> proper
> round trip of the URL containing hex-escapes:
>
> (setq org-url-encoding-use-url-hexify t)
> (org-link-unescape (org-link-escape "http://some.host.com/form?&id=blah%2bblah
> "))
> ==>
> "http://some.host.com/form?&id=blah%2bblah"
>
>
> Setting `org-url-encoding-use-url-hexify' does not fix the complete
> problem however: `org-open-at-point' still did not end up with the
> proper URL. Within `org-open-at-point' there is another call to
> `org-link-escape':
>
> (org-link-escape path org-link-escape-chars-browser)
>
> This time a mapping table is passed in explicitly (the second
> argument).
> However, when `org-url-encoding-use-url-hexify' is set,a this mapping
> table isn't used, resulting (again) in a broken URL.
>
> I have attached a patch that fixes the problem: do not use url-
> hexify in
> `org-link-escape' and `org-link-unescape' when an explicit mapping
> table
> has been specified.
>
> In summary:
> - the default behaviour of `org-link-escape', with
> `org-url-encoding-use-url-hexify' set to nil, has some issues with
> handling URLS which contain url-encoded hex escapes
> - when a mapping table is passed to `org-link-escape' and
> `org-link-unescape', they should probably not use url-hexify.
> Patch attached.
>
> Best regards,
> Geert Kloosterman
>
>
> <org.el.diff>_______________________________________________
> Emacs-orgmode mailing list
> Please use `Reply All' to send replies to the list.
> Emacs-orgmode@gnu.org
> http://lists.gnu.org/mailman/listinfo/emacs-orgmode
- Carsten
prev parent reply other threads:[~2010-01-16 0:11 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-15 13:41 Bug: org-make-link-string incorrect with URL containing escapes [6.34a] Geert Kloosterman
2010-01-16 0:11 ` Carsten Dominik [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.orgmode.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=02C88D47-85C1-4E21-B53E-3128FF7B0FB4@gmail.com \
--to=carsten.dominik@gmail.com \
--cc=emacs-orgmode@gnu.org \
--cc=g.j.kloosterman@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).