Merge lp:~openerp-dev/openobject-addons/trunk-auth_oauth-fix into lp:openobject-addons

Proposed by Pariket Trivedi(OpenERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openobject-addons/trunk-auth_oauth-fix
Merge into: lp:openobject-addons
Diff against target: 18 lines (+4/-1)
1 file modified
auth_oauth/controllers/main.py (+4/-1)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/trunk-auth_oauth-fix
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+218412@code.launchpad.net

Description of the change

Hello,

Fixed Issue For not loging other browser generate traceback.
request.httprequest.full_path replace for request.httprequest.base_url.

Thank you,
ptr

To post a comment you must log in.

Unmerged revisions

9408. By Vidhin Mehta (OpenERP)

[FIX]pass full_path, by base_url and query string. request or httprequest both instance not contain full_path attribute.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'auth_oauth/controllers/main.py'
2--- auth_oauth/controllers/main.py 2014-04-10 11:04:24 +0000
3+++ auth_oauth/controllers/main.py 2014-05-06 13:01:57 +0000
4@@ -67,10 +67,13 @@
5 return providers
6
7 def get_state(self, provider):
8+ url_redirect = request.httprequest.base_url
9+ if request.httprequest.query_string:
10+ url_redirect += '?' + request.httprequest.query_string
11 state = dict(
12 d=request.session.db,
13 p=provider['id'],
14- r=request.httprequest.full_path
15+ r=url_redirect
16 )
17 token = request.params.get('token')
18 if token:

Subscribers

People subscribed via source and target branches

to all changes: