Skip to content

Conversation

@dagarwal-ecs
Copy link
Collaborator

@dagarwal-ecs dagarwal-ecs commented Nov 25, 2025

πŸ—£ Description

Add the 5 discussed indicators to the baseline and key terminology

πŸ’­ Motivation and context

Closes #1864

πŸ§ͺ Testing

Read through the baselines and key terminologies

βœ… Pre-approval checklist

  • This PR has an informative and human-readable title.
  • PR targets the correct parent branch (e.g., main or release-name) for merge.
  • Changes are limited to a single goal - eschew scope creep!
  • Changes are sized such that they do not touch excessive number of files.
  • All future TODOs are captured in issues, which are referenced in code comments.
  • These code changes follow the ScubaGear content style guide.
  • Related issues these changes resolve are linked preferably via closing keywords.
  • All relevant type-of-change labels added.
  • All relevant project fields are set.
  • All relevant repo and/or project documentation updated to reflect these changes.
  • Unit tests added/updated to cover PowerShell and Rego changes.
  • Functional tests added/updated to cover PowerShell and Rego changes.
  • All relevant functional tests passed.
  • All automated checks (e.g., linting, static analysis, unit/smoke tests) passed.

βœ… Pre-merge checklist

  • PR passed smoke test check.

  • Feature branch has been rebased against changes from parent branch, as needed

    Use Rebase branch button below or use this reference to rebase from the command line.

  • Resolved all merge conflicts on branch

  • Notified merge coordinator that PR is ready for merge via comment mention

  • Demonstrate changes to the team for questions and comments.
    (Note: Only required for issues of size Medium or larger)

βœ… Post-merge checklist

  • Feature branch deleted after merge to clean up repository.
  • Verified that all checks pass on parent branch (e.g., main or release-name) after merge.

@dagarwal-ecs dagarwal-ecs added this to the Orca milestone Nov 25, 2025
@dagarwal-ecs dagarwal-ecs self-assigned this Nov 25, 2025
@dagarwal-ecs dagarwal-ecs added the baseline-document Issues relating to the text in the baseline documents themselves label Nov 25, 2025
@dagarwal-ecs dagarwal-ecs changed the title Add Indicators to Reports Add Indicators to Baselines Nov 25, 2025
Copy link
Collaborator

@adhilto adhilto left a comment

Choose a reason for hiding this comment

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

I love how this looks, definitely a big improvement.

Most of my comments apply to all the baselines, but I only left them one time rather than duplicate them. Also a lot them are subjective so feel free to push back on anything.


**Resource Tenant & Home Tenant**: In scenarios where guest users are involved the **resource tenant** hosts the M365 target resources that the guest user is accessing. The **home tenant** is the one that hosts the guest user's identity.

**BOD 25-01 Requirement**: This indicator means that policy is required under CISA BOD 25-01
Copy link
Collaborator

Choose a reason for hiding this comment

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

Minor tweak that I think reads better:

(also I recommend that all of these sentences end with a period for consistency)

Suggested change
**BOD 25-01 Requirement**: This indicator means that policy is required under CISA BOD 25-01
**BOD 25-01 Requirement**: This indicator means that the policy is required under CISA BOD 25-01.

@dagarwal-ecs dagarwal-ecs requested a review from adhilto December 2, 2025 02:29

**BOD 25-01 Requirement**: This indicator means that policy is required under CISA BOD 25-01

**Automated Check**: This indicator means that the policy can be automatically checked via ScubaGear
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
**Automated Check**: This indicator means that the policy can be automatically checked via ScubaGear
**Automated Check**: This indicator means that the policy can be automatically checked via ScubaGear.


**Sensitive Accounts**: This term denotes a set of user accounts that have access to sensitive and high-value information. Certain accountsβ€”like those belonging to CEOs, CFOs, CISOs, and IT administratorsβ€”have access to highly sensitive data and critical systems, making them prime targets for cyberattacks. These accounts, referred to as priority accounts, require enhanced security measures to minimize the risk of compromise.

**BOD 25-01 Requirement**: This indicator means that policy is required under CISA BOD 25-01
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
**BOD 25-01 Requirement**: This indicator means that policy is required under CISA BOD 25-01
**BOD 25-01 Requirement**: This indicator means that the policy is required under CISA BOD 25-01.


**BOD 25-01 Requirement**: This indicator means that policy is required under CISA BOD 25-01

**Automated Check**: This indicator means that the policy can be automatically checked via ScubaGear
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
**Automated Check**: This indicator means that the policy can be automatically checked via ScubaGear
**Automated Check**: This indicator means that the policy can be automatically checked via ScubaGear.

a connector to connect to the Dataverse table and perform create, read,
update, and delete (CRUD) operations.

**BOD 25-01 Requirement**: This indicator means that policy is required under CISA BOD 25-01
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
**BOD 25-01 Requirement**: This indicator means that policy is required under CISA BOD 25-01
**BOD 25-01 Requirement**: This indicator means that the policy is required under CISA BOD 25-01.


**BOD 25-01 Requirement**: This indicator means that policy is required under CISA BOD 25-01

**Automated Check**: This indicator means that the policy can be automatically checked via ScubaGear
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
**Automated Check**: This indicator means that the policy can be automatically checked via ScubaGear
**Automated Check**: This indicator means that the policy can be automatically checked via ScubaGear.

The custom policy SHOULD include an action to block access to sensitive
information by restricted apps and unwanted Bluetooth applications.

[![Manual](https://img.shields.io/badge/Manual-green)](/defender.md#msdefender46v1-instructions)
Copy link
Collaborator

Choose a reason for hiding this comment

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

This link is broken, I think this fixes it

Suggested change
[![Manual](https://img.shields.io/badge/Manual-green)](/defender.md#msdefender46v1-instructions)
[![Manual](https://img.shields.io/badge/Manual-green)](./defender.md#msdefender46v1-instructions)

#### MS.DEFENDER.5.2v1
The alerts SHOULD be sent to a monitored address or incorporated into a Security Information and Event Management (SIEM).

[![Manual](https://img.shields.io/badge/Manual-green)](/defender.md#msdefender52sv1-instructions)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Also broken

Suggested change
[![Manual](https://img.shields.io/badge/Manual-green)](/defender.md#msdefender52sv1-instructions)
[![Manual](https://img.shields.io/badge/Manual-green)](./defender.md#msdefender52sv1-instructions)

#### MS.DEFENDER.6.3v1
Audit logs SHALL be maintained for at least the minimum duration dictated by OMB M-21-31.

[![Manual](https://img.shields.io/badge/Manual-green)](/defender.md#msdefender63v1-instructions)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Also broken

Suggested change
[![Manual](https://img.shields.io/badge/Manual-green)](/defender.md#msdefender63v1-instructions)
[![Manual](https://img.shields.io/badge/Manual-green)](./defender.md#msdefender63v1-instructions)

#### MS.POWERPLATFORM.4.1v1
Content Security Policy (CSP) SHALL be enforced for model-driven and canvas Power Apps.

[![Manual](https://img.shields.io/badge/Manual-green)](./powerplatform.md#mspowerplatform32v1-instructions)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Wrong link

Suggested change
[![Manual](https://img.shields.io/badge/Manual-green)](./powerplatform.md#mspowerplatform32v1-instructions)
[![Manual](https://img.shields.io/badge/Manual-green)](./powerplatform.md#mspowerplatform41v1-instructions)

Domain impersonation protection SHOULD be added for key suppliers and partners in both the standard and strict preset policies.

![Automated Check](https://img.shields.io/badge/Automated_Check-blue)
[![Requires Configuration](https://img.shields.io/badge/Requires_Configuration-orange)](../../../docs/configuration/configuration.md#defender-configurations)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Broken anchor

Suggested change
[![Requires Configuration](https://img.shields.io/badge/Requires_Configuration-orange)](../../../docs/configuration/configuration.md#defender-configurations)
[![Requires Configuration](https://img.shields.io/badge/Requires_Configuration-orange)](../../../docs/configuration/configuration.md#defender-configuration)

#### MS.AAD.3.5v1
The authentication methods SMS, Voice Call, and Email One-Time Passcode (OTP) SHALL be disabled.

[![Manual](https://img.shields.io/badge/Manual-green)](./aad.md#msaad35v1-instructions)
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is an automated check.

Suggested change
[![Manual](https://img.shields.io/badge/Manual-green)](./aad.md#msaad35v1-instructions)
![Automated Check](https://img.shields.io/badge/Automated_Check-blue)

Also, MS.AAD.3.5v1 is currently not included in BOD 25-01, but it's one of the conditional policies and will be included once v1.8.0 is published. Since the conditional policies and indicator features are both slated for v1.8.0, we should be fine to preemptively add the BOD 25-01 Requirement indicator here as well.

If phishing-resistant MFA has not been enforced, an alternative MFA method SHALL be enforced for all users.

[![BOD 25-01 Requirement](https://img.shields.io/badge/BOD_25--01_Requirement-red)](https://www.cisa.gov/news-events/directives/bod-25-01-implementation-guidance-implementing-secure-practices-cloud-services)
![Automated Check](https://img.shields.io/badge/Automated_Check-blue)
Copy link
Collaborator

Choose a reason for hiding this comment

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

MS.AAD.3.2v1 is configurable.

Suggested change
![Automated Check](https://img.shields.io/badge/Automated_Check-blue)
![Automated Check](https://img.shields.io/badge/Automated_Check-blue)
[![Configurable](https://img.shields.io/badge/Configurable-purple)](../../../docs/configuration/configuration.md#conditional-access-policy-exclusions)

#### MS.AAD.3.9v1
Device code authentication SHOULD be blocked.

![Automated Check](https://img.shields.io/badge/Automated_Check-blue)
Copy link
Collaborator

Choose a reason for hiding this comment

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

MS.AAD.3.9v1 is configurable but we don't have it listed in the Conditional Access Policy Exclusion section. Could you also add it to the list of supported policies in configuration.md?

Suggested change
![Automated Check](https://img.shields.io/badge/Automated_Check-blue)
![Automated Check](https://img.shields.io/badge/Automated_Check-blue)
[![Configurable](https://img.shields.io/badge/Configurable-purple)](../../../docs/configuration/configuration.md#conditional-access-policy-exclusions)


<!--Policy: MS.EXO.2.2v3; Criticality: SHALL -->
[![BOD 25-01 Requirement](https://img.shields.io/badge/BOD_25--01_Requirement-red)](https://www.cisa.gov/news-events/directives/bod-25-01-implementation-guidance-implementing-secure-practices-cloud-services)
![Automated Check](https://img.shields.io/badge/Automated_Check-blue)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is configurable with PreferredDnsResolver param.

Suggested change
![Automated Check](https://img.shields.io/badge/Automated_Check-blue)
![Automated Check](https://img.shields.io/badge/Automated_Check-blue)
[![Configurable](https://img.shields.io/badge/Configurable-purple)](../../../docs/configuration/configuration.md#automatic-forwarding-to-remote-domains)

#### MS.EXO.3.1v1
DKIM SHOULD be enabled for all domains.

![Automated Check](https://img.shields.io/badge/Automated_Check-blue)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is configurable with PreferredDnsResolver param.

Suggested change
![Automated Check](https://img.shields.io/badge/Automated_Check-blue)
![Automated Check](https://img.shields.io/badge/Automated_Check-blue)
[![Configurable](https://img.shields.io/badge/Configurable-purple)](../../../docs/configuration/configuration.md#automatic-forwarding-to-remote-domains)

A DMARC policy SHALL be published for every second-level domain.

[![BOD 25-01 Requirement](https://img.shields.io/badge/BOD_25--01_Requirement-red)](https://www.cisa.gov/news-events/directives/bod-25-01-implementation-guidance-implementing-secure-practices-cloud-services)
![Automated Check](https://img.shields.io/badge/Automated_Check-blue)
Copy link
Collaborator

Choose a reason for hiding this comment

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

EXO 4.1 - 4.3 Is configurable with PreferredDnsResolver param.

Suggested change
![Automated Check](https://img.shields.io/badge/Automated_Check-blue)
![Automated Check](https://img.shields.io/badge/Automated_Check-blue)
[![Configurable](https://img.shields.io/badge/Configurable-purple)](../../../docs/configuration/configuration.md#automatic-forwarding-to-remote-domains)


### Policies

#### MS.POWERPLATFORM.5.1v1
Copy link
Collaborator

@mitchelbaker-cisa mitchelbaker-cisa Dec 3, 2025

Choose a reason for hiding this comment

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

If you rebase your feature branch with main, powerplatform.md will get updated with the new MS.POWERPLATFORM.6.1v1 policy. This will need to have the "Automated Check" indicator.

#### MS.SHAREPOINT.1.3v1
External sharing SHALL be restricted to approved external domains and/or users in approved security groups per interagency collaboration needs.

![Automated Check](https://img.shields.io/badge/Automated_Check-blue)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Add "BOD 25-01 Requirement" indicator for Sharepoint 1.3, 3.1, 3.2, and 3.3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

baseline-document Issues relating to the text in the baseline documents themselves

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Indicators to Baseline Documents

4 participants