Bug #16608

Disable the topIcons GNOME Shell extension in one Tails release

Added by intrigeri 2019-03-23 17:37:50 . Updated 2019-05-23 07:36:13 .

Status:
Resolved
Priority:
Normal
Assignee:
segfault
Category:
Target version:
Start date:
2019-03-23
Due date:
% Done:

100%

Feature Branch:
bugfix/16608-disable-topicons
Type of work:
Code
Blueprint:

Starter:
Affected tool:
Deliverable for:

Description

As suggested on Bug #8310#note-21, in order to learn how much our users rely on this extension and — transitively — on OpenPGP Applet, we should disable this extension for one Tails release. People could still enable it again with 1 command line:

gnome-shell-extension-tool --enable-extension=TopIcons@phocean.net

Besides, the OpenPGP Applet will still be available in the bottom-left semi-hidden sysAtray.

:sajolida: is watching this


Subtasks


Related issues

Related to Tails - Feature #16399: Write release notes for 3.14 Resolved 2019-01-29
Related to Tails - Bug #16709: Re-add the topIcons GNOME Shell extension Resolved
Blocks Tails - Feature #16209: Core work: Foundations Team Confirmed
Blocks Tails - Bug #8310: Convert OpenPGP Applet into a proper GNOME Shell extension In Progress 2016-11-16

History

#1 Updated by intrigeri 2019-03-23 17:37:59

#2 Updated by intrigeri 2019-03-23 17:38:06

  • blocks Bug #8310: Convert OpenPGP Applet into a proper GNOME Shell extension added

#3 Updated by intrigeri 2019-03-23 17:46:27

  • Description updated

#4 Updated by intrigeri 2019-03-23 17:46:35

  • Feature Branch set to bugfix/16608-disable-topicons

#5 Updated by intrigeri 2019-03-23 17:47:49

  • Status changed from Confirmed to In Progress

Applied in changeset commit:tails|542ce13375da7e6430b044886cd5b713a8f8ad2d.

#6 Updated by intrigeri 2019-03-23 18:20:11

I confirm that the command line in the ticket description is sufficient to enable topIcons and display OpenPGP Applet, on an ISO built from the topic branch.

#7 Updated by intrigeri 2019-03-24 05:43:46

  • Assignee changed from intrigeri to segfault
  • % Done changed from 0 to 50
  • QA Check set to Ready for QA

CI is happy.

@segfault, perhaps you could batch this with one or two of your other 3.14 reviews? (i.e. merge N branches into a local integration branch so you build & manually test only once)

#8 Updated by segfault 2019-03-24 12:26:12

I can do the review, but I don’t like the proposal, see Bug #8310#note-27.

#9 Updated by intrigeri 2019-03-24 13:32:10

> I can do the review, but I don’t like the proposal, see Bug #8310#note-27.

ACK, I’ll reply there.

#10 Updated by intrigeri 2019-04-08 10:01:09

#11 Updated by intrigeri 2019-04-08 10:06:01

  • Description updated

As per discussion on Bug #8310, please go ahead with the review & merge :)

#12 Updated by segfault 2019-04-12 19:41:03

  • Assignee changed from segfault to intrigeri
  • QA Check changed from Ready for QA to Dev Needed

The commit message is not accurate:

[...] let's disable this extension for one Tails release. Users who need it can still enable it again with 1 command line.

We don’t actually disable the extension, it’s still accessible via the system tray in the bottom left corner. As a result, users also don’t need to use the command line to enable it again. So I propose we change the commit message.

Beside that, the change LGTM and seems to work.

#13 Updated by intrigeri 2019-04-13 05:21:42

  • Assignee changed from intrigeri to segfault
  • QA Check changed from Dev Needed to Ready for QA

> We don’t actually disable the extension,

Well, this branch does disable the TopIcons extension.

> it’s still accessible via the system tray in the bottom left corner
> As a result, users also don’t need to use the command line to enable it again.

These two statements are correct with “it” "OpenPGP Applet", but incorrect if "it" “the TopIcons extension”.

> So I propose we change the commit message.

ACK, done! (The fact we’re having the above discussion makes it clear that there was some misunderstanding so indeed, it’s worth updating this commit message, that predated our noticing that OpenPGP Applet was still available in the systray.)

@segfault, please review and merge :)

#14 Updated by segfault 2019-04-13 06:45:10

  • Status changed from In Progress to Fix committed
  • % Done changed from 50 to 100

Applied in changeset commit:tails|774b3f6090b56a2640ea5efb4f1fee21c818e8b1.

#15 Updated by segfault 2019-04-13 06:47:57

  • Status changed from Fix committed to In Progress
  • % Done changed from 100 to 50
  • QA Check changed from Ready for QA to Pass

intrigeri wrote:
> > We don’t actually disable the extension,
>
> Well, this branch does disable the TopIcons extension.
>
> > it’s still accessible via the system tray in the bottom left corner
> > As a result, users also don’t need to use the command line to enable it again.
>
> These two statements are correct with “it” "OpenPGP Applet", but incorrect if "it" “the TopIcons extension”.

Right, I misunderstood.

#16 Updated by segfault 2019-04-13 06:48:13

  • Status changed from In Progress to Fix committed
  • % Done changed from 50 to 100

#17 Updated by intrigeri 2019-05-05 08:23:50

  • Target version changed from Tails_3.14 to Tails_3.13.2

#18 Updated by anonym 2019-05-06 15:00:21

  • Status changed from Fix committed to Resolved

#19 Updated by anonym 2019-05-06 15:03:10

  • Target version changed from Tails_3.13.2 to Tails_3.14

#20 Updated by intrigeri 2019-05-06 18:15:35

  • Target version changed from Tails_3.14 to Tails_3.13.2

#21 Updated by goupille 2019-05-07 18:11:12

Message-ID: <6df579861634090ec20231b0573ec7cb.squirrel@giyzk7o6dcunb2ry.onion>

#22 Updated by goupille 2019-05-07 18:28:05

Message-ID: <uvzkKngdj_Bz3d6SGHTKTKi5XuNpyBDg4CdcmbHCvneNnlE3gb7TvpmykZpoyimaZzmIZNA_zxGY8ahWTIdWFSj183oZ6yVCKg9ELt2x7Lg=@protonmail.com>

#23 Updated by goupille 2019-05-07 18:36:10

Bug report: 4540eede3f658376d7a95963de18f9e0

#24 Updated by goupille 2019-05-07 18:41:49

Bug report: f8341e300152d317e435e9a25bbc2801

#25 Updated by goupille 2019-05-07 18:51:00

Bug report: fa3f7ffb38cbabdee53b8ed032a543dd

#26 Updated by goupille 2019-05-07 18:54:39

Bug report: 104b6546e87e9fe5e9cf149c9da90943

#27 Updated by goupille 2019-05-07 19:18:20

Message-ID: <evhmvgR7Hu_hHvJCO8OG8iCF6_sR3jfBFJHtLRhGxl_NGlNMRnKLAzyxsMnzy1Ll6xTVSZY9eDhvEzR9hgtYMzXLe7xfumK5Zgy9UJUAlcM=@protonmail.com>

#28 Updated by goupille 2019-05-07 19:20:12

Message-ID: <hVTU9xTrboIeRKvq81bbwN6VFnhljwIClfAIR1CC4OB-kC7akyAJUKE6oJvopJ61PKGbpOeJLY6bmE3zgjn16os3WdfFzfgCHUi7mE0XNS0=@protonmail.com>

#29 Updated by intrigeri 2019-05-08 08:18:00

  • related to Bug #16709: Re-add the topIcons GNOME Shell extension added

#30 Updated by intrigeri 2019-05-08 08:21:22

@goupille, thank you!

(Hint: use wb://$ID so it’s easier to search for such data with programs :)

#31 Updated by sajolida 2019-05-08 09:59:32

  • Description updated

#32 Updated by goupille 2019-05-10 19:44:26

Bug report: 1f13cc0fbea0454d45f9a6dead60d51d

@intrigeri : I think I’d need a hint to understand your hint :)

#33 Updated by goupille 2019-05-10 19:59:51

Bug report: 45131749846d7d2315170f082f86f7b3

#34 Updated by goupille 2019-05-11 00:24:03

Bug report: 6472484b2a667461b7a3766f1df6a9a7

Message-ID: <23a5748242a1b82efb925614f3840fdb@riseup.net>

Bug report: 9b6dc2d4230b69d9cce7982a985565a9

#35 Updated by goupille 2019-05-13 17:55:23

Bug report: 80e6cadd30861f7b2e83e889ee08ff37
Message-ID: <57c7b193611a6e70d2199a90f188db3d@riseup.net>
Message-ID: <CAHoOqk=1=VEpPFf8s1gDPOFuY6u3FTfxFZjTVmHZZcjT3DtmRw@mail.gmail.com>
Bug report: 929e2ca4292f579931031ed890d75028
Message-ID: <1uEbHUwu11sPaa4dKwNcH7LFH26nH6djBD_1EtSjsuJ8fCbaZul-8lrvr687Y7tpDUC-rzowO1zL95ypF3mvuTCome7gL9ZONBXL0IaVudY=@protonmail.com>

#36 Updated by intrigeri 2019-05-14 12:49:08

> intrigeri : I think I’d need a hint to understand your hint :)

@goupille, sure: instead of writing “Bug report: 1f13cc0fbea0454d45f9a6dead60d51d” (or random other formats used by other people), let’s standardize or a URI-like scheme, which in this example would be wb://1f13cc0fbea0454d45f9a6dead60d51d.

#37 Updated by goupille 2019-05-14 20:16:03

@intrigeri: ok, I don’t quite understand the why, but I can write it that way for the whisperback reports…

@sajolida: here are the last ones from my shift :

wb://84da32cd623c608004fc48ca620438f0
wb://160152478768a52c6f0f3d6d638c127d
wb://8f5d5a9a40480c5a844ef74593432f89

#38 Updated by mercedes508 2019-05-15 12:31:25

c240047e8d84f927715d688a93fa909c.squirrel@giyzk7o6dcunb2ry.onion
wb:// e4bfdfa212a5fd29bb660d69619437ff

#39 Updated by sajolida 2019-05-15 13:11:02

“Bug report: 80e6cadd30861f7b2e83e889ee08ff37” is an email subject header while “23a5748242a1b82efb925614f3840fdb@riseup.net” is a message id. My mail client (“Thunderbird”) has good support for search, filter, and threads based on the subject but no built-in support for message id. So actually, the way goupille has been doing it (subject header for WhisperBack report and message-id for the rest) is the most efficient for me.

#40 Updated by mercedes508 2019-05-15 14:32:46

sajolida wrote:
> “Bug report: 80e6cadd30861f7b2e83e889ee08ff37” is an email subject header while “23a5748242a1b82efb925614f3840fdb@riseup.net” is a message id. My mail client (“Thunderbird”) has good support for search, filter, and threads based on the subject but no built-in support for message id. So actually, the way goupille has been doing it (subject header for WhisperBack report and message-id for the rest) is the most efficient for me.

That’s exactly what I’m doing.

#41 Updated by intrigeri 2019-05-18 10:28:40

> My mail client (“Thunderbird”) has […] no built-in support for message id.

I believe it does: https://superuser.com/questions/285736/quick-way-to-find-a-specific-email-in-thunderbird-inbox

#42 Updated by sajolida 2019-05-20 15:45:11

> I believe it does: https://superuser.com/questions/285736/quick-way-to-find-a-specific-email-in-thunderbird-inbox

It’s possible but not out-of-the-box.

#43 Updated by intrigeri 2019-05-23 07:36:13

> It’s possible but not out-of-the-box.

I’m under the impression that there’s a misunderstanding.
I’ve followed up with sajolida privately to try to clear it up :)