Mercurial > hg > nginx
comparison src/http/v2/ngx_http_v2.c @ 6516:ab16126a06a0
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); |