Cast some data read from dump files to str, to prevent unicode promotion.

This fixes a bug that caused a traceback when de-serialized requests were
replayed. Also adds unit tests for the problem.

Author: cortesi

libmproxy/proxy.py

@@ -198,11 +198,11 @@ def get_state(self):
     def from_state(klass, state):
         return klass(
             ClientConnect.from_state(state["client_conn"]),
-            state["host"],
+            str(state["host"]),
             state["port"],
-            state["scheme"],
-            state["method"],
-            state["path"],
+            str(state["scheme"]),
+            str(state["method"]),
+            str(state["path"]),
             utils.Headers.from_state(state["headers"]),
             base64.decodestring(state["content"]),
             state["timestamp"]
@@ -353,7 +353,7 @@ def from_state(klass, request, state):
         return klass(
             request,
             state["code"],
-            state["msg"],
+            str(state["msg"]),
             utils.Headers.from_state(state["headers"]),
             base64.decodestring(state["content"]),
             state["timestamp"],

test/test_flow.py

@@ -385,14 +385,18 @@ def _treader(self):
     def test_roundtrip(self):
         sio = StringIO()
         f = tutils.tflow()
+        f.request.content = "".join(chr(i) for i in range(255))
         w = flow.FlowWriter(sio)
         w.add(f)
 
         sio.seek(0)
         r = flow.FlowReader(sio)
         l = list(r.stream())
         assert len(l) == 1
-        assert l[0] == f
+
+        f2 = l[0]
+        assert f2 == f
+        assert f2.request.assemble() == f.request.assemble()
 
     def test_load_flows(self):
         r = self._treader()