[dev] Re: [cvs] commit: horde/lib MIME.php horde/lib/MIME Message.php Part.php

[Jan Schneider]

Zitat von Michael M Slusarz <slusarz at bigworm.colorado.edu>:

> slusarz     2003/07/18 13:11:11 PDT
>
>   Modified files:
>     lib                  MIME.php
>     lib/MIME             Message.php Part.php
>   Log:
>   Revert (partially) Jan's commit.  We will continue to have
> wrapHeaders() do
>   RFC 2822 breaking by default.  But, as I've tried to make clear in
> MIME_Part,
>   we do *NOT* internally store data with CRLF, only LF.  Thus, mixing and
>   matching linebreaks will cause havoc with mailers (the PHP Mail PEAR
> classes
>   automatically convert from LF to CRLF).
>   Instead, add option to wrapHeaders() to allow for user-defined CRLF
>   determination; this allows us to use the current EOL value in MIME_Part
>   consistently.
>   Jan, whatever problem you were having pre-commit, let me know.  All I
> know is
>   that the "\n" only wrapping seemed to be working fine for everyone else
> for ~5
>   months, and once you changed to "\r\n" things started to break all over
> the
>   place. :(

The problem occurs if you save a draft containing wrapped headers (mostly
Received: headers) while using a Cyrus server that *requires* linebreaks
being CRLF as per RFC 2822.
In this case the headers won't pass the PEAR:Mail of course.

Jan.

--
http://www.horde.org - The Horde Project
http://www.ammma.de - discover your knowledge
http://www.tip4all.de - Deine private Tippgemeinschaft