sxia-aviatrix commented on code in PR #13574:
URL: https://github.com/apache/trafficserver/pull/13574#discussion_r3864932072
##########
tests/gold_tests/slow_post/quick_server.test.py:
##########
@@ -132,3 +151,7 @@ def run(self):
for abort_response_headers in [True, False]:
test = QuickServerTest(abort_request, drain_request,
abort_response_headers)
test.run()
+
+# Partial POST with a request transform plugin: exercises the abort_tunnel()
+# cleanup path for TransformVConnection entries in the vc_table.
+QuickServerTest(abort_request=True, drain_request=False,
abort_response_headers=False, use_request_transform=True).run()
Review Comment:
I have updated the code and comments. But I don't think ASAN/LSAN can catch
this, since this is resource leak and vc pointer is reachable. I also tried
ASAN/LSAN myself and confirmed that they cannot fail the tests.
--
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]
