Skip to content

Correct home installer port requirements and advanced examples - #222

Merged
githubsaturn merged 1 commit into
masterfrom
docs-next-port-review-20260926
Sep 27, 2026
Merged

githubsaturn merged 1 commit into
masterfrom
docs-next-port-review-20260926

Conversation

@githubsaturn

@githubsaturn githubsaturn commented Sep 27, 2026 •

Copy link
Copy Markdown
Collaborator

Review findings addressed

  • Temporarily forward TCP 3000 with 80 and 443 before the default public-IP installer runs on a home server; remove the 3000 rule after dashboard HTTPS works. Explain the NAT loopback caveat and local/private path.
  • State that the default NGINX service publishes UDP 443 even when it is blocked at the external firewall.
  • Make the global DNS-01 example request only the exact domain, with a note on wildcard rule matching.
  • Describe NetData alert delivery verification without assuming a dashboard test button.

Validation

Docusaurus clean build succeeded for en, es-ES, and zh-CN. Combined-site composition and HTTP smoke passed. Certbot JSON example parsed and requests only the exact domain. git diff --check passed. /docs-next remains an unlinked preview; /docs is unchanged.

Summary by CodeRabbit

  • Documentation
    • Clarified that the example Certbot command requests a certificate for the base domain only; wildcard certificates require both the base domain and wildcard name.
    • Updated home-server setup steps to install Docker first, temporarily forward ports 80, 443, and 3000, verify access from outside the LAN, and remove public port 3000 forwarding once dashboard HTTPS works.
    • Listed UDP 443 as a default NGINX port and noted it must be allowed through the firewall for HTTP/3.
    • Updated NetData guidance to verify that alert notifications reach their destination.

@coderabbitai

coderabbitai Bot commented Sep 27, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 369fea86-db3e-4dc6-8c98-a50880dfa71f

📥 Commits

Reviewing files that changed from the base of the PR and between 7cb6f35 and 45b982c.

📒 Files selected for processing (4)
  • content/en/docs-next/domains/advanced-certbot.md
  • content/en/docs-next/guides/home-server.md
  • content/en/docs-next/reference/ports.md
  • content/en/docs-next/server/netdata.md
 _________________________________________________________________________
< Performing code review exorcisms to the eerie strains of Tubular Bells. >
 -------------------------------------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@githubsaturn
githubsaturn merged commit e7cab27 into master Sep 27, 2026
1 of 2 checks passed
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.

1 participant