fix fltering

kayasax · Nov 7, 2024 · dbbbc59 · dbbbc59
1 parent 10205b7
commit dbbbc59
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/EasyPIM/functions/Get-PIMEntraRolePendingApproval.ps1 b/EasyPIM/functions/Get-PIMEntraRolePendingApproval.ps1
@@ -42,7 +42,7 @@ function Get-PIMEntraRolePendingApproval{
 
         Write-Verbose "Get-PIMAzureResourcePendingApproval start with parameters: tenantID => $tenantID"
 
-        $endpoint="/roleManagement/directory/roleAssignmentScheduleRequests/filterByCurrentUser(on='approver')?$filter=status eq 'PendingApproval'"
+        $endpoint="/roleManagement/directory/roleAssignmentScheduleRequests/filterByCurrentUser(on='approver')?`$filter=status eq 'PendingApproval'"
         $response = Invoke-Graph -Endpoint $endpoint -Method "GET"
 
         $out = @()

diff --git a/EasyPIM/internal/functions/Invoke-ARM.ps1 b/EasyPIM/internal/functions/Invoke-ARM.ps1
@@ -44,15 +44,19 @@ function Invoke-ARM {
         write-verbose "`n>> request body: $body"
         write-verbose "requested URI : $restURI ; method : $method"
 
-        $script:subscriptionID=[regex]::Matches($restURI,".*\/subscriptions\/(.*)\/providers.*$").groups[1].Value
+        #TODO need better way to handle mangement group scope!!
+        if($restURI -notmatch "managementgroups"){
+            $script:subscriptionID=[regex]::Matches($restURI,".*\/subscriptions\/(.*)\/providers.*$").groups[1].Value
 
 
-        if ( $null -eq (get-azcontext) -or ( (get-azcontext).Tenant.Id -ne $script:tenantID ) ) {
-            Write-Verbose ">> Connecting to Azure with tenantID $script:tenantID"
-            Connect-AzAccount -Tenantid $script:tenantID -Subscription $script:subscriptionID
+            if ( $null -eq (get-azcontext) -or ( (get-azcontext).Tenant.Id -ne $script:tenantID ) ) {
+                Write-Verbose ">> Connecting to Azure with tenantID $script:tenantID"
+                Connect-AzAccount -Tenantid $script:tenantID -Subscription $script:subscriptionID
+            }    
         }
+
 
-        #todo replace with invoke-azrestmethod
+        #replaced with invoke-azrestmethod
         <#
         # Get access Token
         Write-Verbose ">> Getting access token"