Enhance AAD provider to handle nested PIM groups and refactor Get-PrivilegedUser to reduce code duplication#1310

Merged

nanda-katikaneni merged 13 commits intomainfrom 1288-get-privilegeduser-fails-404-message

Sep 26, 2024

+420-213

Commits on Sep 24, 2024

added try catch for troubleshooting the source line in Get-PrivilegedUser

tkol2022
authored and
mitchelbaker-cisa
committed
changed Get-PrivilegedUser to recursive calls and handled nested groups assigned to PIM groups

tkol2022
authored and
mitchelbaker-cisa
committed
Added module dependency Microsoft.Graph.Beta.DirectoryObjects for Get-MgBetaDirectoryObject

tkol2022
authored and
mitchelbaker-cisa
committed
Added 404 handling for deleted users and groups. Added try catch detailed error stacktrace to Get-PrivilegedRole

tkol2022
authored and
mitchelbaker-cisa
committed
line 375 had some evil whitespace and was exorcised

tkol2022
authored and
mitchelbaker-cisa
committed
changed write-debug to write-warning

tkol2022
authored and
mitchelbaker-cisa
committed
added maximum recursion safeguard for nested groups

tkol2022
authored and
mitchelbaker-cisa
committed
changed $RoleName parameter to string and wrote first unit test for LoadObjectDataIntoPrivilegedUserHashtable

tkol2022
authored and
mitchelbaker-cisa
committed
fixed flaw in how Recursioncount was incremented and added more unit tests

tkol2022
authored and
mitchelbaker-cisa
committed
improved most of the comments and changed incorrect variable name for privileged roles from hashtable to array

tkol2022
authored and
mitchelbaker-cisa
committed

Commits on Sep 26, 2024