https://bugs.exim.org/show_bug.cgi?id=3100
Bug ID: 3100
Summary: Code to strip log header is broken on new release
Product: Exim
Version: 4.97
Hardware: x86-64
OS: Linux
Status: NEW
Severity: bug
Priority: medium
Component: Eximstats
Assignee: unallocated@???
Reporter: bill-exim@???
CC: exim-dev@???
Created attachment 1480
-->
https://bugs.exim.org/attachment.cgi?id=1480&action=edit
Patch to resolve above issues.
When I installed the new release reports were not reporting data in delivery
time data was incorrectly reported. Other data appeared to be incorrectly
reported.
On examining the code using the $extra variable, it appeared to not correctly
compute the location of the ID field and often stripped H= data. I found
debugging more complex than I was comfortable with.
I patched the code to capture that entire header while parsing the record and
use this value to determine the $extra value. This significantly simplified
that code. This version will match both old and new message-id formats.
While debugging, I found the $flag variable gets set for all the records
contributing to the reports. I added a line to skip processing records that
don't have the $flag value set. This may increase processing speed on some
systems. My system can have several local log lines added for most records.
I have some local additions to report the hostname or helo name for reports
based on IP address.
--
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/
