Browse Source

[core] check for Upgrade before h2 upgrade check

(short-circuit if Upgrade header not present)
master
Glenn Strauss 7 months ago
parent
commit
e0a4a7849f
  1. 3
      src/connections.c

3
src/connections.c

@ -803,7 +803,8 @@ static int connection_handle_read_state(connection * const con) {
chunkqueue_mark_written(cq, r->rqst_header_len);
connection_set_state(r, CON_STATE_REQUEST_END);
if (!con->is_ssl_sock && r->conf.h2proto && 0 == r->http_status
if (light_btst(r->rqst_htags, HTTP_HEADER_UPGRADE)
&& !con->is_ssl_sock && r->conf.h2proto && 0 == r->http_status
&& h2_check_con_upgrade_h2c(r)) {
/*(Upgrade: h2c over cleartext does not have SNI; no COMP_HTTP_HOST)*/
r->conditional_is_valid = (1 << COMP_SERVER_SOCKET)

Loading…
Cancel
Save