serrislew commented on PR #10491:
URL: https://github.com/apache/trafficserver/pull/10491#issuecomment-1730512724

   @zwoop that's true but this is called on `TSRemapDoRemap` when the 
transaction is called so this is after starting up, so would we want to error 
out (stop ATS) for this case? is that better than just skipping the config? 
   
   I guess you mean by checking the configs before each reaching this point but 
it also seems like it can be invalid despite the configs 
[TSHttpTxnPristineUrlGet()](https://github.com/apache/trafficserver/blob/master/plugins/cachekey/cachekey.cc#L257-L260)


-- 
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: github-unsubscr...@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to