Bug #12005

Problems with DAVE manual tests

Added by anonym 2016-11-30 23:23:41 . Updated 2018-01-17 18:02:37 .

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

0%

Feature Branch:
Type of work:
Test
Blueprint:

Starter:
Affected tool:
Verification Extension
Deliverable for:

Description

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


Subtasks


Related issues

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

History

#1 Updated by anonym 2016-11-30 23:26:12

> * Visit <https://tails.boum.org/install/debian/usb> in a Firefox-based
> browser. Verify that you can install the Firefox Addon. Start
> downloading a Tails image and copy the used mirror URL.
> - The URL should only start with dl.amnesia.boum.org if Javascript
> is disabled in your browser. Otherwise it should contain a mirror
> URL from <https://tails.boum.org/mirrors.json>

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).
>
> * Test a disabled mirror (Possible only in FF > 51 because of
> <https://bugzilla.mozilla.org/show_bug.cgi?id=1275289>.)
> - Do not use Firefox over Tor.
> - To disable Firefox’s internal DNS cache, navigate to
> `about:config` and set these prefererences:
> * `network.dnsCacheExpirationGracePeriod = 0`
> * `network.dnsCacheExpiration = 0`
> * `network.dnsCacheEntries = 0`
> - To enable logging, in `about:config` add these preferences:
> * `extensions.dave@tails.boum.org.sdk.console.logLevel = “all”`
> * `extensions.sdk.console.logLevel = “all”`

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 2016-11-30 23:27:19

  • Assignee deleted (None)
  • 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 Anonymous 2016-12-01 09:33:52

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 Anonymous 2016-12-01 09:37:57

anonym wrote:
> > * Visit <https://tails.boum.org/install/debian/usb> in a Firefox-based
> > browser. Verify that you can install the Firefox Addon. Start
> > downloading a Tails image and copy the used mirror URL.
> > - The URL should only start with dl.amnesia.boum.org if Javascript
> > is disabled in your browser. Otherwise it should contain a mirror
> > URL from <https://tails.boum.org/mirrors.json>
>
> 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 2016-12-14 20:11:29

  • Target version changed from Tails_2.9.1 to Tails 2.10

#6 Updated by spriver 2017-01-13 16:18:28

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 2017-01-13 16:22:03

  • 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 2017-01-16 08:09:55

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

#9 Updated by intrigeri 2017-01-16 09:32:25

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 2017-01-16 09:35:39

> 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 2017-01-24 12:16:58

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 (commit: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 2017-01-24 20:48:53

  • Target version changed from Tails 2.10 to Tails_2.11

#13 Updated by anonym 2017-03-09 14:00:31

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

#14 Updated by intrigeri 2017-04-20 07:04:40

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

#15 Updated by intrigeri 2017-04-20 07:05:36

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

#16 Updated by intrigeri 2017-06-12 07:49:11

#17 Updated by intrigeri 2017-06-12 08:02:59

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

#18 Updated by intrigeri 2017-06-12 15:52:53

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 (commit: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 (commit: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 2017-06-12 22:48:51

  • Status changed from Confirmed to In Progress

Applied in changeset commit:86c3b4da40e4e11c1829767802c37547bbf28de7.

#20 Updated by anonym 2017-07-06 14:15:04

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

#21 Updated by intrigeri 2017-09-07 08:23:45

  • 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.

#22 Updated by intrigeri 2017-11-06 16:04:47

  • Target version changed from Tails_3.3 to Tails_3.5

Let’s do this once DAVE v2 is out.

#23 Updated by intrigeri 2017-11-18 10:55:16

  • Affected tool changed from Download and Verification Extension to Verification Extension

#24 Updated by anonym 2017-12-14 14:31:32

  • Assignee changed from anonym to sajolida
  • QA Check deleted (Info Needed)

I removed the DAVE1 manual tests in commit:238cb3e008ac220f57395fd76c9588d7c7bc4e18 since they are irrelevant now.

#25 Updated by sajolida 2018-01-15 16:07:32

#26 Updated by sajolida 2018-01-17 18:02:21

  • related to Feature #15188: Write manual tests for Tails Verification added

#27 Updated by sajolida 2018-01-17 18:02:38

  • Status changed from In Progress to Resolved
  • Assignee deleted (sajolida)

I understand that the problems are gone now with 238cb3e008.

See you on Feature #15188!