d9/9b6a1ab54c7bc03cef5d09f14116df68467317

   1 Return-Path: <david@tethera.net>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5  by arlo.cworth.org (Postfix) with ESMTP id B1C156DE17E3\r
   6  for <notmuch@notmuchmail.org>; Sat, 27 Feb 2016 05:14:49 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at cworth.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -0.036\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.036 tagged_above=-999 required=5\r
  12  tests=[AWL=-0.025, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01]\r
  13  autolearn=disabled\r
  14 Received: from arlo.cworth.org ([127.0.0.1])\r
  15  by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16  with ESMTP id c0SvlHGR4cxy for <notmuch@notmuchmail.org>;\r
  17  Sat, 27 Feb 2016 05:14:47 -0800 (PST)\r
  18 Received: from fethera.tethera.net (fethera.tethera.net [198.245.60.197])\r
  19  by arlo.cworth.org (Postfix) with ESMTPS id 16F816DE12E8\r
  20  for <notmuch@notmuchmail.org>; Sat, 27 Feb 2016 05:14:47 -0800 (PST)\r
  21 Received: from remotemail by fethera.tethera.net with local (Exim 4.84)\r
  22  (envelope-from <david@tethera.net>)\r
  23  id 1aZeiU-0006kF-Ky; Sat, 27 Feb 2016 08:15:26 -0500\r
  24 Received: (nullmailer pid 3361 invoked by uid 1000);\r
  25  Sat, 27 Feb 2016 13:14:44 -0000\r
  26 From: David Bremner <david@tethera.net>\r
  27 To: Daniel Kahn Gillmor <dkg@fifthhorseman.net>,\r
  28  Notmuch Mail <notmuch@notmuchmail.org>\r
  29 Subject: Re: [PATCH v3 08/16] reorganize indexing of multipart/signed and\r
  30  multipart/encrypted\r
  31 In-Reply-To: <1454272801-23623-9-git-send-email-dkg@fifthhorseman.net>\r
  32 References: <1454272801-23623-1-git-send-email-dkg@fifthhorseman.net>\r
  33  <1454272801-23623-9-git-send-email-dkg@fifthhorseman.net>\r
  34 User-Agent: Notmuch/0.21+26~g9404723 (http://notmuchmail.org) Emacs/24.5.1\r
  35  (x86_64-pc-linux-gnu)\r
  36 Date: Sat, 27 Feb 2016 09:14:44 -0400\r
  37 Message-ID: <871t7y5m7v.fsf@zancas.localnet>\r
  38 MIME-Version: 1.0\r
  39 Content-Type: text/plain\r
  40 X-BeenThere: notmuch@notmuchmail.org\r
  41 X-Mailman-Version: 2.1.20\r
  42 Precedence: list\r
  43 List-Id: "Use and development of the notmuch mail system."\r
  44  <notmuch.notmuchmail.org>\r
  45 List-Unsubscribe: <https://notmuchmail.org/mailman/options/notmuch>,\r
  46  <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  47 List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
  48 List-Post: <mailto:notmuch@notmuchmail.org>\r
  49 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  50 List-Subscribe: <https://notmuchmail.org/mailman/listinfo/notmuch>,\r
  51  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  52 X-List-Received-Date: Sat, 27 Feb 2016 13:14:49 -0000\r
  53 \r
  54 Daniel Kahn Gillmor <dkg@fifthhorseman.net> writes:\r
  55 \r
  56 > This prepares the codebase for a cleaner changeset for dealing with\r
  57 > indexing some encrypted messages in the clear.\r
  58 > ---\r
  59 >  lib/index.cc | 39 +++++++++++++++++++--------------------\r
  60 >  1 file changed, 19 insertions(+), 20 deletions(-)\r
  61 >\r
  62 > diff --git a/lib/index.cc b/lib/index.cc\r
  63 > index f166aef..ab0fd78 100644\r
  64 > --- a/lib/index.cc\r
  65 > +++ b/lib/index.cc\r
  66 > @@ -333,27 +333,26 @@ _index_mime_part (notmuch_message_t *message,\r
  67 >       GMimeMultipart *multipart = GMIME_MULTIPART (part);\r
  68 >       int i;\r
  69 >  \r
  70 > -     if (GMIME_IS_MULTIPART_SIGNED (multipart))\r
  71 > -       _notmuch_message_add_term (message, "tag", "signed");\r
  72 > -\r
  73 > -     if (GMIME_IS_MULTIPART_ENCRYPTED (multipart))\r
  74 > -       _notmuch_message_add_term (message, "tag", "encrypted");\r
  75 > -\r
  76 > -     for (i = 0; i < g_mime_multipart_get_count (multipart); i++) {\r
  77 > -         if (GMIME_IS_MULTIPART_SIGNED (multipart)) {\r
  78 > -             /* Don't index the signature. */\r
  79 > -             if (i == 1)\r
  80 > -                 continue;\r
  81 > -             if (i > 1)\r
  82 > -                 _notmuch_database_log (_notmuch_message_database (message),\r
  83 > -                                       "Warning: Unexpected extra parts of multipart/signed. Indexing anyway.\n");\r
  84 > -         }\r
  85 > -         if (GMIME_IS_MULTIPART_ENCRYPTED (multipart)) {\r
  86 > -             /* Don't index encrypted parts. */\r
  87 > -             continue;\r
  88 > -         }\r
  89 > +     if (GMIME_IS_MULTIPART_SIGNED (multipart)) {\r
  90 > +         _notmuch_message_add_term (message, "tag", "signed");\r
  91 > +         /* FIXME: should we try to validate the signature? */\r
  92 > +         \r
  93 > +         /* FIXME: is it always just the first part that is signed in\r
  94 > +          all multipart/signed messages?*/\r
  95 >           _index_mime_part (message,\r
  96 > -                           g_mime_multipart_get_part (multipart, i));\r
  97 > +                           g_mime_multipart_get_part (multipart, 0));\r
  98 \r
  99 It took me a long moment to understand the change from "index part if not 1"\r
 100 to "index part 0". If that deserved explanation (which I'm not so certain\r
 101 about), it would be in the commit message.\r