bneradt commented on PR #13586: URL: https://github.com/apache/trafficserver/pull/13586#issuecomment-5429705975
Thanks for the detailed follow-up. Addressed all of these observations in d4c1217b63: - Ordinary King-of-the-Hill contest loss is now treated as a normal untracked event rather than an error. The table reports TRACKED, CONTEST_LOST, or NO_CANDIDATE, with events_untracked and scan_exhausted metrics providing visibility. - Protected-slot scans are capped at 1024 candidates. - TokenBucket refreshes the timestamp on every CAS retry, and the concurrent test now exercises replenishment. - RuleBuckets uses heterogeneous string-view lookup to avoid per-request string allocation. - The block-expiration AuTest uses 127.0.0.1 consistently, verifies the block is active before expiration, and asserts positive actions.blocked and connections.rejected metrics. - The ClientHello summary includes now precede the anonymous namespace. Validation completed successfully with the normal focused build/tests, ASan focused build/tests (including ODR detection), the full abuse_shield AuTest, formatting, and the Sphinx documentation build. -- 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]
