-
Notifications
You must be signed in to change notification settings - Fork 517
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PR:IntuneDeviceEnrollmentPlatformRestriction - Fixes 3730 #4054
Conversation
@William-Francillette Could you please apply this patch below on top of yours? In Test-TargetResource it removes Identity parameter instead of Id since the latter doesn't exist in the resource, it also removes all auth parameters from {Set,Test}-TargetResource. Additionally it removes some leftovers from one of my other PRs that I forgot to remove. diff --git a/Modules/Microsoft365DSC/DSCResources/MSFT_IntuneDeviceEnrollmentPlatformRestriction/MSFT_IntuneDeviceEnrollmentPlatformRestriction.psm1 b/Modules/Microsoft365DSC/DSCResources/MSFT_IntuneDeviceEnrollmentPlatformRestriction/MSFT_IntuneDeviceEnrollmentPlatformRestriction.psm1
index 40e5fe25f..c20427365 100644
--- a/Modules/Microsoft365DSC/DSCResources/MSFT_IntuneDeviceEnrollmentPlatformRestriction/MSFT_IntuneDeviceEnrollmentPlatformRestriction.psm1
+++ b/Modules/Microsoft365DSC/DSCResources/MSFT_IntuneDeviceEnrollmentPlatformRestriction/MSFT_IntuneDeviceEnrollmentPlatformRestriction.psm1
@@ -274,14 +274,7 @@ function Set-TargetResource
#endregion
$currentCategory = Get-TargetResource @PSBoundParameters
- $PSBoundParameters.Remove('Credential') | Out-Null
- $PSBoundParameters.Remove('ApplicationId') | Out-Null
- $PSBoundParameters.Remove('TenantId') | Out-Null
- $PSBoundParameters.Remove('ApplicationSecret') | Out-Null
- $PSBoundParameters.Remove('ManagedIdentity') | Out-Null
- $PSBoundParameters.Remove('Ensure') | Out-Null
- $PSBoundParameters.Remove('CertificateThumbprint') | Out-Null
- $PSBoundParameters.Remove('Verbose') | Out-Null
+ $PSBoundParameters = Remove-M365DSCAuthenticationParameter -BoundParameters $PSBoundParameters
$PSBoundParameters.Remove('Identity') | Out-Null
if ($Ensure -eq 'Present' -and $currentCategory.Ensure -eq 'Absent')
@@ -516,6 +509,9 @@ function Test-TargetResource
$CurrentValues = Get-TargetResource @PSBoundParameters
$ValuesToCheck = ([Hashtable]$PSBoundParameters).clone()
+ $ValuesToCheck = Remove-M365DSCAuthenticationParameter -BoundParameters $ValuesToCheck
+ $ValuesToCheck.Remove('Identity') | Out-Null
+ $ValuesToCheck.Remove('WindowsMobileRestriction') | Out-Null
if ($CurrentValues.Ensure -eq 'Absent')
{
@@ -547,13 +543,6 @@ function Test-TargetResource
}
}
- $ValuesToCheck.Remove('Credential') | Out-Null
- $ValuesToCheck.Remove('ApplicationId') | Out-Null
- $ValuesToCheck.Remove('TenantId') | Out-Null
- $ValuesToCheck.Remove('ApplicationSecret') | Out-Null
- $ValuesToCheck.Remove('Id') | Out-Null
- $ValuesToCheck.Remove('WindowsMobileRestriction') | Out-Null
-
#Convert any DateTime to String
foreach ($key in $ValuesToCheck.Keys)
{
diff --git a/Modules/Microsoft365DSC/DSCResources/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADHybridJoined/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADHybridJoined.psm1 b/Modules/Microsoft365DSC/DSCResources/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADHybridJoined/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADHybridJoined.psm1
index c19fb4f99..5c144e69b 100644
--- a/Modules/Microsoft365DSC/DSCResources/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADHybridJoined/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADHybridJoined.psm1
+++ b/Modules/Microsoft365DSC/DSCResources/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADHybridJoined/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADHybridJoined.psm1
@@ -571,12 +571,6 @@ function Test-TargetResource
}
}
- $ValuesToCheck.Remove('Id') | Out-Null
- $ValuesToCheck.Remove('Credential') | Out-Null
- $ValuesToCheck.Remove('ApplicationId') | Out-Null
- $ValuesToCheck.Remove('TenantId') | Out-Null
- $ValuesToCheck.Remove('ApplicationSecret') | Out-Null
-
Write-Verbose -Message "Current Values: $(Convert-M365DscHashtableToString -Hashtable $CurrentValues)"
Write-Verbose -Message "Target Values: $(Convert-M365DscHashtableToString -Hashtable $ValuesToCheck)"
diff --git a/Modules/Microsoft365DSC/DSCResources/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADJoined/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADJoined.psm1 b/Modules/Microsoft365DSC/DSCResources/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADJoined/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADJoined.psm1
index ff01ef6f8..605ad7f2f 100644
--- a/Modules/Microsoft365DSC/DSCResources/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADJoined/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADJoined.psm1
+++ b/Modules/Microsoft365DSC/DSCResources/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADJoined/MSFT_IntuneWindowsAutopilotDeploymentProfileAzureADJoined.psm1
@@ -591,12 +591,6 @@ function Test-TargetResource
}
}
- $ValuesToCheck.Remove('Id') | Out-Null
- $ValuesToCheck.Remove('Credential') | Out-Null
- $ValuesToCheck.Remove('ApplicationId') | Out-Null
- $ValuesToCheck.Remove('TenantId') | Out-Null
- $ValuesToCheck.Remove('ApplicationSecret') | Out-Null
-
Write-Verbose -Message "Current Values: $(Convert-M365DscHashtableToString -Hashtable $CurrentValues)"
Write-Verbose -Message "Target Values: $(Convert-M365DscHashtableToString -Hashtable $ValuesToCheck)"
|
sorted |
Pull Request (PR) description
This Pull Request (PR) fixes the following issues
FIXES #3730