Bug #11329
Mark all links on the website which should use the mirror pool
Start date:
2016-04-06
Due date:
% Done:
100%
Description
All links to the ISO image and signatures, as well as other files which should be downloaded from a mirror, would need to use the CSS class “use-mirror-pool”, so their URL prefix will be randomly modified.
Subtasks
Related issues
Blocks Tails - |
Resolved | 2015-01-09 | 2016-04-15 |
History
#1 Updated by Anonymous 2016-04-06 08:02:40
sajolida & anonym, I’ve added you as watchers, as you might edit the call for testing posts or other website links and might want to know about that feature.
#2 Updated by Anonymous 2016-04-06 08:02:53
- Status changed from New to In Progress
#3 Updated by Anonymous 2016-04-06 08:13:30
- Assignee set to intrigeri
- QA Check set to Ready for QA
- Feature Branch set to 451f:tails/feature/11329-use-mirror-pool
#4 Updated by intrigeri 2016-04-06 12:18:41
- Target version set to Tails_2.3
#5 Updated by intrigeri 2016-04-07 02:53:53
- Category set to Infrastructure
- Assignee deleted (
intrigeri) - % Done changed from 0 to 60
I did git grep 'dl\.amnesia\.boum\.org' -- wiki/src
and looked at occurrences that could have been overlooked. Let’s check each of those:
wiki/src/inc/stable_i386_iso_url.html
has the ISO’s URL, and is used by:wiki/src/install/expert/usb.mdwn
, for the wget command; it’s not a link so it’s non-trivial to tweak using JS, and that’s a corner case so I say no big deal if it uses the fallback DNS pool;wiki/src/install/inc/steps/download.inline.html
, that is itself used in a number of pages => I have no clue if that link is ever displayed in any real-world scenario, but to play it safe I added the class in commit:23afbcf;
wiki/src/contribute/how/testing.mdwn
=> I’ve fixed it in commit:fd0c7aa;wiki/src/local.css
: look for “dl.amnesia.boum.org” => in practice, if I got it right, we won’t match this anymore, and it means that we’re now going to mark links to our ISO image as “external”, since they won’t be on the list of exceptions we treat as internal links, right? I guess it’s no big deal, I just want to make sure we decide consciously, and based on a good understanding of the consequences :)
=> Please review my additional commits and sanity check my proposals. Then I’ll merge!
#6 Updated by intrigeri 2016-04-07 03:05:45
- Feature Branch changed from 451f:tails/feature/11329-use-mirror-pool to feature/11329-use-mirror-pool
#7 Updated by Anonymous 2016-04-07 03:20:41
- Assignee set to intrigeri
- QA Check changed from Ready for QA to Pass
yay.
#8 Updated by intrigeri 2016-04-07 03:49:52
- Status changed from In Progress to Resolved
- Assignee deleted (
intrigeri) - % Done changed from 60 to 100
Merged.
#9 Updated by intrigeri 2016-04-07 03:59:20
- blocks
Feature #8642: Enable the mirror pool dispatcher on all website pages that need it added
#10 Updated by intrigeri 2016-04-07 03:59:31
- Subject changed from Modify all links on the website which should use the mirror pool to Mark all links on the website which should use the mirror pool