Mailing List Archive

------- You are receiving this mail because: -------
You are on the CC list for the bug.

http://bugs.exim.org/show_bug.cgi?id=758




--- Comment #1 from Tom Kistner <tom@duncanthrax.net> 2008-09-09 22:04:29 ---
Tony, should I fix this?

I'd propose to wrap this in base64 ...


--
Configure bugmail: http://bugs.exim.org/userprefs.cgi?tab=email

--
## List details at http://lists.exim.org/mailman/listinfo/exim-dev Exim details at http://www.exim.org/ ##

------- You are receiving this mail because: -------
You are on the CC list for the bug.

http://bugs.exim.org/show_bug.cgi?id=758




--- Comment #2 from Tony Finch <dot@dotat.at> 2008-09-10 14:15:12 ---
I feel sort of responsible because it was my cockup :-)

I'm undecided between escaping the newlines, adding special parsing for
spam_report (but that would introduce a downgrade problem), and just reverting
the spam_report part of the patch.

I added the feature mainly because I was surprised that it wasn't already
there, and because I thought postmasters might want to add spam headers in
transports in some circumstances (e.g. depending on whether the message is
routed internally or externally).


--
Configure bugmail: http://bugs.exim.org/userprefs.cgi?tab=email

--
## List details at http://lists.exim.org/mailman/listinfo/exim-dev Exim details at http://www.exim.org/ ##

------- You are receiving this mail because: -------
You are on the CC list for the bug.

http://bugs.exim.org/show_bug.cgi?id=758




--- Comment #3 from Stefan Kaltenbrunner <stefan@kaltenbrunner.cc> 2008-09-14 08:25:12 ---
simply reverting at least the spam_report part of the patch might not be the
worst idea. Spam reports can get very large(depending on your report template
definition) and currently one has to pay the price of carrying them in the
spool files unconditionally and I don't think that the added complexity of
encoding them in one form or another is worth it.


--
Configure bugmail: http://bugs.exim.org/userprefs.cgi?tab=email

--
## List details at http://lists.exim.org/mailman/listinfo/exim-dev Exim details at http://www.exim.org/ ##