Merge lp:~openerp-dev/openerp-web/7.0-opw-594067-msh into lp:openerp-web/7.0

Proposed by Mohammed Shekha(Open ERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openerp-web/7.0-opw-594067-msh
Merge into: lp:openerp-web/7.0
Diff against target: 11 lines (+1/-0)
1 file modified
addons/web/controllers/main.py (+1/-0)
To merge this branch: bzr merge lp:~openerp-dev/openerp-web/7.0-opw-594067-msh
Reviewer Review Type Date Requested Status
Martin Trigaux (OpenERP) Pending
Xavier (Open ERP) Pending
Review via email: mp+173404@code.launchpad.net

Description of the change

Hello,

Fixed the issue of monodb redirect, when login called directly with needed parameters, it redirects to wrong db(last used db) instead of you gave db parameter.

Demo:- To reproduce this issue enter URL like /?db=yourdb&login=yourlogin&key=yourkey.

Even if you have defined yourdb in URL it redirects you to last used db.

It should not be redirected to last used db instead it should redirect to Home page as we already logged in with key and already specified db.

Thanks.

To post a comment you must log in.

Unmerged revisions

3993. By Mohammed Shekha<email address hidden>

[FIX]Fixed the issue of monodb redirect, when login called directly with needed parameters, it redirects to wrong db(last used db) instead of you gave db parameter.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'addons/web/controllers/main.py'
--- addons/web/controllers/main.py 2013-07-02 12:20:14 +0000
+++ addons/web/controllers/main.py 2013-07-08 06:05:31 +0000
@@ -388,6 +388,7 @@
388 REMOTE_ADDR=wsgienv['REMOTE_ADDR'],388 REMOTE_ADDR=wsgienv['REMOTE_ADDR'],
389 )389 )
390 req.session.authenticate(db, login, key, env)390 req.session.authenticate(db, login, key, env)
391 redirect_url += '?' + urllib.urlencode({'db': db})
391 return set_cookie_and_redirect(req, redirect_url)392 return set_cookie_and_redirect(req, redirect_url)
392393
393def set_cookie_and_redirect(req, redirect_url):394def set_cookie_and_redirect(req, redirect_url):