diff options
| author | Maxim Dounin <mdounin@mdounin.ru> | 2016-04-18 21:18:24 +0300 |
|---|---|---|
| committer | Maxim Dounin <mdounin@mdounin.ru> | 2016-04-18 21:18:24 +0300 |
| commit | eb38cbda2fcf318b9e1f2e463705658c286ec04e (patch) | |
| tree | 886cb0204e7c9513e7fc5fe375b902939c3b46d1 | |
| parent | 7691b9750e0d3a2137e9d9188db65f7e39624b97 (diff) | |
| download | nginx-eb38cbda2fcf318b9e1f2e463705658c286ec04e.tar.gz nginx-eb38cbda2fcf318b9e1f2e463705658c286ec04e.zip | |
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.
| -rw-r--r-- | src/http/v2/ngx_http_v2.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/http/v2/ngx_http_v2.c b/src/http/v2/ngx_http_v2.c index c88e29728..e960aaacb 100644 --- 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_connection_t *h2c, u_char *pos, 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; } |