[IMP] auth_oauth: refactor js method on_oauth_sign_in() to enable overriding of the oauth state definition
bzr revid: rco@openerp.com-20121126085334-df4xhhcwvwe40xgj
This commit is contained in:
parent
3854441aff
commit
1455afbda3
|
@ -35,27 +35,30 @@ openerp.auth_oauth = function(instance) {
|
|||
on_oauth_sign_in: function(ev) {
|
||||
ev.preventDefault();
|
||||
var index = $(ev.target).data('index');
|
||||
var p = this.oauth_providers[index];
|
||||
var ret = _.str.sprintf('%s//%s/auth_oauth/signin', location.protocol, location.host);
|
||||
var provider = this.oauth_providers[index];
|
||||
var return_url = _.str.sprintf('%s//%s/auth_oauth/signin', location.protocol, location.host);
|
||||
if (instance.session.debug) {
|
||||
ret += '?debug';
|
||||
return_url += '?debug';
|
||||
}
|
||||
var dbname = self.$("form [name=db]").val();
|
||||
var state_object = {
|
||||
d: dbname,
|
||||
p: p.id
|
||||
};
|
||||
var state = JSON.stringify(state_object);
|
||||
var state = self._oauth_state();
|
||||
var params = {
|
||||
response_type: 'token',
|
||||
client_id: p.client_id,
|
||||
redirect_uri: ret,
|
||||
scope: p.scope,
|
||||
state: state,
|
||||
client_id: provider.client_id,
|
||||
redirect_uri: return_url,
|
||||
scope: provider.scope,
|
||||
state: JSON.stringify(state),
|
||||
};
|
||||
var url = p.auth_endpoint + '?' + $.param(params);
|
||||
var url = provider.auth_endpoint + '?' + $.param(params);
|
||||
window.location = url;
|
||||
},
|
||||
_oauth_state: function(provider) {
|
||||
// return the state object sent back with the redirected uri
|
||||
var dbname = self.$("form [name=db]").val();
|
||||
return {
|
||||
d: dbname,
|
||||
p: provider.id,
|
||||
};
|
||||
},
|
||||
});
|
||||
|
||||
};
|
||||
|
|
Loading…
Reference in New Issue