shinrich commented on issue #7390:
URL: https://github.com/apache/trafficserver/issues/7390#issuecomment-754108988
The logic that issued the READ_READY immediately was added in June 2015 by
commit e9eeab6b. As part of that change
```
index 1de4acc..f9962f0 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -545,8 +545,7 @@ HttpSM::setup_client_read_request_header()
ua_entry->read_vio = ua_session->do_io_read(this, INT64_MAX,
ua_buffer_reader->mbuf);
// The header may already be in the buffer if this
// a request from a keep-alive connection
- if (ua_buffer_reader->read_avail() > 0)
- handleEvent(VC_EVENT_READ_READY, ua_entry->read_vio);
+ handleEvent(VC_EVENT_READ_READY, ua_entry->read_vio);
}
```
If the READ_READY is sent conditionally (only if there is data in the
buffer), then in theory, the accept_no_activity timeout should stick around
until the first transaction byte arrives. I'll put up a PR for that for you to
test @c-taylor
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
