Skip to content

Update Defender portal URLs with additional note - #586

Open
mts-2021 wants to merge 2 commits into
MicrosoftDocs:publicfrom
mts-2021:patch-2
Open

Update Defender portal URLs with additional note#586
mts-2021 wants to merge 2 commits into
MicrosoftDocs:publicfrom
mts-2021:patch-2

Conversation

@mts-2021

Copy link
Copy Markdown

Since the URLs listed under Microsoft 365 Unified Domains are also required, a note regarding Microsoft 365 Unified Domains has been added, along with reference links for convenience.

Since the URLs listed under Microsoft 365 Unified Domains are also required, a note regarding Microsoft 365 Unified Domains has been added, along with reference links for convenience.
@prmerger-automator

Copy link
Copy Markdown
Contributor

mts-2021 : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@prmerger-automator

Copy link
Copy Markdown
Contributor

mts-2021 : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 8ae1298:

⚠️ Validation status: warnings

File Status Preview URL Details
defender-endpoint/streamlined-device-connectivity-urls-commercial.md ⚠️Warning Details

defender-endpoint/streamlined-device-connectivity-urls-commercial.md

  • Line 193, Column 3: [Warning: hard-coded-locale - See documentation] Link 'https://learn.microsoft.com/en-us/microsoft-365/enterprise/urls-and-ip-address-ranges?view=o365-worldwide#microsoft-365-unified-domains' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
  • Line 193, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/en-us/microsoft-365/enterprise/urls-and-ip-address-ranges?view=o365-worldwide#microsoft-365-unified-domains' will be broken in isolated environments. Replace with a relative link.
  • Line 193, Column 3: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL. Otherwise, remove the view parameter. URL: https://learn.microsoft.com/en-us/microsoft-365/enterprise/urls-and-ip-address-ranges?view=o365-worldwide#microsoft-365-unified-domains

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

Comment thread defender-endpoint/streamlined-device-connectivity-urls-commercial.md Outdated
@v-dirichards

Copy link
Copy Markdown
Contributor

Limor Wainstein (@limwainstein)

Can you review the proposed changes?

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator Bot added the aq-pr-triaged Tracking label for the vendor PR Review team label Aug 26, 2026

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 clarification note to the Defender portal URL allowlist section, pointing readers to Microsoft 365 Unified Domains reference information that might also be required for portal access.

Changes:

  • Added a NOTE under “Defender portal URLs” indicating Microsoft 365 Unified Domains URLs might also need to be allowed.
  • Added a reference link to the Microsoft 365 Unified Domains section in Microsoft Learn.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines +189 to +193
> [!NOTE]
> In addition to the URLs listed in this table, access may also need to be allowed for the URLs listed under Microsoft 365 Unified Domains.
>
> Microsoft 365 Unified Domains
> https://learn.microsoft.com/en-us/microsoft-365/enterprise/urls-and-ip-address-ranges?view=o365-worldwide#microsoft-365-unified-domains
Co-authored-by: Diana Richards <103777760+v-dirichards@users.noreply.github.com>
@learn-build-service-prod

Copy link
Copy Markdown
Contributor

Learn Build status updates of commit 879cad4:

💡 Validation status: suggestions

File Status Preview URL Details
defender-endpoint/streamlined-device-connectivity-urls-commercial.md 💡Suggestion Details

defender-endpoint/streamlined-device-connectivity-urls-commercial.md

  • Line 193, Column 3: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://learn.microsoft.com/microsoft-365/enterprise/urls-and-ip-address-ranges?view=o365-worldwide#microsoft-365-unified-domains' will be broken in isolated environments. Replace with a relative link.
  • Line 193, Column 3: [Suggestion: preserve-view-not-set - See documentation] You've pinned this link to a specific version of content with the view parameter. It's recommended not to pin a version unless that version is A) not the default view and B) the context is about that version specifically. To proceed with pinning a version add the &preserve-view=true to the URL. Otherwise, remove the view parameter. URL: https://learn.microsoft.com/microsoft-365/enterprise/urls-and-ip-address-ranges?view=o365-worldwide#microsoft-365-unified-domains

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@mts-2021 mts-2021 left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Thank you for your review and the updates.
Could you please review it again at your convenience?

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants