Project

General

Profile

Bug #12005

Problems with DAVE manual tests

Added by anonym 11 months ago. Updated about 1 month ago.

Status:
In Progress
Priority:
Elevated
Assignee:
Category:
-
Target version:
Start date:
11/30/2016
Due date:
% Done:

0%

QA Check:
Info Needed
Feature Branch:
Type of work:
Test
Blueprint:
Easy:
Affected tool:
Download and Verification Extension

Description

I'll post the relevant them below in a comment, since it's too long for a ticket description.


Related issues

Related to Tails - Feature #11799: Write manual tests for DAVE Resolved 09/15/2016
Related to Tails - Bug #12683: Move DAVE manual test suite out of the Tails release process Confirmed 06/12/2017

Associated revisions

Revision 86c3b4da (diff)
Added by intrigeri 4 months ago

Remove a DAVE test that seems invalid to me (refs: #12005).

The page that's being used for this
test (https://tails.boum.org/install/debian/usb/) is DAVE-enabled. It doesn't
display any link to the ISO, and DAVE picks a mirror itself, so that should work
regardless of JavaScript settings. So I'm not surprised that we've been reported
several time that this test doesn't pass :)

History

#1 Updated by anonym 11 months ago

First access with NoScript allowing boum.org: mirror with name in .de.
Second access in private mode, no boum.org in whitelist, name in tux33.fr
FF 51 (debian experimental), new profile, no preacceptance of boum.org: url in 24.dl.amnesia.boum.org
(I guess the extension keeps a cache and it bypass the private mode)
new empty profile with ff 0.51b from debian experimental: works only if adblock is disabled (allowing tails.boum.org is not enough)

- Verify that pausing and resuming the download from this URL works.
- Verify that when you start the download, you can see it appear in
the download list (Ctrl+Shift+Y).

Works!

- Then edit your `/etc/hosts` file to point the URL of the previously
used mirror to 127.0.0.1.
- Now reload the download page, and try to resume the download
again.

This does not work on 51.0~b1-1. The download will fail when trying to resume

- In the Firefox console (Ctrl+Shift+J) you should see the
`mirror.blob` variable pointing to a different mirror. This
should work.

Log entry after download fails:

    "Pagemod received ({downloadInfo:{url:"http://ftp.halifax.rwth-aachen.de/tails//stable/tails-i386-2.7/tails-i386-2.7.iso", path:"/home/nodens/T\xE9l\xE9chargements/Tails/tails-i386-2.7.iso", size:0, totalBytes:337, startTime:1480543331724, currentBytes:0, progress:0, hasProgress:true, stopped:true, canceled:false, succeeded:false, error:{name:"DownloadError", result:2152398880, message:"[Exception... \"It was attempted to resume the request, but the entity has changed in the meantime\"  nsresult: \"0x804b0020 (NS_ERROR_ENTITY_CHANGED)\"  location: \"JS frame :: resource://gre/modules/DownloadCore.jsm :: this.DownloadError :: line 1516\"  data: no]", becauseSourceFailed:true, becauseTargetFailed:false, stack:"this.DownloadError@resource://gre/modules/DownloadCore.jsm:1552:16\ntask_DCS_execute/backgroundFileSaver.observer.onSaveComplete@resource://gre/modules/commonjs/toolkit/loader.js -> resource://dave-at-tails-dot-boum-dot-org/lib/downloader.js line 50 > eval:72:42\n"}}, blob:{url:"http://ftp.halifax.rwth-aachen.de/tails//stable/tails-i386-2.7/tails-i386-2.7.iso", hash:"164dd41ddc3c1713df570e2379c35796edea93b2561afe82eb112252ec078651", size:"1167990784", version:"2.7", fallback_download_url_prefix:"http://dl.amnesia.boum.org/tails", url_suffix:"/stable/tails-i386-2.7/tails-i386-2.7.iso", mirrors:{version:1, mirrors:[{url_prefix:"https://tails.mirror.metalgamer.eu/tails/", weight:1, email:"metalgamer@c3l.lu"}, {url_prefix:"http://tails.mirror.iphh.net/tails/", weight:1, email:"repository@iphh.net"}, {url_prefix:"https://tails.bgadmin.com/tails/", weight:1, email:"root@bgadmin.com"}, {url_prefix:"https://tails.ybti.net/tails/", weight:1, email:"alexander@dietrich.cx"}, {url_prefix:"https://tails.c3l.lu/tails/", weight:1, email:"virii@c3l.lu"}, {url_prefix:"https://tails.interpipe.net/tails/", weight:1, email:"server-admin@interpipe.net"}, {url_prefix:"http://ftp.halifax.rwth-aachen.de/tails/", weight:1, notes:"stats: http://ftp.halifax.rwth-aachen.de/~cotto/logan
alyzer/", email:"ftp@halifax.rwth-aachen.de"}, {url_prefix:"http://13.dl.amnesia.boum.org/tails/", weight:1, email:"me@andreatartaglia.com"}, {url_prefix:"http://192.42.116.116/tails/", weight:1, email:"tails-mirror@as1101.net"}, {url_prefix:"https://28.dl.amnesia.boum.org/tails/", weight:1, notes:"stats (not only Tails mirror): https://dustri.org/vnstat/", email:"julien.voisin+tails[at]boum.org"}, {url_prefix:"http://17.dl.amnesia.boum.org/tails/", weight:1, email:"fr33tux+tails_mirror@general-changelog-team.fr"}, {url_prefix:"http://mirror.bytemark.co.uk/tails/", weight:1, email:"hello@bytemark.co.uk"}, {url_prefix:"https://dl1.crypto-rebels.de/tails/", weight:1, email:"spriver@autistici.org"}, {url_prefix:"http://tails.rlogin.net/tails/", weight:1, email:"tails@rlogin.net", notes:"stats: http://beam.rlogin.net/cgi-bin/awstats.pl?config=tails"}, {url_prefix:"https://tails.freiwuppertal.de/tails/", weight:1, email:"tobias@freiwuppertal.de"}, {url_prefix:"https://tails.void.gr/tails/", weight:1, email:"tails@void.gr"}, {url_prefix:"https://dl2.crypto-rebels.de/tails/", weight:1, email:"spriver@autistici.org"}, {url_prefix:"https://tails.hactar.xyz/tails/", weight:1, email:"root@hactar.xyz"}, {url_prefix:"https://tails.adamas.ai/tails/", weight:1, email:"virii@c3l.lu"}, {url_prefix:"https://tails.bl0m.de/tails/", weight:1, email:"webmaster@bl0m.de", notes:"stats: http://stats.tails.bl0m.de/"}, {url_prefix:"https://tails.fr33tux.org/tails/", weight:1, email:"fr33tux+tails_mirror@general-changelog-team.fr"}, {url_prefix:"https://tails.ph3x.at/tails/", weight:1, email:"hosting@ph3x.at"}, {url_prefix:"http://dl3.crypto-rebels.de/tails/", weight:1, email:"spriver@autistici.org", notes:"2016-07-10: Connection refused"}, {url_prefix:"https://tails.pplacic.com/tails/", weight:1, email:"admin@pplacic.com"}, {url_prefix:"https://tails.unixcorn.org/tails/", weight:0, email:"root@unixcorn.org", notes:"2016-05-27: Out-of-sync; 2016-09-11: NXDOMAIN"}, {url_prefix:"https://tails-dl.urown.net/tails/", weight:1, email:"tormaster@urown.net"}, {url_prefix:"http://amnesia.thetechstewart.com/tails/", weight:1, email:"amnesia-mirror@thetechstewart.com"}, {url_prefix:"http://24.dl.amnesia.boum.org/tails/", weight:1, email:"noc@babylon.network"}, {url_prefix:"http://25.dl.amnesia.boum.org/tails/", weight:1, email:"noc@babylon.network"}, {url_prefix:"https://mirrors.ocf.berkeley.edu/tails/", weight:1, email:"root@ocf.berkeley.edu", notes:"late during 2.4~rc1"}, {url_prefix:"https://mirrors.wikimedia.org/tails/", weight:1, email:"noc@wikimedia.org"}, {url_prefix:"https://ftp.fau.de/tails/", weight:1, email:"rrze-ftp-admins@fau.de", notes:"stats: https://ftp.fau.de/cgi-bin/show-ftp-stats.cgi, https://ftp.fau.de/cgi-bin/show-mirror-status.cgi?mirror=tails"}, {url_prefix:"https://mirrors.kernel.org/tails/", weight:1, email:"ftpadmin@kernel.org"}, {url_prefix:"http://ftp.nluug.nl/os/Linux/distr/tails/", weight:1, email:"ftp-admin@nluug.nl", notes:"stats: http://ftp.nluug.nl/.statistics/"}, {url_prefix:"https://tails.lasca.ic.unicamp.br/", weight:0, email:"pedro.alves@ic.unicamp.br", notes:"2016-09-11: No route to host"}, {url_prefix:"http://tails.mirrorservice.org/tails/", weight:1, email:"archive-admin@mirrorservice.org"}, {url_prefix:"https://tails.u-strasbg.fr/", weight:1, email:"ftpmaint@u-strasbg.fr"}, {url_prefix:"http://ftp.free.fr/mirrors/tails.boum.org/tails/", weight:1, email:"ftpmaint@proxad.net"}, {url_prefix:"http://ftp.wcss.pl/tails/", weight:0, email:"admin@wcss.pl", notes:"2016-08-01, 2016-09-12: outdated"}, {url_prefix:"https://mirror.netcologne.de/tails/tails/", weight:1, email:"mirror-service@netcologne.de"}, {url_prefix:"https://tails.mipnamic.net/tails/", weight:0, email:"mipnamic@mipnamic.net", notes:"2016-08-01, 2016-09-12: outdated"}, {url_prefix:"https://tails.freedif.org/tails/", weight:1, email:"karibu@freedif.org"}, {url_prefix:"https://26.dl.amnesia.boum.org/tails/", weight:0, email:"relays@riseup.net", notes:"stats: https://26.
dl.amnesia.boum.org/stats/tails.html, outdated SSL certificate 2016-11-17."}, {url_prefix:"https://mirrors.dotsrc.org/tails/", weight:1, email:"anders.trier.olesen@gmail.com"}, {url_prefix:"http://27.dl.amnesia.boum.org/tails/", weight:1, email:"noc@babylon.network"}, {url_prefix:"http://ftp.acc.umu.se/tails/", weight:1, email:"ftp-adm@acc.umu.se"}, {url_prefix:"https://repo.stdout.net/tails-mirror/tails/", weight:1, email:"kar@fsfe.org"}, {url_prefix:"https://onion.codingrat.eu/tails/", weight:1, email:"catatafish@mail.codingrat.eu"}]}}, phase:"downloading", verifyInfo:{progress:0, path:null, hash:null, computedHash:null}, failed:false, error:null})"  sandbox.js:340
tails-download-and-verify:Download state: failed  sandbox.js:340
A promise chain failed to handle a rejection. Did you forget to '.catch', or did you forget to 'return'?
See https://developer.mozilla.org/Mozilla/JavaScript_code_modules/Promise.jsm/Promise

Date: Wed Nov 30 2016 23:02:11 GMT+0100 (CET)
Full Message: DownloadError: [Exception... "It was attempted to resume the request, but the entity has changed in the meantime"  nsresult: "0x804b0020 (NS_ERROR_ENTITY_CHANGED)"  location: "JS frame :: resource://gre/modules/DownloadCore.jsm :: this.DownloadError :: line 1516"  data: no]
Full Stack: this.DownloadError@resource://gre/modules/DownloadCore.jsm:1552:16
task_DCS_execute/backgroundFileSaver.observer.onSaveComplete@resource://gre/modules/commonjs/toolkit/loader.js -> resource://dave-at-tails-dot-boum-dot-org/lib/downloader.js line 50 > eval:72:42
  (unknown)

New try after reboot to make sure no old version of a library was used, same pb, different exception (actually now it's the one in the mozilla bugreport). Note that download is not failed now, but is still marked as "going", just stalled. Works again when removing the hosts entry and pause/start again.

Log:

tails-download-and-verify:Sending command Download.resume  sandbox.js:340
tails-download-and-verify:({cmd:"resume", page:"https://tails.boum.org/install/debian/usb/", domain:"tails.boum.org"})  index.js:24
tails-download-and-verify:"Pagemod received ({downloadInfo:{url:"https://tails.pplacic.com/tails//stable/tails-i386-2.7/tails-i386-2.7.iso", path:"/home/nodens/T\xE9l\xE9chargements/Tails/tails-i386-2.7.iso", size:0, totalBytes:0, startTime:1480545048942, currentBytes:0, progress:0, hasProgress:false, stopped:false, canceled:false, succeeded:false, error:null}, blob:{url:"https://tails.pplacic.com/tails//stable/tails-i386-2.7/tails-i386-2.7.iso", hash:"164dd41ddc3c1713df570e2379c35796edea93b2561afe82eb112252ec078651", size:"1167990784", version:"2.7", fallback_download_url_prefix:"http://dl.amnesia.boum.org/tails", url_suffix:"/stable/tails-i386-2.7/tails-i386-2.7.iso", mirrors:{version:1, mirrors:[{url_prefix:"https://tails.mirror.metalgamer.eu/tails/", weight:1, email:"metalgamer@c3l.lu"}, {url_prefix:"http://tails.mirror.iphh.net/tails/", weight:1, email:"repository@iphh.net"}, {url_prefix:"https://tails.bgadmin.com/tails/", weight:1, email:"root@bgadmin.com"}, {url_prefix:"https://tails.ybti.net/tails/", weight:1, email:"alexander@dietrich.cx"}, {url_prefix:"https://tails.c3l.lu/tails/", weight:1, email:"virii@c3l.lu"}, {url_prefix:"https://tails.interpipe.net/tails/", weight:1, email:"server-admin@interpipe.net"}, {url_prefix:"http://ftp.halifax.rwth-aachen.de/tails/", weight:1, notes:"stats: http://ftp.halifax.rwth-aachen.de/~cotto/loganalyzer/", email:"ftp@halifax.rwth-aachen.de"}, {url_prefix:"http://13.dl.amnesia.boum.org/tails/", weight:1, email:"me@andreatartaglia.com"}, {url_prefix:"http://192.42.116.116/tails/", weight:1, email:"tails-mirror@as1101.net"}, {url_prefix:"https://28.dl.amnesia.boum.org/tails/", weight:1, notes:"stats (not only Tails mirror): https://dustri.org/vnstat/", email:"julien.voisin+tails[at]boum.org"}, {url_prefix:"http://17.dl.amnesia.boum.org/tails/", weight:1, email:"fr33tux+tails_mirror@general-changelog-team.fr"}, {url_prefix:"http://mirror.bytemark.co.uk/tails/", weight:1, email:"hello@bytemark.co.uk"}, {url_prefix:"https://d
l1.crypto-rebels.de/tails/", weight:1, email:"spriver@autistici.org"}, {url_prefix:"http://tails.rlogin.net/tails/", weight:1, email:"tails@rlogin.net", notes:"stats: http://beam.rlogin.net/cgi-bin/awstats.pl?config=tails"}, {url_prefix:"https://tails.freiwuppertal.de/tails/", weight:1, email:"tobias@freiwuppertal.de"}, {url_prefix:"https://tails.void.gr/tails/", weight:1, email:"tails@void.gr"}, {url_prefix:"https://dl2.crypto-rebels.de/tails/", weight:1, email:"spriver@autistici.org"}, {url_prefix:"https://tails.hactar.xyz/tails/", weight:1, email:"root@hactar.xyz"}, {url_prefix:"https://tails.adamas.ai/tails/", weight:1, email:"virii@c3l.lu"}, {url_prefix:"https://tails.bl0m.de/tails/", weight:1, email:"webmaster@bl0m.de", notes:"stats: http://stats.tails.bl0m.de/"}, {url_prefix:"https://tails.fr33tux.org/tails/", weight:1, email:"fr33tux+tails_mirror@general-changelog-team.fr"}, {url_prefix:"https://tails.ph3x.at/tails/", weight:1, email:"hosting@ph3x.at"}, {url_prefix:"http://dl3.crypto-rebels.de/tails/", weight:1, email:"spriver@autistici.org", notes:"2016-07-10: Connection refused"}, {url_prefix:"https://tails.pplacic.com/tails/", weight:1, email:"admin@pplacic.com"}, {url_prefix:"https://tails.unixcorn.org/tails/", weight:0, email:"root@unixcorn.org", notes:"2016-05-27: Out-of-sync; 2016-09-11: NXDOMAIN"}, {url_prefix:"https://tails-dl.urown.net/tails/", weight:1, email:"tormaster@urown.net"}, {url_prefix:"http://amnesia.thetechstewart.com/tails/", weight:1, email:"amnesia-mirror@thetechstewart.com"}, {url_prefix:"http://24.dl.amnesia.boum.org/tails/", weight:1, email:"noc@babylon.network"}, {url_prefix:"http://25.dl.amnesia.boum.org/tails/", weight:1, email:"noc@babylon.network"}, {url_prefix:"https://mirrors.ocf.berkeley.edu/tails/", weight:1, email:"root@ocf.berkeley.edu", notes:"late during 2.4~rc1"}, {url_prefix:"https://mirrors.wikimedia.org/tails/", weight:1, email:"noc@wikimedia.org"}, {url_prefix:"https://ftp.fau.de/tails/", weight:1, email:"rrze-ft
p-admins@fau.de", notes:"stats: https://ftp.fau.de/cgi-bin/show-ftp-stats.cgi, https://ftp.fau.de/cgi-bin/show-mirror-status.cgi?mirror=tails"}, {url_prefix:"https://mirrors.kernel.org/tails/", weight:1, email:"ftpadmin@kernel.org"}, {url_prefix:"http://ftp.nluug.nl/os/Linux/distr/tails/", weight:1, email:"ftp-admin@nluug.nl", notes:"stats: http://ftp.nluug.nl/.statistics/"}, {url_prefix:"https://tails.lasca.ic.unicamp.br/", weight:0, email:"pedro.alves@ic.unicamp.br", notes:"2016-09-11: No route to host"}, {url_prefix:"http://tails.mirrorservice.org/tails/", weight:1, email:"archive-admin@mirrorservice.org"}, {url_prefix:"https://tails.u-strasbg.fr/", weight:1, email:"ftpmaint@u-strasbg.fr"}, {url_prefix:"http://ftp.free.fr/mirrors/tails.boum.org/tails/", weight:1, email:"ftpmaint@proxad.net"}, {url_prefix:"http://ftp.wcss.pl/tails/", weight:0, email:"admin@wcss.pl", notes:"2016-08-01, 2016-09-12: outdated"}, {url_prefix:"https://mirror.netcologne.de/tails/tails/", weight:1, email:"mirror-service@netcologne.de"}, {url_prefix:"https://tails.mipnamic.net/tails/", weight:0, email:"mipnamic@mipnamic.net", notes:"2016-08-01, 2016-09-12: outdated"}, {url_prefix:"https://tails.freedif.org/tails/", weight:1, email:"karibu@freedif.org"}, {url_prefix:"https://26.dl.amnesia.boum.org/tails/", weight:0, email:"relays@riseup.net", notes:"stats: https://26.dl.amnesia.boum.org/stats/tails.html, outdated SSL certificate 2016-11-17."}, {url_prefix:"https://mirrors.dotsrc.org/tails/", weight:1, email:"anders.trier.olesen@gmail.com"}, {url_prefix:"http://27.dl.amnesia.boum.org/tails/", weight:1, email:"noc@babylon.network"}, {url_prefix:"http://ftp.acc.umu.se/tails/", weight:1, email:"ftp-adm@acc.umu.se"}, {url_prefix:"https://repo.stdout.net/tails-mirror/tails/", weight:1, email:"kar@fsfe.org"}, {url_prefix:"https://onion.codingrat.eu/tails/", weight:1, email:"catatafish@mail.codingrat.eu"}]}}, phase:"downloading", verifyInfo:{progress:0, path:null, hash:null, computedHash:null}, fai
led:false, error:null})"  sandbox.js:340
tails-download-and-verify:Download state: going  sandbox.js:340
NS_ERROR_NOT_AVAILABLE: Component returned failure code: 0x80040111 (NS_ERROR_NOT_AVAILABLE) [nsIHttpChannel.responseStatus]  downloader.js%20line%2050%20%3E%20eval:139
NS_ERROR_NOT_AVAILABLE: Component returned failure code: 0x80040111 (NS_ERROR_NOT_AVAILABLE) [nsIChannel.contentLength]  downloader.js%20line%2050%20%3E%20eval:217
Object { type: "COLLABROOM", data: Object }  pad.js:3327:25

#2 Updated by anonym 11 months ago

  • Assignee set to u
  • Affected tool set to Download and Verification Extension

It should be noted that it worked fine for me on Debian Sid, using Debian experimental's firefox 51.0~b1-1, which nodens too used.

What do you think, u?

#3 Updated by u 11 months ago

NS_ERROR_NOT_AVAILABLE: Component returned failure code: 0x80040111 (NS_ERROR_NOT_AVAILABLE) [nsIHttpChannel.responseStatus]  downloader.js%20line%2050%20%3E%20eval:139
NS_ERROR_NOT_AVAILABLE: Component returned failure code: 0x80040111 (NS_ERROR_NOT_AVAILABLE) [nsIChannel.contentLength]  downloader.js%20line%2050%20%3E%20eval:217

That's due to a bug in Firefox. (https://bugzilla.mozilla.org/show_bug.cgi?id=1275289)
According to https://hg.mozilla.org/mozilla-central/rev/0fc7e282d053 it should be fixed in 51.0a1, but I did not verify that commit is in there.

#4 Updated by u 11 months ago

anonym wrote:

First access with NoScript allowing boum.org: mirror with name in .de.
Second access in private mode, no boum.org in whitelist, name in tux33.fr
FF 51 (debian experimental), new profile, no preacceptance of boum.org: url in 24.dl.amnesia.boum.org
(I guess the extension keeps a cache and it bypass the private mode)

Yes there is caching of the mirror URL in the extension.

new empty profile with ff 0.51b from debian experimental: works only if adblock is disabled (allowing tails.boum.org is not enough)

That means: we need to modify the manual test instructions saying that AdBlock cannot be enabled, correct?

#5 Updated by anonym 10 months ago

  • Target version changed from Tails_2.9.1 to Tails 2.10

#6 Updated by spriver 9 months ago

With FF45.6esr

First access with NoScript allowing boum.org: mirror with name in .de.

Similar for me (just different mirror).

Second access in private mode, no boum.org in whitelist, name in tux33.fr

Same here.

FF 51 (debian experimental), new profile, no preacceptance of boum.org: url in 24.dl.amnesia.boum.org
(I guess the extension keeps a cache and it bypass the private mode)
new empty profile with ff 0.51b from debian experimental: works only if adblock is disabled (allowing tails.boum.org is not enough)

Apart from that, after deactivating JS with NoScript on the website, the download crashes directly after starting (no matter which URL is being used).

And I can also confirm with FF45.6esr:

NS_ERROR_NOT_AVAILABLE: Component returned failure code: 0x80040111 (NS_ERROR_NOT_AVAILABLE) [nsIHttpChannel.responseStatus]  downloader.js%20line%2050%20%3E%20eval:139

#7 Updated by anonym 9 months ago

  • Subject changed from Problems with DAVE manual tests during Tails 2.7.1 manual test session to Problems with DAVE manual tests manual test session
  • QA Check set to Info Needed

Spriver's results were from the Tails 2.10~rc1 manual test session, so the problems are persisting.

u wrote:

That means: we need to modify the manual test instructions saying that AdBlock cannot be enabled, correct?

I think that is one of the steps needed, but to me it seems that more stuff is broken. Can you please try the manual test steps yourself, and really make sure it works as intended?

#8 Updated by intrigeri 9 months ago

FWIW I've received reports that uBlock breaks DAVE. Didn't triage/forward them yet.

#9 Updated by intrigeri 9 months ago

spriver wrote:

Apart from that, after deactivating JS with NoScript on the website, the download crashes directly after starting (no matter which URL is being used).

Please report this as a separate bug. Thanks!

#10 Updated by intrigeri 9 months ago

FWIW I've received reports that uBlock breaks DAVE.

I'm told this can be reproduced:

  • even with all uBlock filters disabled
  • even toggling the 4 features of uBlock "for this site"

I've received this stack trace:

console.error: tails-download-and-verify: 
  Message: DownloadError: [Exception... "Component is not available"  nsresult: "0x80040111 (NS_ERROR_NOT_AVAILABLE)"  location: "JS frame :: resource://gre/modules/DownloadCore.jsm :: this.DownloadError :: line 1501"  data: no]
  Stack:
    this.DownloadError@resource://gre/modules/DownloadCore.jsm:1533:16
DCS_execute/task_DCS_execute/backgroundFileSaver.observer.onSaveComplete@resource://gre/modules/commonjs/toolkit/loader.js -> resource://dave-at-tails-dot-boum-dot-org/lib/downloader.js line 50 > eval:74:42

This part of our lib/downloader.js is monkey patching the Firefox download manager. I'm not sure if this code is still needed nowadays, so perhaps the best option would simply be to drop it (if that's enough to fix the bug). Maybe uBlock Origin is doing something similar in a way that conflicts?

Next step: document simple steps to reproduce and ask Giorgio what's going on, and if we can drop the monkeypatching.

#11 Updated by anonym 9 months ago

During the Tails 2.10 manual test sessuin:

  • I noticed that "The URL should only start with dl.amnesia.boum.org if Javascript is disabled in your browser" only seem to work reliably if the browser is restarted, so I added that to the test instructions (142dea45)
  • I also added two questions/requests for clarifications (see `XXX`:es) to the test instructions for "Test a disabled mirror". I think I got how to do this test the first time (when I merged it) but now I don't remember how I did it, and cannot make sense out of it. Or maybe I did it wrong that first time as well. :S

#12 Updated by anonym 9 months ago

  • Target version changed from Tails 2.10 to Tails_2.11

#13 Updated by anonym 8 months ago

  • Target version changed from Tails_2.11 to Tails_2.12

#14 Updated by intrigeri 6 months ago

  • Subject changed from Problems with DAVE manual tests manual test session to Problems with DAVE manual tests

#15 Updated by intrigeri 6 months ago

  • Assignee changed from u to anonym
  • Target version changed from Tails_2.12 to Tails_3.1

#16 Updated by intrigeri 4 months ago

#17 Updated by intrigeri 4 months ago

  • Related to Bug #12683: Move DAVE manual test suite out of the Tails release process added

#18 Updated by intrigeri 4 months ago

anonym wrote:

  • I noticed that "The URL should only start with dl.amnesia.boum.org if Javascript is disabled in your browser" only seem to work reliably if the browser is restarted, so I added that to the test instructions (142dea45)

I think this specific test is invalid, and I don't understand how it can possibly have ever worked: the page that's being used for this test (https://tails.boum.org/install/debian/usb/) is DAVE-enabled. It doesn't display any link to the ISO, and DAVE runs its own JS in order to pick a mirror for the download itself, so that should work regardless of the browser's JavaScript settings. I've thus removed the bits that seemed wrong to me (86c3b4da40e4e11c1829767802c37547bbf28de7). Sorry if I misunderstood something!

I guess the intention behind this test was to validate the JavaScript code we use on other parts of our website to pick a mirror; such code is run just like any other web active content, and not by an add-on, so it is affected by the browser's JS settings. Such a test could be valuable. This could be tested on another, non-DAVE-enabled download page, such as https://tails.boum.org/install/download/openpgp/; but then it's not testing DAVE, so if we reintroduce this part of the test, then it should be moved to another section.

#19 Updated by intrigeri 4 months ago

  • Status changed from Confirmed to In Progress

#20 Updated by anonym 4 months ago

  • Target version changed from Tails_3.1 to Tails_3.2

#21 Updated by intrigeri about 1 month ago

  • Priority changed from Normal to Elevated
  • Target version changed from Tails_3.2 to Tails_3.3

I suggest you ignore DAVE until the 3.2 release and then put it on your list of top priorities for the 3.3 cycle.

Also available in: Atom PDF