Mercurial > hg > nginx

comparison src/http/v2/ngx_http_v2.c @ 6516:ab16126a06a0

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
HTTP/2: write logs when refusing streams with data. Refusing streams is known to be incorrectly handled at least by IE, Edge and Safari. Make sure to provide appropriate logging to simplify fixing this in the affected browsers.
author Maxim Dounin <mdounin@mdounin.ru>
date Mon, 18 Apr 2016 21:18:24 +0300
parents 8df664ebe037
children 7760b54d5458
comparison
equal deleted inserted replaced
6515:8df664ebe037 6516:ab16126a06a0
1058 goto rst_stream; 1058 goto rst_stream;
1059 } 1059 }
1060 1060
1061 if (!h2c->settings_ack && !(h2c->state.flags & NGX_HTTP_V2_END_STREAM_FLAG)) 1061 if (!h2c->settings_ack && !(h2c->state.flags & NGX_HTTP_V2_END_STREAM_FLAG))
1062 { 1062 {
1063 ngx_log_error(NGX_LOG_INFO, h2c->connection->log, 0,
1064 "client sent stream with data "
1065 "before settings were acknowledged");
1066
1063 status = NGX_HTTP_V2_REFUSED_STREAM; 1067 status = NGX_HTTP_V2_REFUSED_STREAM;
1064 goto rst_stream; 1068 goto rst_stream;
1065 } 1069 }
1066 1070
1067 node = ngx_http_v2_get_node_by_id(h2c, h2c->state.sid, 1); 1071 node = ngx_http_v2_get_node_by_id(h2c, h2c->state.sid, 1);