Skip to content

SecOps - 41083 - Step-up authentication is required upon risky in-session action - #1487

Merged
Thomas Detzner (tdetzner) merged 8 commits into
devfrom
Sec-41083
Aug 11, 2026
Merged

SecOps - 41083 - Step-up authentication is required upon risky in-session action#1487
Thomas Detzner (tdetzner) merged 8 commits into
devfrom
Sec-41083

Conversation

@Manoj-Kesana

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a new SecOps assessment (41083) to evaluate whether Microsoft Entra Conditional Access enforces step-up authentication for risky in-session actions by checking enabled CA policies for risk conditions and authentication strength controls, and provides accompanying markdown guidance.

Changes:

  • Introduces Test-Assessment-41083 to query enabled Conditional Access policies via Microsoft Graph (beta), evaluate risk + auth-strength configuration, and emit a markdown results table with portal links.
  • Adds Test-Assessment.41083.md with background and remediation references, integrating the %TestResult% placeholder.

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.41083.ps1 New assessment implementation: Graph query, evaluation logic, and markdown report generation.
src/powershell/tests/Test-Assessment.41083.md New assessment documentation/remediation links and results placeholder.

Comment thread src/powershell/tests/Test-Assessment.41083.ps1 Outdated
Comment thread src/powershell/tests/Test-Assessment.41083.ps1
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@Manoj-Kesana Manoj Kesana (Manoj-Kesana) changed the title Secops - 41083 - Step-up authentication is required upon risky in-session action SecOps - 41083 - Step-up authentication is required upon risky in-session action Aug 6, 2026
@Manoj-Kesana Manoj Kesana (Manoj-Kesana) added the ready for review PR is ready for review and merging label Aug 6, 2026
@Manoj-Kesana
Manoj Kesana (Manoj-Kesana) marked this pull request as ready for review August 6, 2026 12:34

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Manoj Kesana (@Manoj-Kesana) Please, address my feedback.

There are also some spec issues:

The Specification Contains Conflicting Pass Rules

Locations:

  • Strong phishing-resistant requirement: /ztspecs/specs/secops/41083.md#L74-L78
  • Simplified non-null requirement: /ztspecs/specs/secops/41083.md#L86-L95

The pass criteria require a phishing-resistant built-in or custom strength, while the later evaluation steps require only a non-null authenticationStrength. The current implementation follows the weaker evaluation steps, but the title, customer explanation, security rationale, and remediation all support the stronger requirement.

Recommendation: Update the evaluation section to explicitly classify the strength ID and custom allowedCombinations.

4.3 The Specification Describes An Unsupported Grant-Control Combination

Location: /ztspecs/specs/secops/41083.md#L78

grantControls.operator is OR containing mfa plus authenticationStrength,
or simply authenticationStrength.

Microsoft documents that Require multifactor authentication and Require authentication strength cannot be configured together in the same Conditional Access policy. Authentication strength replaces the general MFA grant control for this scenario.

Reference: Conditional Access authentication strengths - limitations.

Recommendation: Remove the mfa plus authenticationStrength alternative from the specification and describe the expected operator behavior when authentication strength is combined with other supported grant controls.

Comment thread src/powershell/tests/Test-Assessment.41083.ps1 Outdated
Comment thread src/powershell/tests/Test-Assessment.41083.ps1 Outdated
@Manoj-Kesana Manoj Kesana (Manoj-Kesana) removed the ready for review PR is ready for review and merging label Aug 7, 2026
@Manoj-Kesana
Manoj Kesana (Manoj-Kesana) marked this pull request as draft August 7, 2026 02:36
@Manoj-Kesana Manoj Kesana (Manoj-Kesana) added the ready for review PR is ready for review and merging label Aug 10, 2026
@Manoj-Kesana
Manoj Kesana (Manoj-Kesana) marked this pull request as ready for review August 10, 2026 11:21

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Manoj Kesana (@Manoj-Kesana) Please, address my feedback.

Comment thread src/powershell/tests/Test-Assessment.41083.ps1
Comment thread src/powershell/tests/Test-Assessment.41083.ps1

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Manoj Kesana (@Manoj-Kesana) Please, address my feedback.

Comment thread src/powershell/tests/Test-Assessment.41083.ps1

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@tdetzner
Thomas Detzner (tdetzner) merged commit 86b8624 into dev Aug 11, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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