Skip to content

Conversation

@Manoj-Kesana
Copy link
Collaborator

DC Agent is deployed and enforcing strong authentication policies.

@Manoj-Kesana Manoj-Kesana self-assigned this Dec 17, 2025
@Manoj-Kesana Manoj-Kesana added enhancement New feature or request ready for review PR is ready for review and merging labels Dec 17, 2025
@alexandair
Copy link
Collaborator

@KManoj0000 How did you test this when isActiveMode is not implemented?
Also, I don't get any sensors in 8mile tenant.

@Manoj-Kesana Manoj-Kesana marked this pull request as draft December 18, 2025 04:50
@Manoj-Kesana Manoj-Kesana marked this pull request as ready for review December 22, 2025 07:16
@Manoj-Kesana Manoj-Kesana marked this pull request as draft December 24, 2025 07:50
@Manoj-Kesana Manoj-Kesana reopened this Jan 2, 2026
@Manoj-Kesana Manoj-Kesana marked this pull request as ready for review January 2, 2026 16:32
@alexandair alexandair requested a review from Copilot January 4, 2026 20:04
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request implements a new assessment test (ID: 25403) to validate that Microsoft Entra Private Access Sensors are deployed on domain controllers and actively enforcing strong authentication policies. The test queries the Private Access Sensors API and evaluates whether sensors are in active status and not running in audit mode.

Key changes:

  • Adds PowerShell test script to check sensor deployment status and enforcement configuration
  • Includes detailed report generation showing sensor statistics and problematic configurations
  • Provides markdown documentation explaining security risks and remediation steps

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
src/powershell/tests/Test-Assessment.25403.ps1 Implements the assessment test that queries Private Access Sensors via Microsoft Graph API and evaluates their status and configuration
src/powershell/tests/Test-Assessment.25403.md Provides security context documentation explaining threats from weak domain controller authentication and links to remediation resources

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@alexandair
Copy link
Collaborator

@KManoj0000
I still don't understand how can this test work when isAuditMode property is not implemented.

@Manoj-Kesana
Copy link
Collaborator Author

@KManoj0000 I still don't understand how can this test work when isAuditMode property is not implemented.

Hi @alexandair , isAuditMode property has implemented please check below snip for reference.

image

@alexandair
Copy link
Collaborator

alexandair commented Jan 5, 2026

@KManoj0000
Then the spec needs to be updated, right?
The spec states that isAuditMode and isBreakglassEnabled are not implemented (Challenges section).

Copy link
Collaborator

@alexandair alexandair left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@SagarSathe SagarSathe left a comment

Choose a reason for hiding this comment

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

LGTM

@SagarSathe SagarSathe merged commit a9869ae into main Jan 6, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request ready for review PR is ready for review and merging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants