2013-07-24 15:35:09 +02:00
|
|
|
|
As described on [Tails issue #5879](https://labs.riseup.net/code/issues/5879),
|
|
|
|
|
|
cgit wants URLs like:
|
|
|
|
|
|
|
|
|
|
|
|
https://git-tails.immerda.ch/tails/commit/wiki/src/todo/fix_localized_iceweasel_search_engine.mdwn?id=c682a361418bef7bce040ea2905691d54be192ab
|
|
|
|
|
|
|
|
|
|
|
|
... while ikiwiki generates links like:
|
|
|
|
|
|
|
|
|
|
|
|
https://git-tails.immerda.ch/tails/commit/wiki/src/todo%2Ffix_localized_iceweasel_search_engine.mdwn?id=c682a361418bef7bce040ea2905691d54be192ab
|
|
|
|
|
|
|
|
|
|
|
|
on the recentchanges page.
|
|
|
|
|
|
|
|
|
|
|
|
Would it make sense to add an option to skip URL-encoding when doing
|
2013-07-24 20:53:47 +02:00
|
|
|
|
the substitution of `\[[file]]` in `diffurl` and `historyurl`?
|
2013-07-24 15:35:09 +02:00
|
|
|
|
|
2013-07-24 15:37:27 +02:00
|
|
|
|
--[[intrigeri]]
|
2013-07-24 15:35:09 +02:00
|
|
|
|
|
2013-07-24 20:53:47 +02:00
|
|
|
|
> In `91b477c` I made a similar change to how the cvs backend encodes
|
|
|
|
|
|
> `diffurl`. Perhaps it applies to all VCS backends? --[[schmonz]]
|
|
|
|
|
|
|
2013-12-31 01:53:36 +01:00
|
|
|
|
>> 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]]
|
|
|
|
|
|
|
2013-07-24 15:35:09 +02:00
|
|
|
|
[[wishlist]]
|
