Skip to content

Commit

Permalink
Added CA name to Auditing Issue for clarity
Browse files Browse the repository at this point in the history
  • Loading branch information
TrimarcJake committed Dec 9, 2023
1 parent 092c74f commit f9d9a38
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Private/Find-AuditingIssue.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
$Issue | Add-Member -MemberType NoteProperty -Name 'Technique' -Value 'DETECT' -Force
}
else {
$Issue | Add-Member -MemberType NoteProperty -Name 'Issue' -Value "Auditing is not fully enabled. Current value is $($_.AuditFilter)" -Force
$Issue | Add-Member -MemberType NoteProperty -Name 'Issue' -Value "Auditing is not fully enabled on $($_.CAFullName). Current value is $($_.AuditFilter)" -Force
$Issue | Add-Member -MemberType NoteProperty -Name 'Fix' `
-Value "certutil.exe -config `'$($_.CAFullname)`' -setreg `'CA\AuditFilter`' 127; Invoke-Command -ComputerName `'$($_.dNSHostName)`' -ScriptBlock { Get-Service -Name `'certsvc`' | Restart-Service -Force }" -Force
$Issue | Add-Member -MemberType NoteProperty -Name 'Revert' `
Expand Down

2 comments on commit f9d9a38

@SamErde
Copy link
Collaborator

@SamErde SamErde commented on f9d9a38 Dec 9, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

YES.

@TrimarcJake
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'M GLAD YOU AGREE!

Please sign in to comment.