Skip to content

Update dynamic port range in Auditor 10.8 requirements#1022

Merged
pavelshabanov2025 merged 2 commits into
devfrom
pt4/pshabanov/requirements_ports
May 27, 2026
Merged

Update dynamic port range in Auditor 10.8 requirements#1022
pavelshabanov2025 merged 2 commits into
devfrom
pt4/pshabanov/requirements_ports

Conversation

@pavelshabanov2025
Copy link
Copy Markdown
Contributor

@pavelshabanov2025 pavelshabanov2025 commented May 27, 2026

Summary

  • Corrected dynamic TCP port range from 1024-65535 to 49152-65535 in the Netwrix Auditor 10.8 requirements ports page
  • Per Microsoft KB929851, Windows Vista / Server 2008 and later use 49152–65535 as the default dynamic port range (IANA compliant); the old range 1024–65535 was never a standard Microsoft range

Files changed

  • docs/auditor/10.8/requirements/ports.md

Generated with AI

Co-Authored-By: Claude claude-sonnet-4-6 noreply@anthropic.com
Co-Authored-By: Claude Code ai@netwrix.com

- Correct dynamic TCP port range from 1024-65535 to 49152-65535
  in docs/auditor/10.8/requirements/ports.md
- Per Microsoft KB929851, Windows Vista/Server 2008+ uses 49152-65535
  as the default dynamic port range (IANA compliant)

Generated with AI

Co-Authored-By: Claude claude-sonnet-4-6 <noreply@anthropic.com>
Co-Authored-By: Claude Code <ai@netwrix.com>
@github-actions
Copy link
Copy Markdown
Contributor

Documentation PR Review

Editorial Review

docs/auditor/10.8/requirements/ports.md

  • Clarity — Line 88: The range "Dynamic: 49152 -65535" has an inconsistent space before the hyphen but not after. Suggested fix: use either "Dynamic: 49152-65535" (no spaces) or "Dynamic: 49152 - 65535" (spaces on both sides) so the en-dash/hyphen formatting reads as a range rather than a negative number prefix.
  • Completeness — Line 88: The dynamic range was narrowed from 1024-65535 to 49152-65535, but the surrounding prose still says "one dynamic port will be opened automatically" (line 17). Readers comparing the table to the prose may be confused about whether one port or the full ephemeral range needs to be opened. Suggested fix: confirm the prose on line 17 still matches the new range, and consider noting that 49152-65535 is the standard Windows ephemeral port range used by RPC dynamic endpoints.

Summary

2 editorial suggestions across 1 file. Vale and Dale issues are auto-fixed separately.


What to do next:

Comment @claude on this PR followed by your instructions to get help:

  • @claude fix all issues — fix all editorial issues
  • @claude help improve the flow of this document — get writing assistance
  • @claude explain the voice issues — understand why something was flagged

You can ask Claude anything about the review or about Netwrix writing standards.

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

@github-actions
Copy link
Copy Markdown
Contributor

Auto-Fix Summary

12 issues fixed, 3 skipped across 1 files

Category Fixes
Contractions 1
Substitutions 3
FirstPersonPlural (rewrite) 1
FollowTheStepsTo (rewrite) 1
Dale: passive-voice 3
Dale: positional-references 1
Dale: wordiness 2
Skipped (needs manual review) Reason
docs/auditor/10.8/requirements/ports.md:17 — Netwrix.OxfordComma False positive. The phrase 'For outbound rules, create or enable predefined Windows Firewall rules.' contains a prepositional phrase ('For outbound rules') followed by a compound verb ('create or enable') with only two items. Adding an Oxford comma would be grammatically incorrect.
docs/auditor/10.8/requirements/ports.md:9 — Dale: passive-voice 'requires some ports to be opened' — active rewrites ('requires open ports' / 'requires you to open ports') subtly change meaning or agent; left as-is to preserve author intent
docs/auditor/10.8/requirements/ports.md:78 — Dale: passive-voice 'where Netwrix Auditor client is installed (source)' — the parenthetical '(source)' depends on this phrasing for the table-reading guidance; rewriting risks losing the source/target framing

Ask @claude on this PR if you'd like an explanation of any fix.

@pavelshabanov2025 pavelshabanov2025 merged commit 1b15e7b into dev May 27, 2026
10 checks passed
@pavelshabanov2025 pavelshabanov2025 deleted the pt4/pshabanov/requirements_ports branch May 27, 2026 14:43
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.

3 participants