masaori335 commented on code in PR #12502:
URL: https://github.com/apache/trafficserver/pull/12502#discussion_r2353959449
##########
tests/gold_tests/h2/http2_rst_stream.test.py:
##########
@@ -58,7 +58,8 @@
tr.Processes.Default.Streams.All += Testers.ContainsExpression(
'Submitted RST_STREAM frame for key 1 on stream 1.', 'Send RST_STREAM
frame.')
-server.Streams.All += Testers.ExcludesExpression('RST_STREAM', 'Server is not
affected.')
+# NOTE: Background Fill fix changed this behavior
+# server.Streams.All += Testers.ExcludesExpression('RST_STREAM', 'Server is
not affected.')
Review Comment:
Thank you for taking a look, I was wondering what this check should do.
Changed to `ContainsExpression` to make sure origin server also received the
`RST_STREAM` frame.
--
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]
