X-Git-Url: http://git.vanrenterghem.biz/git.ikiwiki.info.git/blobdiff_plain/c3d31409c34e5088e88318cfdab0a62d8445d2f1..701ecfc2bb339d438d1138233e8a9250e580adf1:/doc/todo/support_linking_to_cgit.mdwn

diff --git a/doc/todo/support_linking_to_cgit.mdwn b/doc/todo/support_linking_to_cgit.mdwn
index bb408231c..5c61ff42f 100644
--- a/doc/todo/support_linking_to_cgit.mdwn
+++ b/doc/todo/support_linking_to_cgit.mdwn
@@ -10,8 +10,23 @@ cgit wants URLs like:
 on the recentchanges page.
 
 Would it make sense to add an option to skip URL-encoding when doing
-the substitution of `[[file]]` in `diffurl` and `historyurl`?
+the substitution of `\[[file]]` in `diffurl` and `historyurl`?
 
---intrigeri
+--[[intrigeri]]
+
+> In `91b477c` I made a similar change to how the cvs backend encodes
+> `diffurl`. Perhaps it applies to all VCS backends? --[[schmonz]]
+
+>> Thanks a lot! I've not looked at it yet, but will try to do so
+>> soonish, as this bug is a real problem for Tails when welcoming potential
+>> new contributors.
+>> 
+>> Also, note that this does not affect recentchanges only:  we have
+>> the same problem for URLs generated by the shortcut plugin, e.g.
+>> `\[[!tails_gitweb config/chroot_local-includes/usr/local/sbin/unsafe-browser]]`
+>> in a page (with the shortcut defined as
+>> `\[[!shortcut name=tails_gitweb url="https://git-tails.immerda.ch/tails/plain/%s"]]`)
+>> produces URLs with `/` encoded as `%2F`, which fails the same way
+>> in combination with cgit. --[[intrigeri]]
 
 [[wishlist]]