Skip to content

Commit

Permalink
Document that a system property is required for ADFS (#91)
Browse files Browse the repository at this point in the history
  • Loading branch information
Vlatombe authored Dec 6, 2020
1 parent 49d8f35 commit 74c0b68
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions doc/ADFS_CONFIG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,15 @@ Example: Setting up Active Directory Federation Services (ADFS) to use with Jenk
*Note:* replace adfs.example.com with the name of your ADFS, replace jenkins.example.com with the name of your Jenkins host.

**On the Jenkins side:**
Download the IdP file to paste into the Jenkins config from your ADFS. It is generally exposed under the following URL:
* Download the IdP file to paste into the Jenkins config from your ADFS. It is generally exposed under the following URL:

```
https://adfs.example.com/FederationMetadata/2007-06/FederationMetadata.xml
```

* Set the system property `-Dorg.apache.xml.security.ignoreLineBreaks=true` when starting Jenkins.


**On the Windows side:**

Open the Management console (mmc), make sure you have the AD FS Management snap-in. Add a Relying Party Trust:
Expand All @@ -22,7 +25,7 @@ Go through the wizard. The properties at the end should look like indicated on t
------------ | -------------
![](images/Screen_Shot_2015-12-10_at_16.11.42.png) | ![](images/Screen_Shot_2015-12-10_at_16.11.44.png)

**Encryption:** import key from the JENKIS_HOME/saml-sp.metadata.xml file | **Signature:** import key from the JENKIS_HOME/saml-sp.metadata.xml file
**Encryption:** import key from the JENKINS_HOME/saml-sp.metadata.xml file | **Signature:** import key from the JENKINS_HOME/saml-sp.metadata.xml file
------------ | -------------
![](images/Screen_Shot_2015-12-10_at_16.11.46.png) | ![](images/Screen_Shot_2015-12-10_at_16.11.49.png)

Expand Down Expand Up @@ -51,4 +54,3 @@ Allow all users to connect, or modify depending on your setup: | -
![](images/Screen_Shot_2015-12-10_at_16.12.45.png)



1 comment on commit 74c0b68

@sailajalingam
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi We tried the same way as mentioned in the docoument .. but cannot get it working .. Do we have to enable the signature ?

Please sign in to comment.