Skip to content

Update docs auditor fsa windows ldap port#414

Merged
nwnikacc merged 4 commits intodevfrom
update-docs-auditor-winfsa
Mar 6, 2026
Merged

Update docs auditor fsa windows ldap port#414
nwnikacc merged 4 commits intodevfrom
update-docs-auditor-winfsa

Conversation

@nwnikacc
Copy link
Contributor

@nwnikacc nwnikacc commented Mar 6, 2026

Doc Task 311803: Enable LDAP encryption for FSA and NLA

nwnikacc added 2 commits March 6, 2026 11:28
Added note about enabled LDAP packet encryption for port 389.
Updated the description for port 389 to include LDAP packet encryption. 10.8
@nwnikacc nwnikacc requested a review from a team as a code owner March 6, 2026 10:37
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Issues in PR changes

docs/auditor/10.7/configuration/fileservers/windows/ports.md

Line 24: Awkward word order in added phrase. "enabled LDAP packet encryption" should be "with LDAP packet encryption enabled" for grammatical clarity. The preposition "with" and proper positioning of "enabled" make the phrase clearer.

docs/auditor/10.8/configuration/fileservers/windows/ports.md

Line 24: Awkward word order in added phrase. "enabled LDAP packet encryption" should be "with LDAP packet encryption enabled" for grammatical clarity. The preposition "with" and proper positioning of "enabled" make the phrase clearer.


To apply suggested fixes to the updated documentation, individually or in bulk, comment @claude on this PR followed by your instructions (@claude fix all issues or @claude fix all linting issues or @claude fix only the spelling errors).

To review the updated documentation for preexisting issues, comment @claude on this PR followed by your instructions (@claude detect preexisting issues).

Note: Automated fixes are only available for branches in this repository, not forks.

nwnikacc and others added 2 commits March 6, 2026 11:41
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Issues in PR changes

None.


To apply suggested fixes to the updated documentation, individually or in bulk, comment @claude on this PR followed by your instructions (@claude fix all issues or @claude fix all linting issues or @claude fix only the spelling errors).

To review the updated documentation for preexisting issues, comment @claude on this PR followed by your instructions (@claude detect preexisting issues).

Note: Automated fixes are only available for branches in this repository, not forks.

@nwnikacc nwnikacc merged commit 54cc581 into dev Mar 6, 2026
7 checks passed
@nwnikacc nwnikacc deleted the update-docs-auditor-winfsa branch March 6, 2026 10:46
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Issues in PR changes

docs/auditor/10.7/configuration/fileservers/windows/ports.md

Line 24: The phrase "(with LDAP packet encryption enabled)" uses passive voice. The style guide requires active voice throughout. Change to "(uses encrypted LDAP packets)" or "(with encrypted LDAP packets)" for active construction.

docs/auditor/10.8/configuration/fileservers/windows/ports.md

Line 24: The phrase "(with LDAP packet encryption enabled)" uses passive voice. The style guide requires active voice throughout. Change to "(uses encrypted LDAP packets)" or "(with encrypted LDAP packets)" for active construction.


To apply suggested fixes to the updated documentation, individually or in bulk, comment @claude on this PR followed by your instructions (@claude fix all issues or @claude fix all linting issues or @claude fix only the spelling errors).

To review the updated documentation for preexisting issues, comment @claude on this PR followed by your instructions (@claude detect preexisting issues).

Note: Automated fixes are only available for branches in this repository, not forks.

| -------------------------- | -------- | ---------------------- | ------------------ | ----------------------------------------------------------------------------------------------------------- |
| Windows File Servers | | | | |
| 389 | TCP/UDP | Netwrix Auditor Server | Domain controllers | LDAP DC query Account resolve |
| 389 | TCP/UDP | Netwrix Auditor Server | Domain controllers | LDAP DC query Account resolve (with LDAP packet encryption enabled) |
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| 389 | TCP/UDP | Netwrix Auditor Server | Domain controllers | LDAP DC query Account resolve (with LDAP packet encryption enabled) |
| 389 | TCP/UDP | Netwrix Auditor Server | Domain controllers | LDAP DC query Account resolve (with encrypted LDAP packets) |

| -------------------------- | -------- | ---------------------- | ------------------ | ----------------------------------------------------------------------------------------------------------- |
| Windows File Servers | | | | |
| 389 | TCP/UDP | Netwrix Auditor Server | Domain controllers | LDAP DC query Account resolve |
| 389 | TCP/UDP | Netwrix Auditor Server | Domain controllers | LDAP DC query Account resolve (enabled LDAP packet encryption) |
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| 389 | TCP/UDP | Netwrix Auditor Server | Domain controllers | LDAP DC query Account resolve (enabled LDAP packet encryption) |
| 389 | TCP/UDP | Netwrix Auditor Server | Domain controllers | LDAP DC query Account resolve (with encrypted LDAP packets) |

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants