[exim-dev] [Bug 3100] New: Code to strip log header is broke…

Αρχική Σελίδα
Delete this message
Reply to this message
Συντάκτης: Exim Bugzilla
Ημερομηνία:  
Προς: exim-dev
Αντικείμενο: [exim-dev] [Bug 3100] New: Code to strip log header is broken on new release
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/