Skip to content
This repository has been archived by the owner on Jan 10, 2025. It is now read-only.

Update ensure-privileged-accounts-with-sensitive-flag.md #693

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

HerbertMauerer
Copy link

There are a few problems with this article:

  • It does not explicitly mention RWDC, I assume that when RODCs are sensitive enough this also applies to RWDCs.
  • The remediation steps in the article only work for users, computers do not have the tab with this flag. E.g. computers can use the “set-adcomputer -AccountNotDelegated $true”.

There are a few problems with this article:
-	It does not explicitly mention RWDC, I assume that when RODCs are sensitive enough this also applies to RWDCs.
-	The remediation steps in the article only work for users, computers do not have the tab with this flag. E.g. computers can use the “set-adcomputer -AccountNotDelegated $true”.
Copy link
Contributor

@HerbertMauerer : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Copy link

Learn Build status updates of commit 3851b73:

⚠️ Validation status: warnings

File Status Preview URL Details
ATPDocs/ensure-privileged-accounts-with-sensitive-flag.md ⚠️Warning Details

ATPDocs/ensure-privileged-accounts-with-sensitive-flag.md

  • Line 34, Column 23: [Warning: disallowed-html-tag - See documentation] HTML tag 'user' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.
  • Line 35, Column 27: [Warning: disallowed-html-tag - See documentation] HTML tag 'computer' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.

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.

For any questions, please:

@ShannonLeavitt
Copy link
Contributor

@LiorShapiraa

Can you review the proposed changes? If you approve, we can add edits including backticks around the code in lines 34-36 to resolve the build warning that appears above.

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 PR review team label Oct 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants