Mercurial > hg > nginx

changeset 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 657e029bac28
files src/http/v2/ngx_http_v2.c
diffstat 1 files changed, 4 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/src/http/v2/ngx_http_v2.c
+++ b/src/http/v2/ngx_http_v2.c
@@ -1060,6 +1060,10 @@ ngx_http_v2_state_headers(ngx_http_v2_co
 
     if (!h2c->settings_ack && !(h2c->state.flags & NGX_HTTP_V2_END_STREAM_FLAG))
     {
+        ngx_log_error(NGX_LOG_INFO, h2c->connection->log, 0,
+                      "client sent stream with data "
+                      "before settings were acknowledged");
+
         status = NGX_HTTP_V2_REFUSED_STREAM;
         goto rst_stream;
     }