maskit commented on PR #12618: URL: https://github.com/apache/trafficserver/pull/12618#issuecomment-3465344510
Looks like you guys figured it out. There is cryptic code that was added when the setting value 2 was introduced (70815db3b237d5e572ff9bc0a4df81d64157ba92). I bet the line fixed on this PR was supposed to be checked in a similar way on the commit, but it's too cryptic IMO. I like the explicit comparison better 👍 https://github.com/apache/trafficserver/blob/577cbed448295a263e7911c7baadabbed6f927de/src/iocore/cache/HttpTransactCache.cc#L313-L332 -- 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]
