http://bugzilla.spamassassin.org/show_bug.cgi?id=3600





------- Additional Comments From [EMAIL PROTECTED]  2004-11-05 09:34 -------
Subject: Re:  Parse Received header lines starting with "by" not only "from"

On Fri, Nov 05, 2004 at 09:16:47AM -0800, [EMAIL PROTECTED] wrote:
> Heiko sent me a note with Exim headers in it...  After some digging I found 
> that the Exim stuff has a 
> specific section in the parser, none of which match the /^by .../ headers in 
> the message.  The headers 
> then fall through and hit an explicit ignore rule:

Oh, and if I comment out that ignore, it falls through and hits another
ignore:

  # ------------------------------------------------------------------------
  # OK, if we still haven't figured out at least the basics (IP and by), or
  # returned due to it being a known-crap format, let's warn so the user can
  # file a bug report or something.

  dbg("received-header: unknown format: $_");
  # and skip the line entirely!  We can't parse it...
  return;


A problem with how the code currently works is a return of undef can
explicitly mean two things: parsing failed, parsing wants to ignore the
header ...

I'd like to change the functionality so that undef means failed, and something
else means "ignore" (like return a hash with skipped=>1 or something).


BTW: The script I posted in a previous comment ...  doesn't work
apparently due to the undef issue and a slight misreading of the code...
So ignore my comments about it.





------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

Reply via email to