6a/8dbca179d3d9e17fcae220d86d0ea7708ece1c

   1 Return-Path: <BATV+fd0185dc9217407f92b2+2423+infradead.org+hohndel@bombadil.srs.infradead.org>\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 olra.theworths.org (Postfix) with ESMTP id 91A074196F3\r
   6         for <notmuch@notmuchmail.org>; Mon, 12 Apr 2010 11:44:38 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -4.2\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-4.2 tagged_above=-999 required=5\r
  12         tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3] autolearn=ham\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id NFg2j29EiPOn for <notmuch@notmuchmail.org>;\r
  16         Mon, 12 Apr 2010 11:44:37 -0700 (PDT)\r
  17 Received: from bombadil.infradead.org (bombadil.infradead.org [18.85.46.34])\r
  18         by olra.theworths.org (Postfix) with ESMTP id CB33C431FC1\r
  19         for <notmuch@notmuchmail.org>; Mon, 12 Apr 2010 11:44:37 -0700 (PDT)\r
  20 Received: from localhost ([::1] helo=localhost.localdomain)\r
  21         by bombadil.infradead.org with esmtp (Exim 4.69 #1 (Red Hat Linux))\r
  22         id 1O1OcZ-0006rl-Ke; Mon, 12 Apr 2010 18:44:32 +0000\r
  23 Received: by localhost.localdomain (Postfix, from userid 500)\r
  24         id 0C762C005D; Mon, 12 Apr 2010 08:53:15 -0700 (PDT)\r
  25 From: Dirk Hohndel <hohndel@infradead.org>\r
  26 To: Tomas Carnecky <tom@dbservice.com>\r
  27 Subject: Re: Use of strcasestr\r
  28 In-Reply-To: <4BC326F2.10300@dbservice.com>\r
  29 References: <4BC326F2.10300@dbservice.com>\r
  30 Date: Mon, 12 Apr 2010 08:53:14 -0700\r
  31 Message-ID: <m3wrwc651x.fsf@x200.gr8dns.org>\r
  32 MIME-Version: 1.0\r
  33 Content-Type: text/plain; charset=us-ascii\r
  34 X-SRS-Rewrite: SMTP reverse-path rewritten from <hohndel@infradead.org> by\r
  35         bombadil.infradead.org See http://www.infradead.org/rpr.html\r
  36 Cc: notmuch <notmuch@notmuchmail.org>\r
  37 X-BeenThere: notmuch@notmuchmail.org\r
  38 X-Mailman-Version: 2.1.13\r
  39 Precedence: list\r
  40 List-Id: "Use and development of the notmuch mail system."\r
  41         <notmuch.notmuchmail.org>\r
  42 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  43         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  44 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  45 List-Post: <mailto:notmuch@notmuchmail.org>\r
  46 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  47 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  48         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  49 X-List-Received-Date: Mon, 12 Apr 2010 18:44:38 -0000\r
  50 \r
  51 On Mon, 12 Apr 2010 15:58:10 +0200, Tomas Carnecky <tom@dbservice.com> wrote:\r
  52 > In 4fd9ea0 (guess From address from Received headers, 2010-04-06) you \r
  53 > introduced strcasestr, which is not portable, see 82e47ec (notmuch \r
  54 > reply: Use strstr instead of strcasestr for portability., 2010-02-04).\r
  55 > \r
  56 > Is strcasestr really necessary there or can it be replaced with strstr?\r
  57 \r
  58 Well, I'm trying to make sure that rAndOm case in email addresses\r
  59 doesn't throw of the matching - it's quite common for people to write\r
  60 email addresses as First.Last@Company.Com - the RFCs that I've seen\r
  61 (can't double check, on a plane right now) seem to indicate that you\r
  62 can't rely on the headers being lowercase - so even preprocessing the\r
  63 configured email addresses might not be enough.\r
  64 I guess we could selectively force some of the headers to be stored as\r
  65 lower case - Received, X-Original-To, Envelope-To in my case - but that\r
  66 seems wrong somehow.\r
  67 \r
  68 Open to better ideas...\r
  69 \r
  70 /D\r
  71 \r
  72 -- \r
  73 Dirk Hohndel\r
  74 Intel Open Source Technology Center\r