From: Sung-hoon C. <sh...@dr...> - 2005-09-22 00:58:00
|
In dkim-filter generated header [Example], line 10 is "\t\n". ( not "\n" ) This is correct header line( RFC 2822 standard: 2.2.3. Long Header Fields ), but some broken MTAs( mostly not open source ) treated line 10 as EOH( end of header ). This is the problem of borken MTAs. But blank folding line is unnecessary, following [Patch] help to solve this problem. Thanks. ========== Example ========== 1 Received: from pmail0.dreamwiz.com (pmail0.dreamwiz.com [10.0.0.65]) 2 by mx-s1.dreamwiz.com (8.13.5/8.13.5) with ESMTP id j8LA3tsI069112 3 for <sh...@dr...>; Wed, 21 Sep 2005 19:03:55 +0900 (KST) 4 DKIM-Signature: a=rsa-sha1; c=nowsp; d=dreamwiz.com; s=beta; t=1127297035; h=Received: 5 Received:Date:From:Reply-To:Subject:To:Message-ID:Organization: 6 X-Priority:X-Originating-IP:X-Originating-Email:X-Mailer: 7 MIME-Version:Content-Type; b=UjFcyWtc0itYcPP8uubCSxV8UdMYqXYIQFo461 8 IB9Zll1DD7vDHluPENmcEy6OUdWsuLYDDrPYvYeLcT0FO1mwHUOY4xklvNfMfc+07/B 9 uMMuw8S+O1058/yaMtGbECBpW24zWu0kpCG+w2oR3nVj+W9jz7xvpZSibZvjDLmC6c= 10 11 Received: from [127.0.0.1] (localhost.dreamwiz.com [127.0.0.1]) 12 by pmail0.dreamwiz.com (8.13.5/8.13.5) with ESMTP id j8LA3t7u019959 13 for <sh...@dr...>; Wed, 21 Sep 2005 19:03:55 +0900 (KST) ========== Patch ========== *** libdkim/dkim.c.org Thu Jul 21 08:37:15 2005 --- libdkim/dkim.c Thu Sep 22 09:06:36 2005 *************** *** 3439,3444 **** --- 3439,3462 ---- } } + { + int i, ok = 0; + len = strlen( buf ); + for ( i = len-1 ; i >= 0 ; i-- ) + { + if ( buf[i] == '\n' ) + { + if ( !ok ) + { + buf[i] = 0; + } + break; + } + if ( buf[i] != ' ' && buf[i] != '\t' ) + ok = 1; + } + } + --------------------------------------------------------- Choi, Sung-hoon DreamWiz Inc. - Development Team/Senior Manager Major : Mail/Web/Network/System/Security MSN : sh...@dr... WWW : http://my.dreamwiz.com/shoon/ E-mail: sh...@dr... |
From: Murray S. K. <ms...@se...> - 2005-10-18 21:54:54
|
On Thu, 22 Sep 2005, Sung-hoon Choi wrote: > In dkim-filter generated header [Example], line 10 is "\t\n". ( not "\n" > ) This is correct header line( RFC 2822 standard: 2.2.3. Long Header > Fields ), but some broken MTAs( mostly not open source ) treated line 10 > as EOH( end of header ). This is the problem of borken MTAs. But blank > folding line is unnecessary, following [Patch] help to solve this > problem. > [...] Thanks for the report. I'm going to solve the problem in a slightly different way; the only time this happens is when a boundary condition occurs while doing line wrapping of "b=" values. The wrapping of "h=" and other values is implemented such that a trailing \n\t\n can't happen. It will be fixed in the next release, which I'm starting to compile now. |