bryancall opened a new pull request, #12876: URL: https://github.com/apache/trafficserver/pull/12876
## Summary - Remove the `proxy-response: status: 502` assertion from the request-block test because the response status is inherently racy -- the origin may respond before ATS acts on the block - Update the test comment to explain why no status assertion is made - Plugin correctness is still verified via diags.log checks that confirm the rule matched and blocking action was taken ## Details Request body transforms run after headers are sent to the origin. By the time the body is inspected and the plugin decides to block, the origin may have already responded. The client can see a 502, a closed connection, or the origin's 200 response depending on timing. The diags.log validations already verify: - The block rule matched (`Matched rule: xxe_request_block`) - The blocking action was taken (`Blocking request due to rule`) These are the meaningful checks -- the HTTP status code to the client is a side effect of the race. Fixes #12875 -- 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]
