Fix a problem in ODictCaseless that could cause duplicate headers.

1 files changed, 4 insertions, 1 deletions

diff --git a/libmproxy/flow.py b/libmproxy/flow.py
index e25460b5..21a8f3a2 100644
--- a/libmproxy/flow.py
+++ b/libmproxy/flow.py
@@ -142,6 +142,7 @@ class ODict:
         return ret
 
     def _filter_lst(self, k, lst):
+        k = self._kconv(k)
         new = []
         for i in lst:
             if self._kconv(i[0]) != k:
@@ -163,7 +164,7 @@ class ODict:
             raise ValueError("ODict valuelist should be lists.")
         new = self._filter_lst(k, self.lst)
         for i in valuelist:
-            new.append((k, i))
+            new.append([k, i])
         self.lst = new
 
     def __delitem__(self, k):
@@ -478,6 +479,8 @@ class Request(HTTPMsg):
             appropriate content-type header. Note that this will destory the
             existing body if there is one.
         """
+        # FIXME: If there's an existing content-type header indicating a
+        # url-encoded form, leave it alone.
         self.headers["Content-Type"] = [HDR_FORM_URLENCODED]
         self.content = utils.urlencode(odict.lst)