bneradt commented on PR #13563:
URL: https://github.com/apache/trafficserver/pull/13563#issuecomment-5499425162

   Thanks @cmcfarlen — these findings were valid and are addressed in 
18e48cce9d.
   
   - The origin now keys on {PATH}{%Range}; allowed requests receive 206 
responses, and unique request markers prove their background fetches start.
   - The Content-Length coverage now pairs a matching less-than-1000 exclusion 
with a non-matching greater-than-1000 rule. The test asserts the parse log, 
positive background-fetch markers, and absence of markers for excluded 
requests. It also documents that the operators are inclusive.
   - StillRunningAfter now accumulates both ATS and the origin process.
   - The waiter now uses the final unique positive marker, emitted after both 
exclusion runs, so it also acts as a log-flush barrier.
   - The trailing comma, wildcard description, and implicit host-to-path 
derivation are cleaned up.
   
   Validation in asfats5:
   - Full CMake build and install
   - format target
   - ./autest.sh --sandbox /tmp/sb-pr13563 --clean=none -f background_fetch (1 
passed)


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