Unverified Commit 931fd8a5 by Matt Drayer Committed by GitHub

Merge pull request #16886 from edx/mattdrayer/ENT-807

ENT-807: Utilize configured user for SuccessFactors BizX OData API callbacks
parents b159c271 885ddb99
...@@ -230,6 +230,10 @@ class SapSuccessFactorsIdentityProvider(EdXSAMLIdentityProvider): ...@@ -230,6 +230,10 @@ class SapSuccessFactorsIdentityProvider(EdXSAMLIdentityProvider):
def odata_client_id(self): def odata_client_id(self):
return self.conf['odata_client_id'] return self.conf['odata_client_id']
@property
def oauth_user_id(self):
return self.conf.get('oauth_user_id')
def invalid_configuration(self): def invalid_configuration(self):
""" """
Check that we have all the details we need to properly retrieve rich data from the Check that we have all the details we need to properly retrieve rich data from the
...@@ -282,12 +286,15 @@ class SapSuccessFactorsIdentityProvider(EdXSAMLIdentityProvider): ...@@ -282,12 +286,15 @@ class SapSuccessFactorsIdentityProvider(EdXSAMLIdentityProvider):
""" """
Obtain a SAML assertion from the SAP SuccessFactors BizX OAuth2 identity provider service using Obtain a SAML assertion from the SAP SuccessFactors BizX OAuth2 identity provider service using
information specified in the third party authentication configuration "Advanced Settings" section. information specified in the third party authentication configuration "Advanced Settings" section.
Utilizes the OAuth user_id if defined in Advanced Settings in order to generate the SAML assertion,
otherwise utilizes the user_id for the current user in context.
""" """
session = requests.Session() session = requests.Session()
oauth_user_id = self.oauth_user_id if self.oauth_user_id else user_id
transaction_data = { transaction_data = {
'token_url': self.sapsf_token_url, 'token_url': self.sapsf_token_url,
'client_id': self.odata_client_id, 'client_id': self.odata_client_id,
'user_id': user_id, 'user_id': oauth_user_id,
'private_key': self.sapsf_private_key, 'private_key': self.sapsf_private_key,
} }
try: try:
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment