Add the ability to set AttributeConsumingServiceIndex in the authn request.

Alexander Schrijver · Alexander Schrijver · commit 7ed8932d36d5 · 2021-01-28T15:01:34.000+01:00
diff --git a/src/onelogin/saml2/authn_request.py b/src/onelogin/saml2/authn_request.py
@@ -108,7 +108,7 @@ def __init__(self, settings, force_authn=False, is_passive=False, set_nameid_pol
 
         attr_consuming_service_str = ''
         if 'attributeConsumingService' in sp_data and sp_data['attributeConsumingService']:
-            attr_consuming_service_str = "\n    AttributeConsumingServiceIndex=\"1\""
+            attr_consuming_service_str = "\n    AttributeConsumingServiceIndex=\"%s\"" % sp_data['attributeConsumingService'].get('index', '1')
 
         request = OneLogin_Saml2_Templates.AUTHN_REQUEST % \
             {

Original file line number	Diff line number	Diff line change
`@@ -108,7 +108,7 @@ def __init__(self, settings, force_authn=False, is_passive=False, set_nameid_pol`
`108`	`108`
`109`	`109`	`attr_consuming_service_str = ''`
`110`	`110`	`if 'attributeConsumingService' in sp_data and sp_data['attributeConsumingService']:`
`111`		`- attr_consuming_service_str = "\n AttributeConsumingServiceIndex=\"1\""`
	`111`	`+ attr_consuming_service_str = "\n AttributeConsumingServiceIndex=\"%s\"" % sp_data['attributeConsumingService'].get('index', '1')`
`112`	`112`
`113`	`113`	`request = OneLogin_Saml2_Templates.AUTHN_REQUEST % \`
`114`	`114`	`{`