Skip to content

Commit

Permalink
Put VC_EVENT_READ_READY on top per review comment
Browse files Browse the repository at this point in the history
  • Loading branch information
lzx404243 committed Jun 13, 2023
1 parent e72d015 commit 745c61a
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions proxy/http/HttpSM.cc
Original file line number Diff line number Diff line change
Expand Up @@ -4119,6 +4119,11 @@ HttpSM::tunnel_handler_ssl_producer(int event, HttpTunnelProducer *p)
STATE_ENTER(&HttpSM::tunnel_handler_ssl_producer, event);

switch (event) {
case VC_EVENT_READ_READY:
// This event is triggered when receiving DATA frames without the END_STREAM
// flag set in a HTTP/2 CONNECT request. Breaking as there are more DATA
// frames to come.
break;
case VC_EVENT_READ_COMPLETE:
// This event is triggered during an HTTP/2 CONNECT request when a DATA
// frame with the END_STREAM flag set is received, indicating the end of the
Expand Down Expand Up @@ -4155,11 +4160,6 @@ HttpSM::tunnel_handler_ssl_producer(int event, HttpTunnelProducer *p)
}
}
break;
case VC_EVENT_READ_READY:
// This event is triggered when receiving DATA frames without the END_STREAM
// flag set in a HTTP/2 CONNECT request. Breaking as there are more DATA
// frames to come.
break;
case HTTP_TUNNEL_EVENT_PRECOMPLETE:
// We should never get these event since we don't know
// how long the stream is
Expand Down

0 comments on commit 745c61a

Please sign in to comment.