You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@trafficserver.apache.org by GitBox <gi...@apache.org> on 2022/05/26 20:18:25 UTC
[GitHub] [trafficserver] serrislew opened a new pull request, #8871: Handle hit as miss when cach obj is invalid
serrislew opened a new pull request, #8871:
URL: https://github.com/apache/trafficserver/pull/8871
When cache object is invalid but has a hit status, it should be handled as a miss so no assumptions or invalid access to the object will be made
--
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-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] vmamidi merged pull request #8871: Handle hit as miss when cach obj is invalid
Posted by GitBox <gi...@apache.org>.
vmamidi merged PR #8871:
URL: https://github.com/apache/trafficserver/pull/8871
--
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-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] serrislew commented on pull request #8871: Handle hit as miss when cach obj is invalid
Posted by GitBox <gi...@apache.org>.
serrislew commented on PR #8871:
URL: https://github.com/apache/trafficserver/pull/8871#issuecomment-1139934074
Yes 9.2.x would be better. We encountered a crash where the cached object was a hit and suspect a plugin marked it as null before utilizing it, ultimately handling an invalid object. It relates to the "serve stale when DNS is down" PR since all hit status objects go to `HandleCacheOpenReadHit` to revalidate before determining how to serve the cached object
--
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-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] zwoop commented on pull request #8871: Handle hit as miss when cach obj is invalid
Posted by GitBox <gi...@apache.org>.
zwoop commented on PR #8871:
URL: https://github.com/apache/trafficserver/pull/8871#issuecomment-1144166349
Cherry-picked to v9.2.x
--
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-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] bneradt commented on pull request #8871: Handle hit as miss when cach obj is invalid
Posted by GitBox <gi...@apache.org>.
bneradt commented on PR #8871:
URL: https://github.com/apache/trafficserver/pull/8871#issuecomment-1139913198
@serrislew: thank you for the fix. I've marked this just for 10.x for now. Do you think this should go to 9.2.x? If so, please add a comment with your reasons and I can mark as such.
--
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-unsubscribe@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org