https://bugs.exim.org/show_bug.cgi?id=3100

--- Comment #4 from Bill Thorsteinson <[email protected]> ---
(In reply to Bill Thorsteinson from comment #2)
> Created attachment 1481 [details]
> Updated patch
> 
> My original patch did not strip the $flag from delivery records breaking
> parsing of addresses used in local destination addresses. It also broke
> address reporting in relayed message reporting.
> 
> Always removing the flag values appears to break matching "H=" values which
> can be the first field in rejection messages. 
> 
> I also mistakenly removed Completed messages which broke time on queue
> reporting. Skipping non-essential records based on $flag values was already
> implemented correctly.

My Perl is rusty. I inserted a redundant check to skip records with an
incorrect header. This line following the header regex can be removed.

   next unless (defined($1));

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

-- 
## subscription configuration (requires account):
##   https://lists.exim.org/mailman3/postorius/lists/exim-dev.lists.exim.org/
## unsubscribe (doesn't require an account):
##   [email protected]
## Exim details at http://www.exim.org/
## Please use the Wiki with this list - http://wiki.exim.org/

Reply via email to