Merge lp:~openerp-dev/openerp-web/trunk-bug-1211305 into lp:openerp-web

[Thibault Delavallée (OpenERP)]

I don't know because people seems to want me to merge things without merge proposal.

[Thibault Delavallée (OpenERP) (tde-openerp)]

Hello,

Please fill a bug report with a way to reproduce your bug, so we can check that effectively something is solved.

Best regards,

Thibault.

review: Needs Information

[Xavier (Open ERP) (xmo-deactivatedaccount)]

If the cleanup of the session_id key was too early, maybe it could just have been moved down instead of duplicating it entirely...

review: Disapprove

Unmerged revisions

3830. By Amit Bhavsar (Open ERP) on 2013-09-05

[Import] solved the import error 404.

=== modified file 'addons/web/http.py'
--- addons/web/http.py	2013-10-04 08:46:18 +0000
+++ addons/web/http.py	2013-10-10 08:52:43 +0000
@@ -409,6 +409,11 @@
         self.params = params
 
     def dispatch(self):
+        ex = set(["session_id"])
+        for k in self.params.keys():
+            if k in ex:
+                del self.params[k]
+
         akw = {}
         for key, value in self.httprequest.args.iteritems():
             if isinstance(value, basestring) and len(value) < 1024: