aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorThomas Kriechbaumer <thomas@kriechbaumer.name>2015-08-31 22:20:12 +0200
committerThomas Kriechbaumer <thomas@kriechbaumer.name>2015-08-31 22:22:10 +0200
commitc4d6b357262a5964a8d10ea20b92d22efc9c68a4 (patch)
treebdd671932122c99d42a2e7d4357a3f5b402f6c31
parent481cc6ea842dc3c531c45a4bd228bdd6ebcc4229 (diff)
downloadmitmproxy-c4d6b357262a5964a8d10ea20b92d22efc9c68a4.tar.gz
mitmproxy-c4d6b357262a5964a8d10ea20b92d22efc9c68a4.tar.bz2
mitmproxy-c4d6b357262a5964a8d10ea20b92d22efc9c68a4.zip
do not log WindowUpdateFrame frames
-rw-r--r--libmproxy/protocol/http.py12
1 files changed, 10 insertions, 2 deletions
diff --git a/libmproxy/protocol/http.py b/libmproxy/protocol/http.py
index f0f4ac24..7f57d17c 100644
--- a/libmproxy/protocol/http.py
+++ b/libmproxy/protocol/http.py
@@ -7,6 +7,7 @@ from netlib import odict
from netlib.tcp import NetLibError, Address
from netlib.http.http1 import HTTP1Protocol
from netlib.http.http2 import HTTP2Protocol
+from netlib.http.http2.frame import WindowUpdateFrame
from .. import utils
from ..exceptions import InvalidCredentials, HttpException, ProtocolException
@@ -187,8 +188,15 @@ class Http2Layer(_HttpLayer):
layer = HttpLayer(self, self.mode)
layer()
- def handle_unexpected_frame(self, frm):
- self.log("Unexpected HTTP2 Frame: %s" % frm.human_readable(), "info")
+ def handle_unexpected_frame(self, frame):
+ if isinstance(frame, WindowUpdateFrame):
+ # Clients are sending WindowUpdate frames depending on their flow control algorithm.
+ # Since we cannot predict these frames, and we do not need to respond to them,
+ # simply accept them, and hide them from the log.
+ # Ideally we should keep track of our own flow control window and
+ # stall transmission if the outgoing flow control buffer is full.
+ return
+ self.log("Unexpected HTTP2 Frame: %s" % frame.human_readable(), "info")
class ConnectServerConnection(object):