Merge pull request #1148 from jensotto/fix-missing-authentication-realm

SPTrustedSecurityTokenIssuer: Fixed RegisteredIssuerNameRealm not applied
dsccommunity · Jan 24, 2020 · 0e32879 · 0e32879
2 parents 5cca8dd + b618e7c
commit 0e32879
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -12,6 +12,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   - Updated PULL_REQUEST_TEMPLATE.md to match DSC standard
 - SPFarm
   - Added possibility to set application credential key.
+- SPTrustedSecurityTokenIssuer
+  - Fixed RegisteredIssuerNameRealm not applied if specified.
 
 ### Changed
 

diff --git a/...Dsc/DSCResources/MSFT_SPTrustedSecurityTokenIssuer/MSFT_SPTrustedSecurityTokenIssuer.psm1 b/...Dsc/DSCResources/MSFT_SPTrustedSecurityTokenIssuer/MSFT_SPTrustedSecurityTokenIssuer.psm1
@@ -300,6 +300,11 @@ function Set-TargetResource
                     Write-Verbose -Message "RegisteredIssuerNameRealm is not specified, use Get-SPAuthenticationRealm instead."
                     $registeredIssuerNameRealm = Get-SPAuthenticationRealm
                 }
+                else
+                {
+                    $registeredIssuerNameRealm = $params.RegisteredIssuerNameRealm
+                }
+
                 $registeredIssuerName = "$($params.RegisteredIssuerNameIdentifier)@$registeredIssuerNameRealm"
 
                 $runParams.Add("RegisteredIssuerName", $registeredIssuerName)