Mailing-List: contact mime4j-dev-help@james.apache.org; run by ezmlm
Precedence: bulk
Reply-To: mime4j-dev@james.apache.org
Received-SPF: pass (athena.apache.org: domain of apache@bago.org designates
 74.125.78.150 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <f470f68e0912281510n12559602sd78c9f73124882b0@mail.gmail.com>
References: <330063705.1261941090173.JavaMail.jira@brutus.apache.org>
	 <69500553.1261943129408.JavaMail.jira@brutus.apache.org>
	 <f470f68e0912281510n12559602sd78c9f73124882b0@mail.gmail.com>
Date: Tue, 29 Dec 2009 00:51:17 +0100
Message-ID: <9426afb70912281551o499c8cf4i51a4918eb6c4a960@mail.gmail.com>
Subject: Re: [jira] Resolved: (MIME4J-146) [possible backward incompatibility]
	Make sure "body" is always unfolded, "raw" is always folded,
 both in RawField
	and ParsedFields.
From: Stefano Bagnara <apache@bago.org>
To: mime4j-dev <mime4j-dev@james.apache.org>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

2009/12/29 Robert Burrell Donkin <robertburrelldonkin@gmail.com>:
> On Sun, Dec 27, 2009 at 7:45 PM, Stefano Bagnara (JIRA)
> <mime4j-dev@james.apache.org> wrote:
>>
>> =A0 =A0 [ https://issues.apache.org/jira/browse/MIME4J-146?page=3Dcom.at=
lassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
>>
>> Stefano Bagnara resolved MIME4J-146.
>> ------------------------------------
>>
>> =A0 =A0Resolution: Fixed
>>
>> IMO this works as expected now.
>> All tests still pass.
>> If you have external code depending on the previous behaviour please pro=
vide a testcase so that we can discuss the need of body to be unfolded in t=
he parser and not by RawField.
>
> i'm worried that this kind of change will break IMAP and/or jsieve in
> a way that's really hard to fix. this will really suck for james since
> we'll have to fork a known good version of mime4j. if someone could
> run the IMAP and JSieve build after any incompatible semantic change
> then at least there'd be a chance that the regression would be
> understood.

Well, there was already changes wrt folding with MIME4J-141 so the
compatibility issue was already open. Did you check imap/jsieve after
MIME4J-141 was changed? All of mime4j testcases passed before
MIME4J-141, after MIME4J-141 and after MIME4J-146. IMHO if external
code is "break in a way that's really hard to fix" then we should have
testcases for them. If you already have any pointer about your worries
this will help adding tests.

As far as I can tell neither IMAP or Jsieve do care about
RawField.getBody and I'd say that if they even call getBody they
expect the unfolded data (that is how it is handled now).

Stefano