Feature #8640
Have the mirror pool dispatcher library audited
100%
Description
Related issues
Blocked by Tails - |
Resolved | 2015-01-09 | 2016-04-15 |
History
#1 Updated by intrigeri 2015-01-09 16:45:54
- blocked by
Feature #8639: Write a mirror pool dispatcher script added
#2 Updated by intrigeri 2015-01-09 16:49:10
- blocks
Feature #8641: Have the mirror pool dispatcher script deployed on tails.b.o added
#3 Updated by Dr_Whax 2015-01-10 10:22:55
I can help out with this, maybe jvoisin is also interested in this.
#4 Updated by intrigeri 2015-01-10 15:26:36
> I can help out with this, maybe jvoisin is also interested in this.
Cool. Would be good to find someone who has a good knowledge of potential security pitfalls in PHP.
#5 Updated by intrigeri 2015-05-28 15:17:40
- blocks #8668 added
#6 Updated by intrigeri 2015-05-28 15:18:16
- Target version changed from Sustainability_M1 to Tails_2.2
#7 Updated by intrigeri 2015-08-02 09:56:55
- Due date set to 2016-04-15
#8 Updated by Anonymous 2015-09-26 14:55:24
- Subject changed from Have the server-side mirror pool dispatcher script audited to Have the mirror pool dispatcher script audited
We decided to implement this feature differently, changing the title accordingly.
#9 Updated by intrigeri 2016-02-13 15:47:28
- blocked by deleted (
)Feature #8641: Have the mirror pool dispatcher script deployed on tails.b.o
#10 Updated by Anonymous 2016-03-18 23:32:51
- Target version changed from Tails_2.2 to Tails_2.3
#12 Updated by Anonymous 2016-04-06 08:37:54
The script lives here: https://git-tails.immerda.ch/mirror-pool-dispatcher/tree/lib/js/mirror-dispatcher-promise.js
There is a test HTML page: https://git-tails.immerda.ch/mirror-pool-dispatcher/tree/lib/js/mirror-dispatcher-web.html
And the mirror.json file: https://git-tails.immerda.ch/mirror-pool-dispatcher/tree/lib/js
#13 Updated by geb 2016-04-06 10:24:53
Hi,
The code looks great for me. If I may, I would like to do a few suggests, most (if not any) of them are cosmetic ones
- max_weight, url_fallback and mirrors.json could be global variable wrote in CAPS on top of the code for readability.
- getRandomMirrorUrlPrefix() could be split in two functions, one that generate the new array, one that select it.
* get(path) could maybe be avoided by using