diff options
| author | Fabien Meghazi <fme@openerp.com> | 2012-09-26 11:43:00 +0200 |
|---|---|---|
| committer | Fabien Meghazi <fme@openerp.com> | 2012-09-26 11:43:00 +0200 |
| commit | 35f5ea03f8cf558e07dce6f9ca507a1271978220 (patch) | |
| tree | a1fc4c54e388782ef5591e681dacc5cd4f43dd33 | |
| parent | 3120f9e0485a180ba08ed9e364d3698c00580f0b (diff) | |
[FIX] oauth provider: do not return email key if user has no email
bzr revid: fme@openerp.com-20120926094300-09qcpak6qg5eokeh
| -rw-r--r-- | addons/auth_oauth_provider/res_users.py | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/addons/auth_oauth_provider/res_users.py b/addons/auth_oauth_provider/res_users.py index 20d9e9c7ade..833286b5642 100644 --- a/addons/auth_oauth_provider/res_users.py +++ b/addons/auth_oauth_provider/res_users.py @@ -46,14 +46,15 @@ class res_users(osv.osv): } user = self.browse(cr, uid, user_id[0], context=context) if access_token == user.last_oauth_token: - return { + r = { "user_id": user.id, "scope": user.last_oauth_token_scope, - "email": user.partner_id.email or '', # TODO: should deliver only according to scopes - "scope": user.last_oauth_token_scope, #"audience": "8819981768.apps.googleusercontent.com", #"expires_in": 436 } + if user.partner_id.email: # TODO: should deliver only according to scopes + r['email'] = user.partner_id.email + return r else: return { "error": "invalid_token" |