From 532326bbc2aa92e87ccd8d1e01197911fd38d044 Mon Sep 17 00:00:00 2001 From: klzgrad Date: Thu, 25 Jan 2018 12:50:06 +0800 Subject: [PATCH] h2: Reduce warnings about RST on invalid streams Per RFC 7540#6.4: However, after sending the RST_STREAM, the sending endpoint MUST be prepared to receive and process additional frames sent on the stream that might have been sent by the peer prior to the arrival of the RST_STREAM. --- net/spdy/chromium/spdy_session.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/net/spdy/chromium/spdy_session.cc b/net/spdy/chromium/spdy_session.cc index dc4f23bb8f..f1c25c6831 100644 --- a/net/spdy/chromium/spdy_session.cc +++ b/net/spdy/chromium/spdy_session.cc @@ -2642,7 +2642,7 @@ void SpdySession::OnRstStream(SpdyStreamId stream_id, ActiveStreamMap::iterator it = active_streams_.find(stream_id); if (it == active_streams_.end()) { // NOTE: it may just be that the stream was cancelled. - LOG(WARNING) << "Received RST for invalid stream" << stream_id; + VLOG(1) << "Received RST for invalid stream" << stream_id; return; } @@ -2858,7 +2858,7 @@ void SpdySession::OnWindowUpdate(SpdyStreamId stream_id, if (it == active_streams_.end()) { // NOTE: it may just be that the stream was cancelled. - LOG(WARNING) << "Received WINDOW_UPDATE for invalid stream " << stream_id; + VLOG(1) << "Received WINDOW_UPDATE for invalid stream " << stream_id; return; } @@ -2911,7 +2911,7 @@ void SpdySession::OnHeaders(SpdyStreamId stream_id, ActiveStreamMap::iterator it = active_streams_.find(stream_id); if (it == active_streams_.end()) { // NOTE: it may just be that the stream was cancelled. - LOG(WARNING) << "Received HEADERS for invalid stream " << stream_id; + VLOG(1) << "Received HEADERS for invalid stream " << stream_id; return; }