Feature #15705

Review current state of feature/15023-tor-browser-8

Added by intrigeri 2018-07-03 13:21:05 . Updated 2018-08-08 14:30:03 .

Status:
Resolved
Priority:
Elevated
Assignee:
Category:
Target version:
Tails_3.9
Start date:
2018-08-05
Due date:
% Done:

100%

Feature Branch:
Type of work:
Code
Blueprint:

Starter:
Affected tool:
Browser
Deliverable for:

Description

Subtasks

Related issues

Blocks Tails - Feature #15334: Core work 2018Q3: Foundations Team Resolved 2018-02-20

History

#1 Updated by intrigeri 2018-07-03 13:22:40

  • Estimated time set to 4 h

#2 Updated by intrigeri 2018-07-03 13:22:58

#3 Updated by intrigeri 2018-07-10 21:28:54

I won’t be working on this branch until July 21 for sure, and most likely not until our sprint (except perhaps importing new alpha Tor Browser releases). So feel free to do this review whenever you want, starting from now. It’s quite a big set of changes (74 files changed, 431 insertions(+), 540 deletions(-)), most of them in areas of Tails that AFAIK you’re not familiar with (test suite, browser profile / omni.ja mangling), hence the big chunk of allocated time for this ticket.

It would be nice if you could do this review before our August sprint, so that I have work to do as soon as we get there: fixing the most important among the issues your review will have raised. But if higher priority stuff, such as your VeraCrypt work, makes this impossible, that’s OK: this will then be your first task during our sprint.

The remaining known issues are tracked as subtasks of Feature #15023.

#4 Updated by intrigeri 2018-08-08 08:11:42

  • Status changed from Confirmed to In Progress

#5 Updated by segfault 2018-08-08 14:12:33

  • Status changed from In Progress to Resolved
  • Assignee deleted (segfault)
  • % Done changed from 0 to 100

Reviewed up to commit 85392012d5dc8b5934286fa92a3f3c7e6829998c (the currently most recent one) on Feature #15023.

#6 Updated by intrigeri 2018-08-08 14:30:03

Thanks a lot for this excellent review :)