Fix double test flakiness due to EOS/TXN_CLOSE race (#7956)

shinrich · web-flow · commit fc703a63d1cd · 2021-06-21T08:28:22.000-05:00
diff --git a/proxy/http/Http1ClientTransaction.cc b/proxy/http/Http1ClientTransaction.cc
@@ -28,6 +28,10 @@
 void
 Http1ClientTransaction::release()
 {
+  // Turn off reading until we are done with the SM
+  // At that point the transaction/session with either be closed
+  // or be put into keep alive state to wait from the next transaction
+  this->do_io_read(this, 0, nullptr);
   _proxy_ssn->clear_session_active();
 }
 

Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,10 @@`
`28`	`28`	`void`
`29`	`29`	`Http1ClientTransaction::release()`
`30`	`30`	`{`
	`31`	`+ // Turn off reading until we are done with the SM`
	`32`	`+ // At that point the transaction/session with either be closed`
	`33`	`+ // or be put into keep alive state to wait from the next transaction`
	`34`	`+ this->do_io_read(this, 0, nullptr);`
`31`	`35`	`_proxy_ssn->clear_session_active();`
`32`	`36`	`}`
`33`	`37`