Feature #11204

Make sure we get all the benefits from TorBirdy's own account wizard

Added by anonym 2016-03-09 04:22:59 . Updated 2016-05-24 20:15:24 .

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Target version:
Start date:
2016-03-09
Due date:
% Done:

100%

Feature Branch:
feature/6154-secure-autoconfig-in-icedove
Type of work:
Code
Blueprint:

Starter:
Affected tool:
Email Client
Deliverable for:
268

Description

Once we enable the automatic configuration wizard, we will lose out from some of the features of TorBirdy’s wizard:

  • Disables check_new_mail so mail is not fetched at some regular interval (every 10 minutes by default).
  • Make sure that drafts are saved to Local Folders if it is an IMAP account. This is extremely important! When Enigmail is used, drafts will be encrypted with user’s own key, but only from the point where the user makes it clear that the email should be encrypted. If the user decides this late, the plaintext would leak through any drafts.
  • Do not check for new messages at startup.
  • Do not automatically download new messages for POP, which I’m not entirely sure what it does (see download_on_biff here.

While I believe some of these can be fixed with prefs, some (e.g. the drafts thing) might require some fixes in TorBirdy,


Subtasks


History

#1 Updated by anonym 2016-03-09 04:28:01

anonym wrote:
> * Disables check_new_mail so mail is not fetched at some regular interval (every 10 minutes by default).

Note to self: or is this about IMAP’s auto-sync thing? I don’t think so, but that feature (which I believe is enabled by default) is also worth looking at even if TorBirdy doesn’t do anything about it.

#2 Updated by anonym 2016-03-09 04:28:22

  • Subject changed from Make sure we get the benefits from TorBirdy's own account wizard to Make sure we get all the benefits from TorBirdy's own account wizard

#3 Updated by intrigeri 2016-03-09 09:06:26

  • Deliverable for set to 268

#4 Updated by anonym 2016-03-10 02:06:16

  • Status changed from Confirmed to In Progress
  • % Done changed from 0 to 30
  • Feature Branch set to feature/6154-secure-autoconfig-in-icedove
  • Type of work changed from Research to Code

I’ve addded a patch on the installed TorBirdy (commit:3622a24) in the feature branch that fixes this. Upstreaming next!

#5 Updated by anonym 2016-03-10 03:19:21

  • % Done changed from 30 to 20

Hrm. It seems that one simple thing does not work: when the automatic configuration is used, mail.server.%server_id%.login_at_startup is not set to false, so the newly added account will immediately fetch mail. That will also happen on Icedove startup. This does not happen (as intended) when TorBirdy’s account creation wizard is used.

I’ve tried every trick I know, but it remains set to true. I mean, at one point it is set like we want it, but something in the crazy callback stack when the automatic configuration is used must set this at some point. Argh.

#6 Updated by anonym 2016-03-17 13:55:11

  • Assignee deleted (anonym)
  • % Done changed from 20 to 40

So I managed to figure out the issue I was having problems with, yay! :) The current state of the post-secure-autoconfig branch in your repo is now ready for to be sent as a pull request.

Also, the Tails branch feature/6154-secure-autoconfig-in-icedove patches Torbirdy to the state of that, and you can test it, if you like, from an autobuild ISO here: http://nightly.tails.boum.org/build_Tails_ISO_feature-6154-secure-autoconfig-in-icedove/builds/lastSuccessfulBuild/archive/build-artifacts/

#7 Updated by Anonymous 2016-03-23 14:06:51

  • % Done changed from 40 to 50

Thanks! Created a pull request at https://github.com/ioerror/torbirdy/pull/30 and pinged the maintainers in private.

#8 Updated by Anonymous 2016-04-14 06:54:18

I’ve received a privaty reply that the review will be done soon.

#9 Updated by anonym 2016-05-08 05:10:23

  • Target version changed from Tails_2.3 to Tails_2.4

#10 Updated by Anonymous 2016-05-10 02:05:36

  • Assignee set to anonym

This has been merged upstream!

As said, I’ll package that for Debian as soon as a new upstream version has been released with this code.

Or, if we think it’s urgent, I can prepare Debian specific patches and drop them as soon as upstream releases a new version, what do you think?

#11 Updated by Anonymous 2016-05-10 02:10:37

  • QA Check set to Info Needed

#12 Updated by anonym 2016-05-13 06:48:24

  • Status changed from In Progress to Fix committed
  • Assignee deleted (anonym)
  • % Done changed from 50 to 100
  • QA Check changed from Info Needed to Pass

u wrote:
> This has been merged upstream!

Thanks a bunch for helping that happen! \o/

> As said, I’ll package that for Debian as soon as a new upstream version has been released with this code.

I can see that quite a few other things happened recently in torbirdy, so if a 0.1.5 is released, I’d like you to abuse your power as a package maintainer to make sure it’s not backported until after Tails 2.4 is released. I’d rather not have to deal with potential issues due to this late in the release process. :)

> Or, if we think it’s urgent, I can prepare Debian specific patches and drop them as soon as upstream releases a new version, what do you think?

I don’t care much — if you package a bump 0.1.4-1~bpo8+1 + my patches we’d get the exact same thing as we have now, so let’s not bother.

So I’m closing this ticket now since it’s upstreamed. If that’s a mistake, please re-open and clarify.

#13 Updated by anonym 2016-05-24 20:15:24

  • Status changed from Fix committed to Resolved