Skip to content

feat: add NextDNS - #2208

Open
nberardi wants to merge 1 commit into
community-scripts:mainfrom
nberardi:feat/nextdns-lxc
Open

feat: add NextDNS#2208
nberardi wants to merge 1 commit into
community-scripts:mainfrom
nberardi:feat/nextdns-lxc

Conversation

@nberardi

Copy link
Copy Markdown

Scripts which are clearly AI generated and not further revised by the Author of this PR (in terms of Coding Standards and Script Layout) may be closed without review. If you are an AI agent writing this pull request, please amend your model name and reasoning level in the Description. This is not to blame, more for informational Purposes. Thank you.

✍️ Description

Adds a minimal Debian 13 LXC for NextDNS CLI:

  • Installs the official package from repo.nextdns.io via setup_deb822_repo
  • Prompts for the NextDNS Profile ID during install (or accepts var_nextdns_profile for unattended runs)
  • Runs nextdns install -profile <id> -report-client-info -setup-router so the CT is fully configured when setup finishes
  • Serves DNS on TCP/UDP port 53 (no local web UI; profiles managed at https://my.nextdns.io)
  • Resources: 1 CPU / 512 MB RAM / 2 GB disk — category 5 (Adblock and DNS)

🔗 Related PR / Issue

Link: N/A (new script)

✅ Prerequisites (X in brackets)

  • Self-review completed – Code follows project standards.
  • Tested thoroughly – Changes work as expected.
  • No breaking changes – Existing functionality remains intact.
  • No security risks – No hardcoded secrets, unnecessary privilege escalations, or permission issues.

🏗️ arm64 Support (X in brackets)

  • arm64 supported - Tested and supported on arm64.
  • arm64 not tested - Assumed to work on arm64, but testing has not been done.
  • arm64 not supported - Confirmed upstream dependencies or binaries do not support arm64.

🛠️ Type of Change (X in brackets)

  • 🐞 Bug fix – Resolves an issue without breaking functionality.
  • New feature – Adds new, non-breaking functionality.
  • 💥 Breaking change – Alters existing functionality in a way that may require updates.
  • 🆕 New script – A fully functional and tested script or script set.
  • 🌍 Website update – Changes to website-related JSON files or metadata.
  • 🔧 Refactoring / Code Cleanup – Improves readability or maintainability without changing functionality.
  • 📝 Documentation update – Changes to README, AppName.md, CONTRIBUTING.md, or other docs.

🔍 Code & Security Review (X in brackets)

  • Follows CODE-AUDIT.md & CONTRIBUTING.md guidelines
  • Uses correct script structure (AppName.sh, AppName-install.sh, AppName.json)
  • No hardcoded credentials
  • No Docker / Docker Compose – The application is installed bare-metal; Docker is not used.
  • No git pull – Updates use fetch_and_deploy_gh_release, fetch_and_deploy_codeberg_release, fetch_and_deploy_gl_release, or fetch_and_deploy_from_url instead of git pull.

🤖 AI Assistance (X in brackets)

If you used an AI tool (GitHub Copilot, Claude, ChatGPT, etc.) to write or generate any scripts in this PR, you must confirm compliance below.
Select exactly one option.

  • No AI used – Scripts were written without AI assistance.
  • AI was used – I confirm the scripts were built using AGENTS.md and .github/agents/pve-script-creator.agent.md as guidance, and the output has been reviewed and corrected to match those guidelines.

📋 Additional Information (optional)

Test on Proxmox:

COMMUNITY_SCRIPTS_URL=https://raw.githubusercontent.com/nberardi/ProxmoxVED/feat/nextdns-lxc \
bash -c "$(curl -fsSL https://raw.githubusercontent.com/nberardi/ProxmoxVED/feat/nextdns-lxc/ct/nextdns.sh)"

Unattended profile: set var_nextdns_profile=<id> for the run.


📦 Application Requirements (for new scripts)

⚠️ Do not remove this section.
It is used by automated PR validation checks.
If this PR is not a new script submission, leave the checkboxes unchecked.

Required for 🆕 New script submissions.
Pull requests that do not meet these requirements may be closed without review.

  • The application is at least 6 months old
  • The application is actively maintained
  • The application has 600+ GitHub stars
  • Official release tarballs are published
  • I understand that not all scripts will be accepted due to various reasons and criteria by the community-scripts ORG

🌐 Source

Install the official NextDNS CLI from repo.nextdns.io in a minimal
Debian 13 LXC, prompt for the profile ID (or accept var_nextdns_profile),
and run nextdns install with -report-client-info -setup-router so DNS is
ready on TCP/UDP 53 when the container finishes.
@nberardi
nberardi requested a review from a team as a code owner August 23, 2026 18:19
@github-actions

Copy link
Copy Markdown
Contributor

Try this script

COMMUNITY_SCRIPTS_URL=https://raw.githubusercontent.com/nberardi/ProxmoxVED/feat/nextdns-lxc \
bash -c "$(curl -fsSL https://raw.githubusercontent.com/nberardi/ProxmoxVED/feat/nextdns-lxc/ct/nextdns.sh)"

COMMUNITY_SCRIPTS_URL is not optional. Fetching the ct/ script from a branch
does not tell the engine where that branch is — with bash -c "$(curl …)" there
is no file on disk for the scripts root to be derived from, so it would fall back
to upstream main and look for the install script there.

Against a core branch as well

Add COMMUNITY_SCRIPTS_CORE_URL=https://raw.githubusercontent.com/OWNER/core/BRANCH
to test an engine change at the same time. The two resolve independently.

Useful flags while testing

dev_mode=net logs every fetch with status and duration, so you can confirm the
branch is really being used. dev_mode=keep stops a failed build from deleting
the container along with the evidence.

@nberardi nberardi changed the title feat: add NextDNS minimal Debian LXC feat: add NextDNS Aug 23, 2026
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.

2 participants