bryancall opened a new pull request, #12897: URL: https://github.com/apache/trafficserver/pull/12897
* **Check both start and end milestones for zero** -- `difference_msec()` only checked `ms_end == 0` before returning the `missing` sentinel. When `ms_start` was unset (zero) but `ms_end` was set, the subtraction `ms_end - 0` produced a large epoch-relative value (~1.7 trillion ms) instead of `-1`. This affects any `msdms` log field where the start milestone is never set for a given transaction type. For example, on cache hits the `SERVER_FIRST_CONNECT` milestone is never marked, so `dns` (`SERVER_FIRST_CONNECT - CACHE_OPEN_READ_END`) would leak an epoch value. **Testing:** Validated with a side-car log (50K transactions) on a production host -- zero epoch leaks after fix. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
