Skip to content

Commit ab5bca9

Browse files
authored
Merge pull request KelvinTegelaar#1526 from Zacgoose/alert-fix
Fix this dammed alert
2 parents daf1a22 + cddca7d commit ab5bca9

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

Modules/CIPPCore/Public/Alerts/Get-CIPPAlertHuntressRogueApps.ps1

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ function Get-CIPPAlertHuntressRogueApps {
1313
Param (
1414
[Parameter(Mandatory = $false)]
1515
[Alias('input')]
16-
[bool]$InputValue = $false,
16+
$InputValue,
1717
$TenantFilter
1818
)
1919

@@ -22,7 +22,7 @@ function Get-CIPPAlertHuntressRogueApps {
2222
$RogueAppFilter = $RogueApps.appId -join "','"
2323
$ServicePrincipals = New-GraphGetRequest -uri "https://graph.microsoft.com/beta/servicePrincipals?`$filter=appId in ('$RogueAppFilter')" -tenantid $TenantFilter
2424
# If IgnoreDisabledApps is true, filter out disabled service principals
25-
if ($InputValue) {
25+
if ($InputValue -eq $true) {
2626
$ServicePrincipals = $ServicePrincipals | Where-Object { $_.accountEnabled -eq $true }
2727
}
2828

0 commit comments

Comments
 (0)