[exim-dev] [Bug 3100] Code to strip log header is broken on …

Αρχική Σελίδα
Delete this message
Reply to this message
Συντάκτης: Exim Bugzilla
Ημερομηνία:  
Προς: exim-dev
Αντικείμενο: [exim-dev] [Bug 3100] Code to strip log header is broken on new release
https://bugs.exim.org/show_bug.cgi?id=3100

--- Comment #4 from Bill Thorsteinson <bill-exim@???> ---
(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):
## exim-dev-unsubscribe@???
## Exim details at http://www.exim.org/
## Please use the Wiki with this list - http://wiki.exim.org/