Fixing issue microsoft#5625

ykuijs · Jan 14, 2025 · 2c150cf · 2c150cf
1 parent ebe9c90
commit 2c150cf
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,9 @@
 
 # UNRELEASED
 
+* AADAuthenticationRequirement
+  * Filtered guests from the export, to prevent errors during export
+    FIXES [#5625](https://github.com/microsoft/Microsoft365DSC/issues/5625)
 * AADGroupEligibilitySchedule
   * New resource for Privileged Identity Management (PIM) for Groups
 * AADNamingLocationPolicy

diff --git a/...DSC/DSCResources/MSFT_AADAuthenticationRequirement/MSFT_AADAuthenticationRequirement.psm1 b/...DSC/DSCResources/MSFT_AADAuthenticationRequirement/MSFT_AADAuthenticationRequirement.psm1
@@ -310,7 +310,7 @@ function Export-TargetResource
 
     try
     {
-        [array]$getValue = Get-MgUser -ErrorAction Stop -All | Where-Object -FilterScript { $null -ne $_.Id }
+        [array]$getValue = Get-MgUser -Filter "userType eq 'member'" -All -ErrorAction Stop | Where-Object -FilterScript { $null -ne $_.Id }
 
         $i = 1
         $dscContent = ''
-Original file line number
+Diff line change
@@ Expand Up / @@ -310,7 +310,7 @@ function Export-TargetResource @@
         try
         {
-            [array]$getValue = Get-MgUser -ErrorAction Stop -All | Where-Object -FilterScript { $null -ne $_.Id }
+            [array]$getValue = Get-MgUser -Filter "userType eq 'member'" -All -ErrorAction Stop | Where-Object -FilterScript { $null -ne $_.Id }
             $i = 1
             $dscContent = ''
@@ Expand Down @@