Skip to content

Commit d475710

Browse files
committed
[Documentation] Fixing a duplicated assignment of idp_sso_target_url
On the 'Initialization Phase' section the sample do a double assignment for idp_sso_target_url, the one removed is kind of misleading since is pointing to metadata. Apparently this was introduced on d37cdc7.
1 parent debcf23 commit d475710

1 file changed

Lines changed: 0 additions & 1 deletion

File tree

README.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -169,7 +169,6 @@ def saml_settings
169169
170170
settings.assertion_consumer_service_url = "http://#{request.host}/saml/consume"
171171
settings.issuer = "http://#{request.host}/saml/metadata"
172-
settings.idp_sso_target_url = "https://app.onelogin.com/saml/metadata/#{OneLoginAppId}"
173172
settings.idp_entity_id = "https://app.onelogin.com/saml/metadata/#{OneLoginAppId}"
174173
settings.idp_sso_target_url = "https://app.onelogin.com/trust/saml2/http-post/sso/#{OneLoginAppId}"
175174
settings.idp_slo_target_url = "https://app.onelogin.com/trust/saml2/http-redirect/slo/#{OneLoginAppId}"

0 commit comments

Comments
 (0)