Skip to content

Commit 54d02c5

Browse files
committed
cleanup classic api token usage
1 parent 769cf07 commit 54d02c5

File tree

2 files changed

+4
-5
lines changed

2 files changed

+4
-5
lines changed

Modules/CIPPCore/Public/GraphHelper/New-GraphGetRequest.ps1

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,7 @@ function New-GraphGetRequest {
2626

2727
if ($NoAuthCheck -eq $true -or $IsAuthorised) {
2828
if ($scope -eq 'ExchangeOnline') {
29-
$AccessToken = Get-ClassicAPIToken -resource 'https://outlook.office365.com' -Tenantid $tenantid
30-
$headers = @{ Authorization = "Bearer $($AccessToken.access_token)" }
29+
$headers = Get-GraphToken -tenantid $tenantid -scope 'https://outlook.office365.com/.default' -AsApp $asapp -SkipCache $skipTokenCache
3130
} else {
3231
$headers = Get-GraphToken -tenantid $tenantid -scope $scope -AsApp $asapp -SkipCache $skipTokenCache
3332
}

Modules/CIPPCore/Public/GraphHelper/New-TeamsAPIGetRequest.ps1

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,13 @@ function New-TeamsAPIGetRequest($Uri, $tenantID, $Method = 'GET', $Resource = '4
55
#>
66

77
if ((Get-AuthorisedRequest -Uri $uri -TenantID $tenantid)) {
8-
$token = Get-ClassicAPIToken -Tenant $tenantid -Resource $Resource
8+
$token = Get-GraphToken -TenantID $tenantID -Scope "$Resource/.default"
99

1010
$NextURL = $Uri
1111
$ReturnedData = do {
1212
try {
1313
$Data = Invoke-RestMethod -ContentType "$ContentType;charset=UTF-8" -Uri $NextURL -Method $Method -Headers @{
14-
Authorization = "Bearer $($token.access_token)"
14+
Authorization = $token.Authorization
1515
'x-ms-client-request-id' = [guid]::NewGuid().ToString()
1616
'x-ms-client-session-id' = [guid]::NewGuid().ToString()
1717
'x-ms-correlation-id' = [guid]::NewGuid()
@@ -29,4 +29,4 @@ function New-TeamsAPIGetRequest($Uri, $tenantID, $Method = 'GET', $Resource = '4
2929
} else {
3030
Write-Error 'Not allowed. You cannot manage your own tenant or tenants not under your scope'
3131
}
32-
}
32+
}

0 commit comments

Comments
 (0)