bryancall commented on PR #12878:
URL: https://github.com/apache/trafficserver/pull/12878#issuecomment-3893889797

   Found one correctness issue in the new metric verification helper:\n\n can 
false-pass when the expected value is a prefix of the actual value (for example 
expected  also matches ).\n\nWould you mind switching this to an anchored regex 
match so the assertion is exact? Something like  (with  escaped) avoids prefix 
matches and makes the check robust.


-- 
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]

Reply via email to