[FIX] website_sale: update total when the user change a product quantity
[odoo/odoo.git] / addons / web / http.py
index ff74949..7105b4a 100644 (file)
@@ -31,12 +31,10 @@ import werkzeug.utils
 import werkzeug.wrappers
 import werkzeug.wsgi
 import werkzeug.routing as routing
-import urllib
 import urllib2
 
 import openerp
 import openerp.service.security as security
-from openerp.tools import config
 
 import inspect
 import functools
@@ -61,9 +59,8 @@ class WebRequest(object):
 
     .. attribute:: httpsession
 
-        .. deprecated:: 8.0
-
-        Use ``self.session`` instead.
+        a :class:`~collections.Mapping` holding the HTTP session data for the
+        current http session
 
     .. attribute:: params
 
@@ -78,13 +75,16 @@ class WebRequest(object):
 
     .. attribute:: session
 
-        a :class:`OpenERPSession` holding the HTTP session data for the
-        current http session
+        :class:`~session.OpenERPSession` instance for the current request
 
     .. attribute:: context
 
         :class:`~collections.Mapping` of context values for the current request
 
+    .. attribute:: debug
+
+        ``bool``, indicates whether the debug mode is active on the client
+
     .. attribute:: db
 
         ``str``, the name of the database linked to the current request. Can be ``None``
@@ -93,38 +93,80 @@ class WebRequest(object):
     .. attribute:: uid
 
         ``int``, the id of the user related to the current request. Can be ``None``
-        if the current request uses the ``none`` authenticatoin.
+        if the current request uses the ``none`` or the ``db`` authenticatoin.
     """
     def __init__(self, httprequest):
         self.httprequest = httprequest
         self.httpresponse = None
         self.httpsession = httprequest.session
-        self.session = httprequest.session
-        self.session_id = httprequest.session.sid
-        self.disable_db = False
+        self.db = None
         self.uid = None
         self.func = None
         self.auth_method = None
         self._cr_cm = None
         self._cr = None
         self.func_request_type = None
+
+    def init(self, params):
+        self.params = dict(params)
+        # OpenERP session setup
+        self.session_id = self.params.pop("session_id", None)
+        if not self.session_id:
+            i0 = self.httprequest.cookies.get("instance0|session_id", None)
+            if i0:
+                self.session_id = simplejson.loads(urllib2.unquote(i0))
+            else:
+                self.session_id = uuid.uuid4().hex
+        self.session = self.httpsession.get(self.session_id)
+        if not self.session:
+            self.session = OpenERPSession()
+            self.httpsession[self.session_id] = self.session
+
+        with set_request(self):
+            self.db = self.session._db or db_monodb()
+
+        # TODO: remove this
         # set db/uid trackers - they're cleaned up at the WSGI
         # dispatching phase in openerp.service.wsgi_server.application
-        if self.db:
-            threading.current_thread().dbname = self.db
-        if self.session.uid:
-            threading.current_thread().uid = self.session.uid
-        self.context = self.session.context
-        self.lang = self.context["lang"]
+        if self.session._db:
+            threading.current_thread().dbname = self.session._db
+        if self.session._uid:
+            threading.current_thread().uid = self.session._uid
+
+        self.context = self.params.pop('context', {})
+        self.debug = self.params.pop('debug', False) is not False
+        # Determine self.lang
+        lang = self.params.get('lang', None)
+        if lang is None:
+            lang = self.context.get('lang')
+        if lang is None:
+            lang = self.httprequest.cookies.get('lang')
+        if lang is None:
+            lang = self.httprequest.accept_languages.best
+        if not lang:
+            lang = 'en_US'
+        # tranform 2 letters lang like 'en' into 5 letters like 'en_US'
+        lang = babel.core.LOCALE_ALIASES.get(lang, lang)
+        # we use _ as seprator where RFC2616 uses '-'
+        self.lang = lang.replace('-', '_')
 
     def _authenticate(self):
-        if self.session.uid:
+        if self.auth_method == "none":
+            self.db = None
+            self.uid = None
+        elif self.auth_method == "db":
+            self.db = self.session._db or db_monodb()
+            if not self.db:
+                raise SessionExpiredException("No valid database for request %s" % self.httprequest)
+            self.uid = None
+        else: # auth
             try:
                 self.session.check_security()
             except SessionExpiredException, e:
-                self.session.logout()
                 raise SessionExpiredException("Session expired for request %s" % self.httprequest)
-        auth_methods[self.auth_method]()
+            self.db = self.session._db
+            self.uid = self.session._uid
+
     @property
     def registry(self):
         """
@@ -134,14 +176,6 @@ class WebRequest(object):
         return openerp.modules.registry.RegistryManager.get(self.db) if self.db else None
 
     @property
-    def db(self):
-        """
-        The registry to the database linked to this request. Can be ``None`` if the current request uses the
-        ``none'' authentication.
-        """
-        return self.session.db if not self.disable_db else None
-
-    @property
     def cr(self):
         """
         The cursor initialized for the current method call. If the current request uses the ``none`` authentication
@@ -174,47 +208,32 @@ class WebRequest(object):
                 return self.func(*args, **kwargs)
         finally:
             # just to be sure no one tries to re-use the request
-            self.disable_db = True
+            self.db = None
             self.uid = None
 
-def auth_method_user():
-    request.uid = request.session.uid
-
-def auth_method_admin():
-    if not request.db:
-        raise SessionExpiredException("No valid database for request %s" % request.httprequest)
-    request.uid = openerp.SUPERUSER_ID
-
-def auth_method_none():
-    request.disable_db = True
-    request.uid = None
-
-auth_methods = {
-    "user": auth_method_user,
-    "admin": auth_method_admin,
-    "none": auth_method_none,
-}
-
 def route(route, type="http", auth="user"):
     """
     Decorator marking the decorated method as being a handler for requests. The method must be part of a subclass
     of ``Controller``.
 
+    Decorator to put on a controller method to inform it does not require a user to be logged. When this decorator
+    is used, ``request.uid`` will be ``None``. The request will still try to detect the database and an exception
+    will be launched if there is no way to guess it.
+
     :param route: string or array. The route part that will determine which http requests will match the decorated
     method. Can be a single string or an array of strings. See werkzeug's routing documentation for the format of
     route expression ( http://werkzeug.pocoo.org/docs/routing/ ).
     :param type: The type of request, can be ``'http'`` or ``'json'``.
     :param auth: The type of authentication method, can on of the following:
 
-        * ``user``: The user must be authenticated and the current request will perform using the rights of the
-        user.
-        * ``admin``: The user may not be authenticated and the current request will perform using the admin user.
+        * ``auth``: The user must be authenticated.
+        * ``db``: There is no need for the user to be authenticated but there must be a way to find the current
+        database.
         * ``none``: The method is always active, even if there is no database. Mainly used by the framework and
-        authentication modules. There request code will not have any facilities to access the database nor have any
-        configuration indicating the current database nor the current user.
+        authentication modules.
     """
     assert type in ["http", "json"]
-    assert auth in auth_methods.keys()
+    assert auth in ["user", "db", "none"]
     def decorator(f):
         if isinstance(route, list):
             f.routes = route
@@ -239,7 +258,8 @@ class JsonRequest(WebRequest):
 
       --> {"jsonrpc": "2.0",
            "method": "call",
-           "params": {"context": {},
+           "params": {"session_id": "SID",
+                      "context": {},
                       "arg1": "val1" },
            "id": null}
 
@@ -251,7 +271,8 @@ class JsonRequest(WebRequest):
 
       --> {"jsonrpc": "2.0",
            "method": "call",
-           "params": {"context": {},
+           "params": {"session_id": "SID",
+                      "context": {},
                       "arg1": "val1" },
            "id": null}
 
@@ -300,18 +321,20 @@ class JsonRequest(WebRequest):
 
         # Read POST content or POST Form Data named "request"
         self.jsonrequest = simplejson.loads(request, object_hook=reject_nonliteral)
-        self.params = dict(self.jsonrequest.get("params", {}))
-        self.context = self.params.pop('context', self.session.context)
+        self.init(self.jsonrequest.get("params", {}))
 
     def dispatch(self):
         """ Calls the method asked for by the JSON-RPC2 or JSONP request
+
+        :returns: an utf8 encoded JSON-RPC2 or JSONP reply
         """
         if self.jsonp_handler:
             return self.jsonp_handler()
         response = {"jsonrpc": "2.0" }
         error = None
-
         try:
+            #if _logger.isEnabledFor(logging.DEBUG):
+            #    _logger.debug("--> %s.%s\n%s", func.im_class.__name__, func.__name__, pprint.pformat(self.jsonrequest))
             response['id'] = self.jsonrequest.get('id')
             response["result"] = self._call_function(**self.params)
         except AuthenticationError, e:
@@ -333,11 +356,14 @@ class JsonRequest(WebRequest):
         if error:
             response["error"] = error
 
+        if _logger.isEnabledFor(logging.DEBUG):
+            _logger.debug("<--\n%s", pprint.pformat(response))
+
         if self.jsonp:
             # If we use jsonp, that's mean we are called from another host
             # Some browser (IE and Safari) do no allow third party cookies
             # We need then to manage http sessions manually.
-            response['session_id'] = self.session_id
+            response['httpsessionid'] = self.httpsession.sid
             mime = 'application/javascript'
             body = "%s(%s);" % (self.jsonp, simplejson.dumps(response),)
         else:
@@ -378,10 +404,14 @@ def to_jsonable(o):
     return u"%s" % o
 
 def jsonrequest(f):
-    """ 
-        .. deprecated:: 8.0
-
-        Use the ``route()`` decorator instead.
+    """ Decorator marking the decorated method as being a handler for a
+    JSON-RPC request (the exact request path is specified via the
+    ``$(Controller._cp_path)/$methodname`` combination.
+
+    If the method is called, it will be provided with a :class:`JsonRequest`
+    instance and all ``params`` sent during the JSON-RPC request, apart from
+    the ``session_id``, ``context`` and ``debug`` keys (which are stripped out
+    beforehand)
     """
     f.combine = True
     base = f.__name__
@@ -397,13 +427,9 @@ class HttpRequest(WebRequest):
     def __init__(self, *args):
         super(HttpRequest, self).__init__(*args)
         params = dict(self.httprequest.args)
-        ex = set(["session_id"])
-        for k in params.keys():
-            if k in ex:
-                del params[k]
         params.update(self.httprequest.form)
         params.update(self.httprequest.files)
-        self.params = params
+        self.init(params)
 
     def dispatch(self):
         akw = {}
@@ -412,6 +438,7 @@ class HttpRequest(WebRequest):
                 akw[key] = value
             else:
                 akw[key] = type(value)
+        #_logger.debug("%s --> %s.%s %r", self.httprequest.func, func.im_class.__name__, func.__name__, akw)
         try:
             r = self._call_function(**self.params)
         except werkzeug.exceptions.HTTPException, e:
@@ -428,6 +455,10 @@ class HttpRequest(WebRequest):
         else:
             if not r:
                 r = werkzeug.wrappers.Response(status=204)  # no content
+        if isinstance(r, (werkzeug.wrappers.BaseResponse, werkzeug.exceptions.HTTPException)):
+            _logger.debug('<-- %s', r)
+        else:
+            _logger.debug("<-- size: %s", len(r))
         return r
 
     def make_response(self, data, headers=None, cookies=None):
@@ -456,10 +487,14 @@ class HttpRequest(WebRequest):
         return werkzeug.exceptions.NotFound(description)
 
 def httprequest(f):
-    """ 
-        .. deprecated:: 8.0
-
-        Use the ``route()`` decorator instead.
+    """ Decorator marking the decorated method as being a handler for a
+    normal HTTP request (the exact request path is specified via the
+    ``$(Controller._cp_path)/$methodname`` combination.
+
+    If the method is called, it will be provided with a :class:`HttpRequest`
+    instance and all ``params`` sent during the request (``GET`` and ``POST``
+    merged in the same dictionary), apart from the ``session_id``, ``context``
+    and ``debug`` keys (which are stripped out beforehand)
     """
     f.combine = True
     base = f.__name__
@@ -571,8 +606,8 @@ class Model(object):
         def proxy(*args, **kw):
             # Can't provide any retro-compatibility for this case, so we check it and raise an Exception
             # to tell the programmer to adapt his code
-            if not request.db or not request.uid or self.session.db != request.db \
-                or self.session.uid != request.uid:
+            if not request.db or not request.uid or self.session._db != request.db \
+                or self.session._uid != request.uid:
                 raise Exception("Trying to use Model with badly configured database or user.")
                 
             mod = request.registry.get(self.model)
@@ -589,24 +624,32 @@ class Model(object):
             return result
         return proxy
 
-class OpenERPSession(werkzeug.contrib.sessions.Session):
-    def __init__(self, *args, **kwargs):
-        self.inited = False
-        self.modified = False
-        super(OpenERPSession, self).__init__(*args, **kwargs)
-        self.inited = True
-        self._default_values()
-        self.modified = False
-
-    def __getattr__(self, attr):
-        return self.get(attr, None)
-    def __setattr__(self, k, v):
-        if getattr(self, "inited", False):
-            try:
-                object.__getattribute__(self, k)
-            except:
-                return self.__setitem__(k, v)
-        object.__setattr__(self, k, v)
+class OpenERPSession(object):
+    """
+    An OpenERP RPC session, a given user can own multiple such sessions
+    in a web session.
+
+    .. attribute:: context
+
+        The session context, a ``dict``. Can be reloaded by calling
+        :meth:`openerpweb.openerpweb.OpenERPSession.get_context`
+
+    .. attribute:: domains_store
+
+        A ``dict`` matching domain keys to evaluable (but non-literal) domains.
+
+        Used to store references to non-literal domains which need to be
+        round-tripped to the client browser.
+    """
+    def __init__(self):
+        self._creation_time = time.time()
+        self._db = False
+        self._uid = False
+        self._login = False
+        self._password = False
+        self._suicide = False
+        self.context = {}
+        self.jsonp_requests = {}     # FIXME use a LRU
 
     def authenticate(self, db, login=None, password=None, env=None, uid=None):
         """
@@ -615,17 +658,16 @@ class OpenERPSession(werkzeug.contrib.sessions.Session):
 
         :param uid: If not None, that user id will be used instead the login to authenticate the user.
         """
-
         if uid is None:
             uid = openerp.netsvc.dispatch_rpc('common', 'authenticate', [db, login, password, env])
         else:
             security.check(db, uid, password)
-        self.db = db
-        self.uid = uid
-        self.login = login
-        self.password = password
+        self._db = db
+        self._uid = uid
+        self._login = login
+        self._password = password
+        request.db = db
         request.uid = uid
-        request.disable_db = False
 
         if uid: self.get_context()
         return uid
@@ -636,22 +678,9 @@ class OpenERPSession(werkzeug.contrib.sessions.Session):
         should be called at each request. If the authentication fails, a ``SessionExpiredException``
         is raised.
         """
-        if not self.db or not self.uid:
+        if not self._db or not self._uid:
             raise SessionExpiredException("Session expired")
-        security.check(self.db, self.uid, self.password)
-
-    def logout(self):
-        for k in self.keys():
-            del self[k]
-        self._default_values()
-
-    def _default_values(self):
-        self.setdefault("db", None)
-        self.setdefault("uid", None)
-        self.setdefault("login", None)
-        self.setdefault("password", None)
-        self.setdefault("context", {'tz': "UTC", "uid": None})
-        self.setdefault("jsonp_requests", {})
+        security.check(self._db, self._uid, self._password)
 
     def get_context(self):
         """
@@ -661,9 +690,9 @@ class OpenERPSession(werkzeug.contrib.sessions.Session):
 
         :returns: the new context
         """
-        assert self.uid, "The user needs to be logged-in to initialize his context"
+        assert self._uid, "The user needs to be logged-in to initialize his context"
         self.context = request.registry.get('res.users').context_get(request.cr, request.uid) or {}
-        self.context['uid'] = self.uid
+        self.context['uid'] = self._uid
         self._fix_lang(self.context)
         return self.context
 
@@ -687,35 +716,6 @@ class OpenERPSession(werkzeug.contrib.sessions.Session):
 
         context['lang'] = lang or 'en_US'
 
-    """
-        Damn properties for retro-compatibility. All of that is deprecated, all
-        of that.
-    """
-    @property
-    def _db(self):
-        return self.db
-    @_db.setter
-    def _db(self, value):
-        self.db = value
-    @property
-    def _uid(self):
-        return self.uid
-    @_uid.setter
-    def _uid(self, value):
-        self.uid = value
-    @property
-    def _login(self):
-        return self.login
-    @_login.setter
-    def _login(self, value):
-        self.login = value
-    @property
-    def _password(self):
-        return self.password
-    @_password.setter
-    def _password(self, value):
-        self.password = value
-
     def send(self, service_name, method, *args):
         """
         .. deprecated:: 8.0
@@ -737,11 +737,11 @@ class OpenERPSession(werkzeug.contrib.sessions.Session):
 
         Ensures this session is valid (logged into the openerp server)
         """
-        if self.uid and not force:
+        if self._uid and not force:
             return
         # TODO use authenticate instead of login
-        self.uid = self.proxy("common").login(self.db, self.login, self.password)
-        if not self.uid:
+        self._uid = self.proxy("common").login(self._db, self._login, self._password)
+        if not self._uid:
             raise AuthenticationError("Authentication failure")
 
     def ensure_valid(self):
@@ -749,11 +749,11 @@ class OpenERPSession(werkzeug.contrib.sessions.Session):
         .. deprecated:: 8.0
         Use ``check_security()`` instead.
         """
-        if self.uid:
+        if self._uid:
             try:
                 self.assert_valid(True)
             except Exception:
-                self.uid = None
+                self._uid = None
 
     def execute(self, model, func, *l, **d):
         """
@@ -770,7 +770,7 @@ class OpenERPSession(werkzeug.contrib.sessions.Session):
         Use the resistry and cursor in ``openerp.addons.web.http.request`` instead.
         """
         self.assert_valid()
-        r = self.proxy('object').exec_workflow(self.db, self.uid, self.password, model, signal, id)
+        r = self.proxy('object').exec_workflow(self._db, self._uid, self._password, model, signal, id)
         return r
 
     def model(self, model):
@@ -784,11 +784,74 @@ class OpenERPSession(werkzeug.contrib.sessions.Session):
         :type model: str
         :rtype: a model object
         """
-        if not self.db:
+        if self._db == False:
             raise SessionExpiredException("Session expired")
 
         return Model(self, model)
 
+#----------------------------------------------------------
+# Session context manager
+#----------------------------------------------------------
+@contextlib.contextmanager
+def session_context(httprequest, session_store, session_lock, sid):
+    with session_lock:
+        if sid:
+            httprequest.session = session_store.get(sid)
+        else:
+            httprequest.session = session_store.new()
+    try:
+        yield httprequest.session
+    finally:
+        # Remove all OpenERPSession instances with no uid, they're generated
+        # either by login process or by HTTP requests without an OpenERP
+        # session id, and are generally noise
+        removed_sessions = set()
+        for key, value in httprequest.session.items():
+            if not isinstance(value, OpenERPSession):
+                continue
+            if getattr(value, '_suicide', False) or (
+                        not value._uid
+                    and not value.jsonp_requests
+                    # FIXME do not use a fixed value
+                    and value._creation_time + (60*5) < time.time()):
+                _logger.debug('remove session %s', key)
+                removed_sessions.add(key)
+                del httprequest.session[key]
+
+        with session_lock:
+            if sid:
+                # Re-load sessions from storage and merge non-literal
+                # contexts and domains (they're indexed by hash of the
+                # content so conflicts should auto-resolve), otherwise if
+                # two requests alter those concurrently the last to finish
+                # will overwrite the previous one, leading to loss of data
+                # (a non-literal is lost even though it was sent to the
+                # client and client errors)
+                #
+                # note that domains_store and contexts_store are append-only (we
+                # only ever add items to them), so we can just update one with the
+                # other to get the right result, if we want to merge the
+                # ``context`` dict we'll need something smarter
+                in_store = session_store.get(sid)
+                for k, v in httprequest.session.iteritems():
+                    stored = in_store.get(k)
+                    if stored and isinstance(v, OpenERPSession):
+                        if hasattr(v, 'contexts_store'):
+                            del v.contexts_store
+                        if hasattr(v, 'domains_store'):
+                            del v.domains_store
+                        if not hasattr(v, 'jsonp_requests'):
+                            v.jsonp_requests = {}
+                        v.jsonp_requests.update(getattr(
+                            stored, 'jsonp_requests', {}))
+
+                # add missing keys
+                for k, v in in_store.iteritems():
+                    if k not in httprequest.session and k not in removed_sessions:
+                        httprequest.session[k] = v
+
+            session_store.save(httprequest.session)
+
 def session_gc(session_store):
     if random.random() < 0.001:
         # we keep session one week
@@ -866,7 +929,8 @@ class Root(object):
 
         # Setup http sessions
         path = session_path()
-        self.session_store = werkzeug.contrib.sessions.FilesystemSessionStore(path, session_class=OpenERPSession)
+        self.session_store = werkzeug.contrib.sessions.FilesystemSessionStore(path)
+        self.session_lock = threading.Lock()
         _logger.debug('HTTP sessions stored in: %s', path)
 
 
@@ -877,34 +941,22 @@ class Root(object):
 
     def dispatch(self, environ, start_response):
         """
-        Performs the actual WSGI dispatching for the application.
+        Performs the actual WSGI dispatching for the application, may be
+        wrapped during the initialization of the object.
+
+        Call the object directly.
         """
-        try:
-            httprequest = werkzeug.wrappers.Request(environ)
-            httprequest.parameter_storage_class = werkzeug.datastructures.ImmutableDict
-            httprequest.app = self
-
-            session_gc(self.session_store)
-
-            sid = httprequest.args.get('session_id')
-            explicit_session = True
-            if not sid:
-                sid =  httprequest.headers.get("X-Openerp-Session-Id")
-            if not sid:
-                sid = httprequest.cookies.get('session_id')
-                explicit_session = False
-            if sid is None:
-                httprequest.session = self.session_store.new()
-            else:
-                httprequest.session = self.session_store.get(sid)
+        httprequest = werkzeug.wrappers.Request(environ)
+        httprequest.parameter_storage_class = werkzeug.datastructures.ImmutableDict
+        httprequest.app = self
 
-            self._find_db(httprequest)
+        sid = httprequest.cookies.get('sid')
+        if not sid:
+            sid = httprequest.args.get('sid')
 
-            if not "lang" in httprequest.session.context:
-                lang = httprequest.accept_languages.best or "en_US"
-                lang = babel.core.LOCALE_ALIASES.get(lang, lang).replace('-', '_')
-                httprequest.session.context["lang"] = lang
+        session_gc(self.session_store)
 
+        with session_context(httprequest, self.session_store, self.session_lock, sid) as session:
             request = self._build_request(httprequest)
             db = request.db
 
@@ -927,20 +979,10 @@ class Root(object):
             else:
                 response = result
 
-            if httprequest.session.should_save:
-                self.session_store.save(httprequest.session)
-            if not explicit_session and hasattr(response, 'set_cookie'):
-                response.set_cookie('session_id', httprequest.session.sid)
+            if hasattr(response, 'set_cookie'):
+                response.set_cookie('sid', session.sid)
 
             return response(environ, start_response)
-        except werkzeug.exceptions.HTTPException, e:
-            return e(environ, start_response)
-
-    def _find_db(self, httprequest):
-        db = db_monodb(httprequest)
-        if db != httprequest.session.db:
-            httprequest.session.logout()
-            httprequest.session.db = db
 
     def _build_request(self, httprequest):
         if httprequest.args.get('jsonp'):
@@ -981,7 +1023,7 @@ class Root(object):
 
     def _build_router(self, db):
         _logger.info("Generating routing configuration for database %s" % db)
-        routing_map = routing.Map(strict_slashes=False)
+        routing_map = routing.Map()
 
         def gen(modules, nodb_only):
             for module in modules:
@@ -1008,8 +1050,8 @@ class Root(object):
                                         url = url[: -1]
                                 routing_map.add(routing.Rule(url, endpoint=function))
 
-        modules_set = set(controllers_per_module)
-        modules_set.discard('web')
+        modules_set = set(controllers_per_module.keys())
+        modules_set -= set("web")
         # building all none methods
         gen(["web"] + sorted(modules_set), True)
         if not db:
@@ -1019,9 +1061,9 @@ class Root(object):
         with registry.cursor() as cr:
             m = registry.get('ir.module.module')
             ids = m.search(cr, openerp.SUPERUSER_ID, [('state','=','installed')])
-            installed = set(x['name'] for x in m.read(cr, 1, ids, ['name']))
-            modules_set = modules_set & installed
-
+            installed = set([x['name'] for x in m.read(cr, 1, ids, ['name'])])
+            modules_set = modules_set.intersection(set(installed))
+        modules = ["web"] + sorted(modules_set)
         # building all other methods
         gen(["web"] + sorted(modules_set), False)
 
@@ -1038,7 +1080,12 @@ class Root(object):
 
     def find_handler(self):
         """
-        Tries to discover the controller handling the request for the path specified in the request.
+        Tries to discover the controller handling the request for the path
+        specified by the provided parameters
+
+        :param path: path to match
+        :returns: a callable matching the path sections
+        :rtype: ``Controller | None``
         """
         path = request.httprequest.path
         urls = self.get_db_router(request.db).bind("")
@@ -1054,63 +1101,48 @@ class Root(object):
         request.auth_method = getattr(original, "auth", "user")
         request.func_request_type = original.exposed
 
-root = None
-
-def db_list(force=False, httprequest=None):
-    httprequest = httprequest or request.httprequest
-    dbs = openerp.netsvc.dispatch_rpc("db", "list", [force])
-    h = httprequest.environ['HTTP_HOST'].split(':')[0]
+def db_list(force=False):
+    proxy = request.session.proxy("db")
+    dbs = proxy.list(force)
+    h = request.httprequest.environ['HTTP_HOST'].split(':')[0]
     d = h.split('.')[0]
     r = openerp.tools.config['dbfilter'].replace('%h', h).replace('%d', d)
     dbs = [i for i in dbs if re.match(r, i)]
     return dbs
 
-def db_monodb(httprequest=None):
-    """
-        Magic function to find the current database.
-
-        Implementation details:
-
-        * Magic
-        * More magic
-
-        Returns ``None`` if the magic is not magic enough.
-    """
-    httprequest = httprequest or request.httprequest
+def db_monodb():
     db = None
-    redirect = None
 
-    dbs = db_list(True, httprequest)
+    # 1 try the db in the url
+    db_url = request.params.get('db')
+    if db_url:
+        return db_url
+
+    try:
+        dbs = db_list()
+    except Exception:
+        # ignore access denied
+        dbs = []
 
-    # 1 try the db already in the session
-    db_session = httprequest.session.db
-    if db_session in dbs:
-        return db_session
+    # 2 use the database from the cookie if it's listable and still listed
+    cookie_db = request.httprequest.cookies.get('last_used_database')
+    if cookie_db in dbs:
+        db = cookie_db
 
-    # 2 if there is only one db in the db filters, take it
-    if len(dbs) == 1:
-        return dbs[0]
+    # 3 use the first db
+    if dbs and not db:
+        db = dbs[0]
+    return db.lower() if db else db
 
-    # 3 if there are multiple dbs, take the first one only if we can list them
-    if len(dbs) > 1 and config['list_db']:
-        return dbs[0]
-    return None
 
-class CommonController(Controller):
+class JsonRpcController(Controller):
 
     @route('/jsonrpc', type='json', auth="none")
     def jsonrpc(self, service, method, args):
         """ Method used by client APIs to contact OpenERP. """
         return openerp.netsvc.dispatch_rpc(service, method, args)
 
-    @route('/gen_session_id', type='json', auth="none")
-    def gen_session_id(self):
-        nsession = root.session_store.new()
-        return nsession.sid
-
 def wsgi_postload():
-    global root
-    root = Root()
-    openerp.wsgi.register_wsgi_handler(root)
+    openerp.wsgi.register_wsgi_handler(Root())
 
 # vim:et:ts=4:sw=4: