chore: bump version to 0.7.1

Replace text-based chain trace with a visual diagram showing the
verification flow from cloudflare.com through .com TLD to root
trust anchor. Matches site color palette and typography.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

.SRCINFO

@@ -1,19 +0,0 @@
-pkgbase = numa-git
-	pkgdesc = Portable DNS resolver in Rust — .numa local domains, ad blocking, developer overrides, DNS-over-HTTPS
-	pkgver = 0.10.1.r0.g0000000
-	pkgrel = 1
-	url = https://github.com/razvandimescu/numa
-	arch = x86_64
-	license = MIT
-	options = !lto
-	makedepends = cargo
-	makedepends = git
-	depends = gcc-libs
-	depends = glibc
-	provides = numa
-	conflicts = numa
-	backup = etc/numa.toml
-	source = numa::git+https://github.com/razvandimescu/numa.git
-	sha256sums = SKIP
-
-pkgname = numa-git

.github/dependabot.yml

@@ -1,34 +0,0 @@
-version: 2
-updates:
-  - package-ecosystem: "cargo"
-    directory: "/"
-    schedule:
-      interval: "monthly"
-    commit-message:
-      prefix: "chore(deps)"
-    groups:
-      minor-and-patch:
-        patterns: ["*"]
-        update-types: ["minor", "patch"]
-
-  - package-ecosystem: "github-actions"
-    directory: "/"
-    schedule:
-      interval: "monthly"
-    commit-message:
-      prefix: "chore(deps)"
-    groups:
-      minor-and-patch:
-        patterns: ["*"]
-        update-types: ["minor", "patch"]
-
-  - package-ecosystem: "docker"
-    directory: "/"
-    schedule:
-      interval: "monthly"
-    commit-message:
-      prefix: "chore(deps)"
-    groups:
-      minor-and-patch:
-        patterns: ["*"]
-        update-types: ["minor", "patch"]

.github/workflows/ci.yml

@@ -13,7 +13,7 @@ jobs:
   check:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v6
+      - uses: actions/checkout@v4
       - uses: dtolnay/rust-toolchain@stable
         with:
           components: rustfmt, clippy
@@ -27,31 +27,13 @@ jobs:
       - name: audit
         run: cargo install cargo-audit && cargo audit
 
-  check-macos:
-    runs-on: macos-latest
-    steps:
-      - uses: actions/checkout@v6
-      - uses: dtolnay/rust-toolchain@stable
-      - uses: Swatinem/rust-cache@v2
-      - name: clippy
-        run: cargo clippy -- -D warnings
-      - name: test
-        run: cargo test
-
   check-windows:
     runs-on: windows-latest
     steps:
-      - uses: actions/checkout@v6
+      - uses: actions/checkout@v4
       - uses: dtolnay/rust-toolchain@stable
       - uses: Swatinem/rust-cache@v2
       - name: build
         run: cargo build
       - name: clippy
         run: cargo clippy -- -D warnings
-      - name: test
-        run: cargo test
-      - name: Upload binary
-        uses: actions/upload-artifact@v7
-        with:
-          name: numa-windows-x86_64
-          path: target/debug/numa.exe

.github/workflows/homebrew-bump.yml

@@ -1,77 +0,0 @@
-name: Bump Homebrew Tap
-
-on:
-  workflow_call:
-    inputs:
-      version:
-        description: 'Version to bump (e.g. 0.10.0 or v0.10.0)'
-        type: string
-        required: true
-  workflow_dispatch:
-    inputs:
-      version:
-        description: 'Version to bump (e.g. 0.10.0 or v0.10.0)'
-        required: true
-
-permissions:
-  contents: read
-
-jobs:
-  bump:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v6
-
-      - name: Determine version
-        id: ver
-        env:
-          INPUT_VERSION: ${{ inputs.version }}
-        run: |
-          V="${INPUT_VERSION#v}"
-          echo "version=$V" >> "$GITHUB_OUTPUT"
-
-      - name: Fetch sha256 checksums from release assets
-        id: shas
-        env:
-          V: ${{ steps.ver.outputs.version }}
-        run: |
-          set -euo pipefail
-          base="https://github.com/razvandimescu/numa/releases/download/v${V}"
-          for t in macos-aarch64 macos-x86_64 linux-aarch64 linux-x86_64; do
-            sha=$(curl -fsSL "${base}/numa-${t}.tar.gz.sha256" | awk '{print $1}')
-            if [ -z "$sha" ]; then
-              echo "ERROR: failed to fetch sha256 for $t" >&2
-              exit 1
-            fi
-            key=$(echo "$t" | tr '[:lower:]-' '[:upper:]_')
-            echo "SHA_${key}=${sha}" >> "$GITHUB_ENV"
-          done
-
-      - name: Clone homebrew-tap
-        env:
-          HOMEBREW_TAP_GITHUB_TOKEN: ${{ secrets.HOMEBREW_TAP_GITHUB_TOKEN }}
-        run: |
-          git clone "https://x-access-token:${HOMEBREW_TAP_GITHUB_TOKEN}@github.com/razvandimescu/homebrew-tap.git" tap
-
-      - name: Update formula
-        env:
-          VERSION: ${{ steps.ver.outputs.version }}
-        run: |
-          python3 scripts/update-homebrew-formula.py tap/numa.rb
-          echo "--- updated numa.rb ---"
-          cat tap/numa.rb
-
-      - name: Commit and push
-        working-directory: tap
-        env:
-          V: ${{ steps.ver.outputs.version }}
-        run: |
-          if git diff --quiet; then
-            echo "numa.rb already at v${V}, nothing to commit"
-            exit 0
-          fi
-          git config user.name "github-actions[bot]"
-          git config user.email "41898282+github-actions[bot]@users.noreply.github.com"
-          git add numa.rb
-          git commit -m "chore: bump numa to v${V}"
-          git push origin main

.github/workflows/publish-aur.yml

@@ -1,159 +0,0 @@
-# `publish-aur.yml` - Arch Linux AUR Package Workflow
-# --------------------
-# This workflow automates the validation and publishing of the 'numa-git' package to the
-# Arch User Repository (AUR). The AUR is a community-driven repository for Arch Linux users.
-#
-# Workflow Overview:
-# 1. Validate: Builds and tests the package for Arch Linux x86_64 using a clean
-#    Arch Linux container.
-# 2. Audit: Checks Rust dependencies for known security vulnerabilities using
-#    'cargo-audit'.
-# 3. Publish: If on the 'main' branch, it pushes the updated PKGBUILD and
-#    .SRCINFO to the AUR.
-#
-# Security Best Practices:
-# - SHA Pinning: All GitHub Actions are pinned to a full-length commit SHA (e.g., v6.0.2 @ SHA)
-#   to ensure the code is immutable and protects against supply-chain attacks where a tag
-#   might be maliciously moved to a compromised commit.
-# - SSH Hygiene: Uses ssh-agent to keep the private key in memory rather than on disk.
-# - Audit: Runs 'cargo audit' to prevent publishing known vulnerable dependencies.
-
-name: Publish - Arch Linux AUR Package
-
-on:
-  push:
-    branches: [main]
-  workflow_dispatch:
-
-permissions:
-  contents: read
-
-jobs:
-  # The 'validate' job ensures that the PKGBUILD is correct and the software builds/tests
-  # successfully on Arch Linux before we attempt to publish it.
-  validate:
-    name: Validate PKGBUILD (${{ matrix.arch }})
-    runs-on: ubuntu-latest
-    strategy:
-      fail-fast: false
-      matrix:
-        arch: [x86_64]
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
-
-      - name: Build and Test Package
-        timeout-minutes: 60
-        env:
-          AUR_PKGNAME: ${{ secrets.AUR_PACKAGE_NAME }}
-        run: |
-          # We use a temporary directory to avoid Docker permission issues with the workspace.
-          mkdir -p build-dir
-          cp PKGBUILD build-dir/
-
-          docker run --rm -v $PWD/build-dir:/pkg -w /pkg archlinux:latest /bin/bash -c "
-            # ARCH LINUX SECURITY REQUIREMENT:
-            # 'makepkg' (the tool that builds Arch packages) refuses to run as root for safety.
-            # We must create a standard user and give them sudo access.
-
-            # Install build-time dependencies.
-            # 'base-devel' includes essential tools like gcc, make, and binutils.
-            # Install 'rust' directly to avoid the interactive virtual-package
-            # prompt for 'cargo' on current Arch images.
-            pacman -Syu --noconfirm --needed base-devel rust git sudo cargo-audit
-
-            useradd -m builduser
-            chown -R builduser:builduser /pkg
-
-            # Allow the build user to install dependencies during the build process.
-            echo 'builduser ALL=(ALL) NOPASSWD: ALL' > /etc/sudoers.d/builduser
-
-            # Fetch the source tree first so pkgver() and cargo-audit have a
-            # real Cargo.lock to inspect.
-            sudo -u builduser makepkg -o --nobuild --nocheck --nodeps --noprepare
-
-            # SECURITY AUDIT:
-            # Fail early if any dependencies have known security vulnerabilities.
-            sudo -u builduser sh -lc 'cd /pkg/src/numa && cargo audit'
-
-            # BUILD & TEST:
-            # 'makepkg -s' will:
-            # 1. Download source files (cloning this repo)
-            # 2. Run prepare(), build(), and check() (running cargo test)
-            # 3. Create the final .pkg.tar.zst package
-            sudo -u builduser makepkg -s --noconfirm
-          "
-
-  # The 'publish' job updates the AUR repository with our latest PKGBUILD and .SRCINFO.
-  publish:
-    name: Publish to AUR
-    needs: validate
-    runs-on: ubuntu-latest
-    if: github.event_name == 'push' && github.ref == 'refs/heads/main'
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
-
-      # Securely configure SSH for AUR access.
-      - name: Configure SSH
-        run: |
-          mkdir -p ~/.ssh
-          # Official AUR Ed25519 fingerprint (prevents Man-in-the-Middle attacks).
-          echo "aur.archlinux.org ssh-ed25519 AAAAC3NzaC1lZDI1NTE5AAAAIEuBKrPzbawxA/k2g6NcyV5jmqwJ2s+zpgZGZ7tpLIcN" >> ~/.ssh/known_hosts
-
-          # Use ssh-agent to keep the private key in memory rather than writing it to disk.
-          eval $(ssh-agent -s)
-          echo "${{ secrets.AUR_SSH_PRIVATE_KEY }}" | tr -d '\r' | ssh-add -
-
-          # Export the agent socket so subsequent 'git' commands can use it.
-          echo "SSH_AUTH_SOCK=$SSH_AUTH_SOCK" >> $GITHUB_ENV
-          echo "SSH_AGENT_PID=$SSH_AGENT_PID" >> $GITHUB_ENV
-
-      - name: Push to AUR
-        env:
-          AUR_PKGNAME: ${{ secrets.AUR_PACKAGE_NAME }}
-          AUR_EMAIL: ${{ secrets.AUR_EMAIL }}
-          AUR_USER: ${{ secrets.AUR_USERNAME }}
-        run: |
-          # AUR repos are managed via Git. Each package has its own repo at:
-          # ssh://aur@aur.archlinux.org/<package-name>.git
-          git clone ssh://aur@aur.archlinux.org/$AUR_PKGNAME.git aur-repo
-
-          cp PKGBUILD aur-repo/
-          cd aur-repo
-
-          # METADATA GENERATION:
-          # '.SRCINFO' is a machine-readable version of the PKGBUILD.
-          # We must run this as a non-root user ('builduser') inside the container.
-          docker run --rm -v $(pwd):/pkg archlinux:latest /bin/bash -c "
-            pacman -Syu --noconfirm --needed binutils git sudo
-            useradd -m builduser
-            chown -R builduser:builduser /pkg
-            cd /pkg
-            sudo -u builduser git config --global --add safe.directory '*'
-            # makepkg -od fetches the source first so pkgver() can calculate the version.
-            # --noprepare skips the prepare() function, which invokes cargo and would
-            # otherwise require a full rust toolchain in this metadata-only container.
-            # pkgver() runs before prepare(), so .SRCINFO still gets the correct version.
-            sudo -u builduser makepkg -od --noprepare && sudo -u builduser makepkg --printsrcinfo > .SRCINFO
-          "
-
-          # Reclaim ownership: the in-container 'chown -R builduser:builduser /pkg'
-          # propagates through the bind mount, leaving .git/ owned by the container's
-          # builduser UID. Without this, subsequent 'git config' on the host fails with
-          # "could not lock config file .git/config: Permission denied".
-          sudo chown -R "$(id -u):$(id -g)" .
-
-          # Set the commit identity using secrets for security and auditability.
-          git config user.name "$AUR_USER"
-          git config user.email "$AUR_EMAIL"
-
-          # Stage and commit both the human-readable PKGBUILD and machine-readable .SRCINFO.
-          git add PKGBUILD .SRCINFO
-
-          if ! git diff --cached --quiet; then
-            git commit -m "chore: update PKGBUILD to ${{ github.sha }}"
-            git push origin master
-          else
-            echo "No changes to commit (metadata and PKGBUILD are already up-to-date)."
-          fi

.github/workflows/release.yml

@@ -31,7 +31,7 @@ jobs:
 
     runs-on: ${{ matrix.os }}
     steps:
-      - uses: actions/checkout@v6
+      - uses: actions/checkout@v4
 
       - name: Install Rust
         uses: dtolnay/rust-toolchain@stable
@@ -70,7 +70,7 @@ jobs:
           (Get-FileHash "${{ matrix.name }}.zip" -Algorithm SHA256).Hash.ToLower() + "  ${{ matrix.name }}.zip" | Out-File "${{ matrix.name }}.zip.sha256" -Encoding ascii
 
       - name: Upload artifact
-        uses: actions/upload-artifact@v7
+        uses: actions/upload-artifact@v4
         with:
           name: ${{ matrix.name }}
           path: |
@@ -82,7 +82,7 @@ jobs:
   publish:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v6
+      - uses: actions/checkout@v4
 
       - name: Install Rust
         uses: dtolnay/rust-toolchain@stable
@@ -96,7 +96,7 @@ jobs:
     needs: [build, publish]
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/download-artifact@v8
+      - uses: actions/download-artifact@v4
         with:
           merge-multiple: true
 
@@ -108,10 +108,3 @@ jobs:
             *.tar.gz
             *.zip
             *.sha256
-
-  bump-homebrew:
-    needs: release
-    uses: ./.github/workflows/homebrew-bump.yml
-    with:
-      version: ${{ github.ref_name }}
-    secrets: inherit

.github/workflows/static.yml

@@ -30,18 +30,18 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
-        uses: actions/checkout@v6
+        uses: actions/checkout@v4
       - name: Install pandoc
         run: sudo apt-get install -y pandoc
       - name: Generate blog HTML
         run: make blog
       - name: Setup Pages
-        uses: actions/configure-pages@v6
+        uses: actions/configure-pages@v5
       - name: Upload artifact
-        uses: actions/upload-pages-artifact@v4
+        uses: actions/upload-pages-artifact@v3
         with:
           # Upload entire repository
           path: './site'
       - name: Deploy to GitHub Pages
         id: deployment
-        uses: actions/deploy-pages@v5
+        uses: actions/deploy-pages@v4

.gitignore

@@ -1,5 +1,4 @@
 /target
-/build-dir
 CLAUDE.md
 docs/
 site/blog/posts/

Cargo.lock

@@ -17,15 +17,6 @@ dependencies = [
  "memchr",
 ]
 
-[[package]]
-name = "alloca"
-version = "0.4.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "e5a7d05ea6aea7e9e64d25b9156ba2fee3fdd659e34e41063cd2fc7cd020d7f4"
-dependencies = [
- "cc",
-]
-
 [[package]]
 name = "anes"
 version = "0.1.6"
@@ -84,18 +75,18 @@ dependencies = [
 
 [[package]]
 name = "arc-swap"
-version = "1.9.1"
+version = "1.9.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "6a3a1fd6f75306b68087b831f025c712524bcb19aad54e557b1129cfa0a2b207"
+checksum = "a07d1f37ff60921c83bdfc7407723bdefe89b44b98a9b772f225c8f9d67141a6"
 dependencies = [
  "rustversion",
 ]
 
 [[package]]
 name = "asn1-rs"
-version = "0.7.1"
+version = "0.6.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "56624a96882bb8c26d61312ae18cb45868e5a9992ea73c58e45c3101e56a1e60"
+checksum = "5493c3bedbacf7fd7382c6346bbd66687d12bbaad3a89a2d2c303ee6cf20b048"
 dependencies = [
  "asn1-rs-derive",
  "asn1-rs-impl",
@@ -103,15 +94,15 @@ dependencies = [
  "nom",
  "num-traits",
  "rusticata-macros",
- "thiserror",
+ "thiserror 1.0.69",
  "time",
 ]
 
 [[package]]
 name = "asn1-rs-derive"
-version = "0.6.0"
+version = "0.5.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "3109e49b1e4909e9db6515a30c633684d68cdeaa252f215214cb4fa1a5bfee2c"
+checksum = "965c2d33e53cb6b267e148a4cb0760bc01f4904c1cd4bb4002a085bb016d1490"
 dependencies = [
  "proc-macro2",
  "quote",
@@ -377,24 +368,25 @@ dependencies = [
 
 [[package]]
 name = "criterion"
-version = "0.8.2"
+version = "0.5.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "950046b2aa2492f9a536f5f4f9a3de7b9e2476e575e05bd6c333371add4d98f3"
+checksum = "f2b12d017a929603d80db1831cd3a24082f8137ce19c69e6447f54f5fc8d692f"
 dependencies = [
- "alloca",
  "anes",
  "cast",
  "ciborium",
  "clap",
  "criterion-plot",
+ "is-terminal",
  "itertools",
  "num-traits",
+ "once_cell",
  "oorandom",
- "page_size",
  "plotters",
  "rayon",
  "regex",
  "serde",
+ "serde_derive",
  "serde_json",
  "tinytemplate",
  "walkdir",
@@ -402,9 +394,9 @@ dependencies = [
 
 [[package]]
 name = "criterion-plot"
-version = "0.8.2"
+version = "0.5.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "d8d80a2f4f5b554395e47b5d8305bc3d27813bacb73493eb1001e8f76dae29ea"
+checksum = "6b50826342786a51a89e2da3a28f1c32b06e387201bc2d19791f622c673706b1"
 dependencies = [
  "cast",
  "itertools",
@@ -449,9 +441,9 @@ checksum = "d7a1e2f27636f116493b8b860f5546edb47c8d8f8ea73e1d2a20be88e28d1fea"
 
 [[package]]
 name = "der-parser"
-version = "10.0.0"
+version = "9.0.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "07da5016415d5a3c4dd39b11ed26f915f52fc4e0dc197d87908bc916e51bc1a6"
+checksum = "5cd0a5c643689626bec213c4d8bd4d96acc8ffdb4ad4bb6bc16abf27d5f4b553"
 dependencies = [
  "asn1-rs",
  "displaydoc",
@@ -710,6 +702,12 @@ version = "0.16.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "841d1cc9bed7f9236f321df977030373f4a4163ae1a7dbfe1a51a2c1a51d9100"
 
+[[package]]
+name = "hermit-abi"
+version = "0.5.2"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "fc0fef456e4baa96da950455cd02c081ca953b141298e41db3fc7e36b1da849c"
+
 [[package]]
 name = "http"
 version = "1.4.0"
@@ -757,9 +755,9 @@ checksum = "df3b46402a9d5adb4c86a0cf463f42e19994e3ee891101b1841f30a545cb49a9"
 
 [[package]]
 name = "hyper"
-version = "1.9.0"
+version = "1.8.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "6299f016b246a94207e63da54dbe807655bf9e00044f73ded42c3ac5305fbcca"
+checksum = "2ab2d4f250c3d7b1c9fcdff1cece94ea4e2dfbec68614f7b87cb205f24ca9d11"
 dependencies = [
  "atomic-waker",
  "bytes",
@@ -772,6 +770,7 @@ dependencies = [
  "httpdate",
  "itoa",
  "pin-project-lite",
+ "pin-utils",
  "smallvec",
  "tokio",
  "want",
@@ -811,7 +810,7 @@ dependencies = [
  "libc",
  "percent-encoding",
  "pin-project-lite",
- "socket2",
+ "socket2 0.6.3",
  "tokio",
  "tower-service",
  "tracing",
@@ -945,6 +944,17 @@ dependencies = [
  "serde",
 ]
 
+[[package]]
+name = "is-terminal"
+version = "0.4.17"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "3640c1c38b8e4e43584d8df18be5fc6b0aa314ce6ebf51b53313d4306cca8e46"
+dependencies = [
+ "hermit-abi",
+ "libc",
+ "windows-sys 0.61.2",
+]
+
 [[package]]
 name = "is_terminal_polyfill"
 version = "1.70.2"
@@ -953,9 +963,9 @@ checksum = "a6cb138bb79a146c1bd460005623e142ef0181e3d0219cb493e02f7d08a35695"
 
 [[package]]
 name = "itertools"
-version = "0.13.0"
+version = "0.10.5"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "413ee7dfc52ee1a4949ceeb7dbc8a33f2d6c088194d9f922fb8318faf1f01186"
+checksum = "b0fd2260e829bddf4cb6ea802289de2f86d6a7a690192fbe91b3f46e0f2c8473"
 dependencies = [
  "either",
 ]
@@ -1133,7 +1143,7 @@ dependencies = [
 
 [[package]]
 name = "numa"
-version = "0.10.3"
+version = "0.7.1"
 dependencies = [
  "arc-swap",
  "axum",
@@ -1149,10 +1159,9 @@ dependencies = [
  "reqwest",
  "ring",
  "rustls",
- "rustls-pemfile",
  "serde",
  "serde_json",
- "socket2",
+ "socket2 0.5.10",
  "time",
  "tokio",
  "tokio-rustls",
@@ -1162,9 +1171,9 @@ dependencies = [
 
 [[package]]
 name = "oid-registry"
-version = "0.8.1"
+version = "0.7.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "12f40cff3dde1b6087cc5d5f5d4d65712f34016a03ed60e9c08dcc392736b5b7"
+checksum = "a8d8034d9489cdaf79228eb9f6a3b8d7bb32ba00d6645ebd48eef4077ceb5bd9"
 dependencies = [
  "asn1-rs",
 ]
@@ -1187,16 +1196,6 @@ version = "11.1.5"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "d6790f58c7ff633d8771f42965289203411a5e5c68388703c06e14f24770b41e"
 
-[[package]]
-name = "page_size"
-version = "0.6.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "30d5b2194ed13191c1999ae0704b7839fb18384fa22e49b57eeaa97d79ce40da"
-dependencies = [
- "libc",
- "winapi",
-]
-
 [[package]]
 name = "pem"
 version = "3.0.6"
@@ -1219,6 +1218,12 @@ version = "0.2.17"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "a89322df9ebe1c1578d689c92318e070967d1042b512afbe49518723f4e6d5cd"
 
+[[package]]
+name = "pin-utils"
+version = "0.1.0"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "8b870d8c151b6f2fb93e84a13146138f05d02ed11c7e7c54f8826aaaf7c9f184"
+
 [[package]]
 name = "plotters"
 version = "0.3.7"
@@ -1308,8 +1313,8 @@ dependencies = [
  "quinn-udp",
  "rustc-hash",
  "rustls",
- "socket2",
- "thiserror",
+ "socket2 0.6.3",
+ "thiserror 2.0.18",
  "tokio",
  "tracing",
  "web-time",
@@ -1330,7 +1335,7 @@ dependencies = [
  "rustls",
  "rustls-pki-types",
  "slab",
- "thiserror",
+ "thiserror 2.0.18",
  "tinyvec",
  "tracing",
  "web-time",
@@ -1345,7 +1350,7 @@ dependencies = [
  "cfg_aliases",
  "libc",
  "once_cell",
- "socket2",
+ "socket2 0.6.3",
  "tracing",
  "windows-sys 0.60.2",
 ]
@@ -1416,9 +1421,9 @@ dependencies = [
 
 [[package]]
 name = "rcgen"
-version = "0.14.7"
+version = "0.13.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "10b99e0098aa4082912d4c649628623db6aba77335e4f4569ff5083a6448b32e"
+checksum = "75e669e5202259b5314d1ea5397316ad400819437857b90861765f24c4cf80a2"
 dependencies = [
  "pem",
  "ring",
@@ -1541,15 +1546,6 @@ dependencies = [
  "zeroize",
 ]
 
-[[package]]
-name = "rustls-pemfile"
-version = "2.2.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "dce314e5fee3f39953d46bb63bb8a46d40c2f8fb7cc5a3b6cab2bde9721d6e50"
-dependencies = [
- "rustls-pki-types",
-]
-
 [[package]]
 name = "rustls-pki-types"
 version = "1.14.0"
@@ -1649,11 +1645,11 @@ dependencies = [
 
 [[package]]
 name = "serde_spanned"
-version = "1.1.1"
+version = "0.6.9"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "6662b5879511e06e8999a8a235d848113e942c9124f211511b16466ee2995f26"
+checksum = "bf41e0cfaf7226dca15e8197172c295a782857fcb97fad1808a166870dee75a3"
 dependencies = [
- "serde_core",
+ "serde",
 ]
 
 [[package]]
@@ -1692,6 +1688,16 @@ version = "1.15.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "67b1b7a3b5fe4f1376887184045fcf45c69e92af734b7aaddc05fb777b6fbd03"
 
+[[package]]
+name = "socket2"
+version = "0.5.10"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "e22376abed350d73dd1cd119b57ffccad95b4e585a7cda43e286245ce23c0678"
+dependencies = [
+ "libc",
+ "windows-sys 0.52.0",
+]
+
 [[package]]
 name = "socket2"
 version = "0.6.3"
@@ -1745,13 +1751,33 @@ dependencies = [
  "syn",
 ]
 
+[[package]]
+name = "thiserror"
+version = "1.0.69"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "b6aaf5339b578ea85b50e080feb250a3e8ae8cfcdff9a461c9ec2904bc923f52"
+dependencies = [
+ "thiserror-impl 1.0.69",
+]
+
 [[package]]
 name = "thiserror"
 version = "2.0.18"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "4288b5bcbc7920c07a1149a35cf9590a2aa808e0bc1eafaade0b80947865fbc4"
 dependencies = [
- "thiserror-impl",
+ "thiserror-impl 2.0.18",
+]
+
+[[package]]
+name = "thiserror-impl"
+version = "1.0.69"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "4fee6c4efc90059e10f81e6d42c60a18f76588c3d74cb83a0b242a2b6c7504c1"
+dependencies = [
+ "proc-macro2",
+ "quote",
+ "syn",
 ]
 
 [[package]]
@@ -1833,24 +1859,24 @@ checksum = "1f3ccbac311fea05f86f61904b462b55fb3df8837a366dfc601a0161d0532f20"
 
 [[package]]
 name = "tokio"
-version = "1.51.1"
+version = "1.50.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "f66bf9585cda4b724d3e78ab34b73fb2bbaba9011b9bfdf69dc836382ea13b8c"
+checksum = "27ad5e34374e03cfffefc301becb44e9dc3c17584f414349ebe29ed26661822d"
 dependencies = [
  "bytes",
  "libc",
  "mio",
  "pin-project-lite",
- "socket2",
+ "socket2 0.6.3",
  "tokio-macros",
  "windows-sys 0.61.2",
 ]
 
 [[package]]
 name = "tokio-macros"
-version = "2.7.0"
+version = "2.6.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "385a6cb71ab9ab790c5fe8d67f1645e6c450a7ce006a33de03daa956cf70a496"
+checksum = "5c55a2eff8b69ce66c84f85e1da1c233edc36ceb85a2058d11b0d6a3c7e7569c"
 dependencies = [
  "proc-macro2",
  "quote",
@@ -1882,42 +1908,44 @@ dependencies = [
 
 [[package]]
 name = "toml"
-version = "1.1.2+spec-1.1.0"
+version = "0.8.23"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "81f3d15e84cbcd896376e6730314d59fb5a87f31e4b038454184435cd57defee"
+checksum = "dc1beb996b9d83529a9e75c17a1686767d148d70663143c7854d8b4a09ced362"
 dependencies = [
- "indexmap",
- "serde_core",
+ "serde",
  "serde_spanned",
  "toml_datetime",
- "toml_parser",
- "toml_writer",
- "winnow",
+ "toml_edit",
 ]
 
 [[package]]
 name = "toml_datetime"
-version = "1.1.1+spec-1.1.0"
+version = "0.6.11"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "3165f65f62e28e0115a00b2ebdd37eb6f3b641855f9d636d3cd4103767159ad7"
+checksum = "22cddaf88f4fbc13c51aebbf5f8eceb5c7c5a9da2ac40a13519eb5b0a0e8f11c"
 dependencies = [
- "serde_core",
+ "serde",
 ]
 
 [[package]]
-name = "toml_parser"
-version = "1.1.2+spec-1.1.0"
+name = "toml_edit"
+version = "0.22.27"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "a2abe9b86193656635d2411dc43050282ca48aa31c2451210f4202550afb7526"
+checksum = "41fe8c660ae4257887cf66394862d21dbca4a6ddd26f04a3560410406a2f819a"
 dependencies = [
+ "indexmap",
+ "serde",
+ "serde_spanned",
+ "toml_datetime",
+ "toml_write",
  "winnow",
 ]
 
 [[package]]
-name = "toml_writer"
-version = "1.1.1+spec-1.1.0"
+name = "toml_write"
+version = "0.1.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "756daf9b1013ebe47a8776667b466417e2d4c5679d441c26230efd9ef78692db"
+checksum = "5d99f8c9a7727884afe522e9bd5edbfc91a3312b36a77b5fb8926e4c31a41801"
 
 [[package]]
 name = "tower"
@@ -2150,22 +2178,6 @@ dependencies = [
  "rustls-pki-types",
 ]
 
-[[package]]
-name = "winapi"
-version = "0.3.9"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "5c839a674fcd7a98952e593242ea400abe93992746761e38641405d28b00f419"
-dependencies = [
- "winapi-i686-pc-windows-gnu",
- "winapi-x86_64-pc-windows-gnu",
-]
-
-[[package]]
-name = "winapi-i686-pc-windows-gnu"
-version = "0.4.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "ac3b87c63620426dd9b991e5ce0329eff545bccbbb34f3be09ff6fb6ab51b7b6"
-
 [[package]]
 name = "winapi-util"
 version = "0.1.11"
@@ -2175,12 +2187,6 @@ dependencies = [
  "windows-sys 0.61.2",
 ]
 
-[[package]]
-name = "winapi-x86_64-pc-windows-gnu"
-version = "0.4.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "712e227841d057c1ee1cd2fb22fa7e5a5461ae8e48fa2ca79ec42cfc1931183f"
-
 [[package]]
 name = "windows-link"
 version = "0.2.1"
@@ -2345,9 +2351,12 @@ checksum = "d6bbff5f0aada427a1e5a6da5f1f98158182f26556f345ac9e04d36d0ebed650"
 
 [[package]]
 name = "winnow"
-version = "1.0.1"
+version = "0.7.15"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "09dac053f1cd375980747450bfc7250c264eaae0583872e845c0c7cd578872b5"
+checksum = "df79d97927682d2fd8adb29682d1140b343be4ac0f08fd68b7765d9c059d3945"
+dependencies = [
+ "memchr",
+]
 
 [[package]]
 name = "wit-bindgen"
@@ -2363,9 +2372,9 @@ checksum = "9edde0db4769d2dc68579893f2306b26c6ecfbe0ef499b013d731b7b9247e0b9"
 
 [[package]]
 name = "x509-parser"
-version = "0.18.1"
+version = "0.16.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "d43b0f71ce057da06bc0851b23ee24f3f86190b07203dd8f567d0b706a185202"
+checksum = "fcbc162f30700d6f3f82a24bf7cc62ffe7caea42c0b2cba8bf7f3ae50cf51f69"
 dependencies = [
  "asn1-rs",
  "data-encoding",
@@ -2375,7 +2384,7 @@ dependencies = [
  "oid-registry",
  "ring",
  "rusticata-macros",
- "thiserror",
+ "thiserror 1.0.69",
  "time",
 ]
 

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "numa"
-version = "0.10.3"
+version = "0.7.1"
 authors = ["razvandimescu <razvan@dimescu.com>"]
 edition = "2021"
 description = "Portable DNS resolver in Rust — .numa local domains, ad blocking, developer overrides, DNS-over-HTTPS"
@@ -14,7 +14,7 @@ tokio = { version = "1", features = ["rt-multi-thread", "macros", "net", "time",
 axum = "0.8"
 serde = { version = "1", features = ["derive"] }
 serde_json = "1"
-toml = "1.1"
+toml = "0.8"
 log = "0.4"
 env_logger = "0.11"
 reqwest = { version = "0.12", features = ["rustls-tls", "gzip", "http2"], default-features = false }
@@ -22,17 +22,16 @@ hyper = { version = "1", features = ["client", "http1", "server"] }
 hyper-util = { version = "0.1", features = ["client-legacy", "http1", "tokio"] }
 http-body-util = "0.1"
 futures = "0.3"
-socket2 = { version = "0.6", features = ["all"] }
-rcgen = { version = "0.14", features = ["pem", "x509-parser"] }
+socket2 = { version = "0.5", features = ["all"] }
+rcgen = { version = "0.13", features = ["pem", "x509-parser"] }
 time = "0.3"
 rustls = "0.23"
 tokio-rustls = "0.26"
 arc-swap = "1"
 ring = "0.17"
-rustls-pemfile = "2.2.0"
 
 [dev-dependencies]
-criterion = { version = "0.8", features = ["html_reports"] }
+criterion = { version = "0.5", features = ["html_reports"] }
 tower = { version = "0.5", features = ["util"] }
 http = "1"
 

Dockerfile

@@ -1,4 +1,4 @@
-FROM rust:1.94-alpine AS builder
+FROM rust:1.88-alpine AS builder
 RUN apk add --no-cache musl-dev cmake make perl
 WORKDIR /app
 COPY Cargo.toml Cargo.lock ./
@@ -11,7 +11,7 @@ COPY numa.toml com.numa.dns.plist numa.service ./
 RUN touch src/main.rs src/lib.rs
 RUN cargo build --release
 
-FROM alpine:3.23
+FROM alpine:3.20
 COPY --from=builder /app/target/release/numa /usr/local/bin/numa
-EXPOSE 53/udp 80/tcp 443/tcp 853/tcp 5380/tcp
+EXPOSE 53/udp 80/tcp 443/tcp 5380/tcp
 ENTRYPOINT ["numa"]

PKGBUILD

@@ -1,62 +0,0 @@
-# Maintainer: razvandimescu <razvan@dimescu.com>
-pkgname=numa-git
-_pkgname=numa
-pkgver=0.10.1.r0.g0000000 # Placeholder — pkgver() rewrites this on each makepkg run
-pkgrel=1
-pkgdesc="Portable DNS resolver in Rust — .numa local domains, ad blocking, developer overrides, DNS-over-HTTPS"
-arch=('x86_64')
-url="https://github.com/razvandimescu/numa"
-license=('MIT')
-options=('!lto')
-depends=('gcc-libs' 'glibc')
-makedepends=('cargo' 'git')
-provides=("$_pkgname")
-conflicts=("$_pkgname")
-backup=('etc/numa.toml')
-source=("$_pkgname::git+$url.git")
-sha256sums=('SKIP')
-
-pkgver() {
-  cd "$srcdir/$_pkgname"
-  ( set -o pipefail
-    git describe --long --tags 2>/dev/null | sed 's/\([^-]*-g\)/r\1/;s/-/./g' ||
-    printf "r%s.%s" "$(git rev-list --count HEAD)" "$(git rev-parse --short HEAD)"
-  ) | sed 's/^v//'
-}
-
-prepare() {
-  cd "$srcdir/$_pkgname"
-  # numa v0.10.1+ uses FHS-compliant paths on Linux by default
-  # (/var/lib/numa for data, journalctl for logs), so no source
-  # patching is needed. The earlier sed targeted /usr/local/bin/numa,
-  # which only appears in a comment in current main.
-  export RUSTUP_TOOLCHAIN=stable
-  cargo fetch --locked
-}
-
-build() {
-  cd "$srcdir/$_pkgname"
-  export RUSTUP_TOOLCHAIN=stable
-  cargo build --frozen --release
-}
-
-check() {
-  cd "$srcdir/$_pkgname"
-  export RUSTUP_TOOLCHAIN=stable
-  cargo test --frozen
-}
-
-package() {
-  cd "$srcdir/$_pkgname"
-  install -Dm755 "target/release/$_pkgname" "$pkgdir/usr/bin/$_pkgname"
-
-  # numa.service uses {{exe_path}} as a placeholder substituted by
-  # `numa install` at runtime via replace_exe_path(). For an AUR
-  # package install (no `numa install` step), we substitute it
-  # statically here so systemd gets a real ExecStart path.
-  sed 's|{{exe_path}}|/usr/bin/numa /etc/numa.toml|g' numa.service > numa.service.patched
-  install -Dm644 "numa.service.patched" "$pkgdir/usr/lib/systemd/system/numa.service"
-
-  install -Dm644 "numa.toml" "$pkgdir/etc/numa.toml"
-  install -Dm644 "LICENSE" "$pkgdir/usr/share/licenses/$pkgname/LICENSE"
-}

README.md

@@ -8,130 +8,189 @@
 
 A portable DNS resolver in a single binary. Block ads on any network, name your local services (`frontend.numa`), and override any hostname with auto-revert — all from your laptop, no cloud account or Raspberry Pi required.
 
-Built from scratch in Rust. Zero DNS libraries. RFC 1035 wire protocol parsed by hand. Caching, ad blocking, and local service domains out of the box. Optional recursive resolution from root nameservers with full DNSSEC chain-of-trust validation, plus a DNS-over-TLS listener for encrypted client connections (iOS Private DNS, systemd-resolved, etc.). One ~8MB binary, everything embedded.
+Built from scratch in Rust. Zero DNS libraries. RFC 1035 wire protocol parsed by hand. Recursive resolution from root nameservers with full DNSSEC validation (chain-of-trust + NSEC/NSEC3 denial proofs). One ~8MB binary, no PHP, no web server, no database — everything is embedded.
 
 ![Numa dashboard](assets/hero-demo.gif)
 
 ## Quick Start
 
 ```bash
-# macOS
+# Install (pick one)
 brew install razvandimescu/tap/numa
-
-# Linux
+cargo install numa
 curl -fsSL https://raw.githubusercontent.com/razvandimescu/numa/main/install.sh | sh
 
-# Arch Linux (AUR)
-yay -S numa-git
+# Run (port 53 requires root)
+sudo numa
 
-# Windows — download from GitHub Releases
-# All platforms
-cargo install numa
-```
-
-```bash
-sudo numa                              # run in foreground (port 53 requires root/admin)
+# Try it
+dig @127.0.0.1 google.com           # ✓ resolves normally
+dig @127.0.0.1 ads.google.com       # ✗ blocked → 0.0.0.0
 ```
 
 Open the dashboard: **http://numa.numa** (or `http://localhost:5380`)
 
-Set as system DNS:
+### Set as system resolver
 
-| Platform | Install | Uninstall |
-|----------|---------|-----------|
-| macOS | `sudo numa install` | `sudo numa uninstall` |
-| Linux | `sudo numa install` | `sudo numa uninstall` |
-| Windows | `numa install` (admin) + reboot | `numa uninstall` (admin) + reboot |
+```bash
+# Point your system DNS to Numa (saves originals for uninstall)
+sudo numa install
 
-On macOS and Linux, numa runs as a system service (launchd/systemd). On Windows, numa auto-starts on login via registry.
+# Run as a persistent service (auto-starts on boot, restarts if killed)
+sudo numa service start
+```
 
-## Local Services
+To uninstall: `sudo numa service stop` removes the service, `sudo numa uninstall` restores your original DNS.
 
-Name your dev services instead of remembering port numbers:
+### Upgrade
+
+```bash
+# From Homebrew
+brew upgrade numa
+
+# From source
+make deploy    # builds release, copies binary, re-signs, restarts service
+```
+
+### Build from source
+
+```bash
+git clone https://github.com/razvandimescu/numa.git && cd numa
+cargo build --release
+sudo cp target/release/numa /usr/local/bin/numa
+```
+
+## Why Numa
+
+- **Local service proxy** — `https://frontend.numa` instead of `localhost:5173`. Auto-generated TLS certs, WebSocket support for HMR. Like `/etc/hosts` but with auto TLS, a REST API, LAN discovery, and auto-revert.
+- **Path-based routing** — `app.numa/api → :5001`, `app.numa/auth → :5002`. Route URL paths to different backends with optional prefix stripping. Like nginx location blocks, zero config files.
+- **LAN service discovery** — Numa instances on the same network find each other automatically via mDNS. Access a teammate's `api.numa` from your machine. Opt-in via `[lan] enabled = true`.
+- **Developer overrides** — point any hostname to any IP, auto-reverts after N minutes. Full REST API for scripting. Built-in diagnostics: `curl localhost:5380/diagnose/example.com` tells you exactly how any domain resolves.
+- **DNS-over-HTTPS** — upstream queries encrypted via DoH. Your ISP sees HTTPS traffic, not DNS queries. Set `address = "https://9.9.9.9/dns-query"` in `[upstream]` or any DoH provider.
+- **Ad blocking that travels with you** — 385K+ domains blocked via [Hagezi Pro](https://github.com/hagezi/dns-blocklists). Works on any network: coffee shops, hotels, airports.
+- **Sub-microsecond caching** — 691ns cached round-trip, ~2.0M queries/sec throughput, zero heap allocations in the I/O path. [Benchmarks](bench/).
+- **Live dashboard** — real-time stats, query log, blocking controls, service management. LAN accessibility badges show which services are reachable from other devices.
+- **macOS, Linux, and Windows** — `numa install` configures system DNS, `numa service start` runs as launchd/systemd service.
+
+## Local Service Proxy
+
+Name your local dev services with `.numa` domains:
 
 ```bash
 curl -X POST localhost:5380/services \
+  -H 'Content-Type: application/json' \
   -d '{"name":"frontend","target_port":5173}'
+
+open http://frontend.numa            # → proxied to localhost:5173
 ```
 
-Now `https://frontend.numa` works in your browser — green lock, valid cert, WebSocket passthrough for HMR. No mkcert, no nginx, no `/etc/hosts`.
+- **HTTPS with green lock** — auto-generated local CA + per-service TLS certs
+- **WebSocket** — Vite/webpack HMR works through the proxy
+- **Health checks** — dashboard shows green/red status per service
+- **LAN sharing** — services bound to `0.0.0.0` are automatically discoverable by other Numa instances on the network. Dashboard shows "LAN" or "local only" per service.
+- **Path-based routing** — route URL paths to different backends:
+  ```toml
+  [[services]]
+  name = "app"
+  target_port = 3000
+  routes = [
+      { path = "/api", port = 5001 },
+      { path = "/auth", port = 5002, strip = true },
+  ]
+  ```
+  `app.numa/api/users → :5001/api/users`, `app.numa/auth/login → :5002/login` (stripped)
+- **Persistent** — services survive restarts
+- Or configure in `numa.toml`:
 
-Add path-based routing (`app.numa/api → :5001`), share services across machines via LAN discovery, or configure everything in [`numa.toml`](numa.toml).
+```toml
+[[services]]
+name = "frontend"
+target_port = 5173
+```
 
-## Ad Blocking & Privacy
+## LAN Service Discovery
 
-385K+ domains blocked via [Hagezi Pro](https://github.com/hagezi/dns-blocklists). Works on any network — coffee shops, hotels, airports. Travels with your laptop.
-
-Three resolution modes:
-
-- **`forward`** (default) — transparent proxy to your existing system DNS. Everything works as before, just with caching and ad blocking on top. Captive portals, VPNs, corporate DNS — all respected.
-- **`recursive`** — resolve directly from root nameservers. No upstream dependency, no single entity sees your full query pattern. Add `[dnssec] enabled = true` for full chain-of-trust validation.
-- **`auto`** — probe root servers on startup, recursive if reachable, encrypted DoH fallback if blocked.
-
-DNSSEC validates the full chain of trust: RRSIG signatures, DNSKEY verification, DS delegation, NSEC/NSEC3 denial proofs. [Read how it works →](https://numa.rs/blog/posts/dnssec-from-scratch.html)
-
-**DNS-over-TLS listener** (RFC 7858) — accept encrypted queries on port 853 from strict clients like iOS Private DNS, systemd-resolved, or stubby. Two modes:
-
-- **Self-signed** (default) — numa generates a local CA automatically. `numa install` adds it to the system trust store on macOS, Linux (Debian/Ubuntu, Fedora/RHEL/SUSE, Arch), and Windows. On iOS, install the `.mobileconfig` from `numa setup-phone`. Firefox keeps its own NSS store and ignores the system one — trust the CA there manually if you need HTTPS for `.numa` services in Firefox.
-- **Bring-your-own cert** — point `[dot] cert_path` / `key_path` at a publicly-trusted cert (e.g., Let's Encrypt via DNS-01 challenge on a domain pointing at your numa instance). Clients connect without any trust-store setup — same UX as AdGuard Home or Cloudflare `1.1.1.1`.
-
-ALPN `"dot"` is advertised and enforced in both modes; a handshake with mismatched ALPN is rejected as a cross-protocol confusion defense.
-
-## LAN Discovery
-
-Run Numa on multiple machines. They find each other automatically via mDNS:
+Run Numa on multiple machines. They find each other automatically:
 
 ```
 Machine A (192.168.1.5)              Machine B (192.168.1.20)
 ┌──────────────────────┐             ┌──────────────────────┐
 │ Numa                 │    mDNS     │ Numa                 │
-│  - api (port 8000)   │◄───────────►│  - grafana (3000)    │
-│  - frontend (5173)   │  discovery  │                      │
+│  services:           │◄───────────►│  services:           │
+│   - api (port 8000)  │  discovery  │   - grafana (3000)   │
+│   - frontend (5173)  │             │                      │
 └──────────────────────┘             └──────────────────────┘
 ```
 
-From Machine B: `curl http://api.numa` → proxied to Machine A's port 8000. Enable with `numa lan on`.
+From Machine B:
+```bash
+dig @127.0.0.1 api.numa          # → 192.168.1.5
+curl http://api.numa              # → proxied to Machine A's port 8000
+```
 
-**Hub mode**: run one instance with `bind_addr = "0.0.0.0:53"` and point other devices' DNS to it — they get ad blocking + `.numa` resolution without installing anything.
+Enable LAN discovery:
+```bash
+numa lan on
+```
+Or in `numa.toml`:
+```toml
+[lan]
+enabled = true
+```
+Uses standard mDNS (`_numa._tcp.local` on port 5353) — compatible with Bonjour/Avahi, silently dropped by corporate firewalls instead of triggering IPS alerts.
+
+**Hub mode** — don't want to install Numa on every machine? Run one instance as a shared DNS server and point other devices to it:
+
+```bash
+# On the hub machine, bind to LAN interface
+[server]
+bind_addr = "0.0.0.0:53"
+
+# On other devices, set DNS to the hub's IP
+# They get .numa resolution, ad blocking, caching — zero install
+```
 
 ## How It Compares
 
-| | Pi-hole | AdGuard Home | Unbound | Numa |
-|---|---|---|---|---|
-| Local service proxy + auto TLS | — | — | — | `.numa` domains, HTTPS, WebSocket |
-| LAN service discovery | — | — | — | mDNS, zero config |
-| Developer overrides (REST API) | — | — | — | Auto-revert, scriptable |
-| Recursive resolver | — | — | Yes | Yes, with SRTT selection |
-| DNSSEC validation | — | — | Yes | Yes (RSA, ECDSA, Ed25519) |
-| Ad blocking | Yes | Yes | — | 385K+ domains |
-| Web admin UI | Full | Full | — | Dashboard |
-| Encrypted upstream (DoH) | Needs cloudflared | Yes | — | Native |
-| Encrypted clients (DoT listener) | Needs stunnel sidecar | Yes | Yes | Native (RFC 7858) |
-| Portable (laptop) | No (appliance) | No (appliance) | Server | Single binary, macOS/Linux/Windows |
-| Community maturity | 56K stars, 10 years | 33K stars | 20 years | New |
+| | Pi-hole | AdGuard Home | NextDNS | Cloudflare | Numa |
+|---|---|---|---|---|---|
+| Local service proxy | No | No | No | No | `.numa` + HTTPS + WS |
+| Path-based routing | No | No | No | No | Prefix match + strip |
+| LAN service discovery | No | No | No | No | mDNS, opt-in |
+| Developer overrides | No | No | No | No | REST API + auto-expiry |
+| Recursive resolver | No | No | Cloud only | Cloud only | From root hints, DNSSEC |
+| Encrypted upstream (DoH) | No (needs cloudflared) | Yes | Cloud only | Cloud only | Native, single binary |
+| Portable (travels with laptop) | No (appliance) | No (appliance) | Cloud only | Cloud only | Single binary |
+| Zero config | Complex | Docker/setup | Yes | Yes | Works out of the box |
+| Ad blocking | Yes | Yes | Yes | Limited | 385K+ domains |
+| Data stays local | Yes | Yes | Cloud | Cloud | 100% local |
 
-## Performance
+## How It Works
 
-691ns cached round-trip. ~2.0M qps throughput. Zero heap allocations in the hot path. Recursive queries average 237ms after SRTT warmup (12x improvement over round-robin). ECDSA P-256 DNSSEC verification: 174ns. [Benchmarks →](bench/)
+```
+Query → Overrides → .numa TLD → Blocklist → Local Zones → Cache → Recursive/Forward
+```
 
-## Learn More
+Two resolution modes: **forward** (relay to upstream like Quad9/Cloudflare) or **recursive** (resolve from root nameservers — no upstream dependency). Set `mode = "recursive"` in `[upstream]` to resolve independently.
 
-- [Blog: Implementing DNSSEC from Scratch in Rust](https://numa.rs/blog/posts/dnssec-from-scratch.html)
-- [Blog: I Built a DNS Resolver from Scratch](https://numa.rs/blog/posts/dns-from-scratch.html)
-- [Configuration reference](numa.toml) — all options documented inline
-- [REST API](src/api.rs) — 27 endpoints across overrides, cache, blocking, services, diagnostics
+No DNS libraries — no `hickory-dns`, no `trust-dns`. The wire protocol — headers, labels, compression pointers, record types — is parsed and serialized by hand. Runs on `tokio` + `axum`, async per-query task spawning.
+
+[Configuration reference](numa.toml)
 
 ## Roadmap
 
-- [x] DNS forwarding, caching, ad blocking, developer overrides
-- [x] `.numa` local domains — auto TLS, path routing, WebSocket proxy
-- [x] LAN service discovery — mDNS, cross-machine DNS + proxy
-- [x] DNS-over-HTTPS — encrypted upstream
-- [x] DNS-over-TLS listener — encrypted client connections (RFC 7858, ALPN strict)
-- [x] Recursive resolution + DNSSEC — chain-of-trust, NSEC/NSEC3
-- [x] SRTT-based nameserver selection
-- [ ] pkarr integration — self-sovereign DNS via Mainline DHT
-- [ ] Global `.numa` names — DHT-backed, no registrar
+- [x] DNS proxy core — forwarding, caching, local zones
+- [x] Developer overrides — REST API with auto-expiry
+- [x] Ad blocking — 385K+ domains, live dashboard, allowlist
+- [x] System integration — macOS + Linux, launchd/systemd, Tailscale/VPN auto-discovery
+- [x] Local service proxy — `.numa` domains, HTTP/HTTPS proxy, auto TLS, WebSocket
+- [x] Path-based routing — URL prefix routing with optional strip, REST API
+- [x] LAN service discovery — mDNS auto-discovery (opt-in), cross-machine DNS + proxy
+- [x] DNS-over-HTTPS — encrypted upstream via DoH (Quad9, Cloudflare, any provider)
+- [x] Recursive resolution — resolve from root nameservers, no upstream dependency
+- [x] DNSSEC validation — chain-of-trust, NSEC/NSEC3 denial proofs, AD bit (RSA, ECDSA, Ed25519)
+- [ ] pkarr integration — self-sovereign DNS via Mainline DHT (15M nodes)
+- [ ] Global `.numa` names — self-publish, DHT-backed, first-come-first-served
 
 ## License
 

com.numa.dns.plist

@@ -6,7 +6,7 @@
     <string>com.numa.dns</string>
     <key>ProgramArguments</key>
     <array>
-        <string>{{exe_path}}</string>
+        <string>/usr/local/bin/numa</string>
     </array>
     <key>RunAtLoad</key>
     <true/>

install.sh

@@ -70,10 +70,8 @@ echo ""
 echo "  \033[38;2;107;124;78mInstalled:\033[0m $INSTALL_DIR/numa ($TAG)"
 echo ""
 echo "  Get started:"
-echo "    sudo numa install            # install service + set as system DNS"
-echo "    open http://localhost:5380   # dashboard"
-echo ""
-echo "  Other commands:"
-echo "    sudo numa                    # run in foreground (no service)"
-echo "    sudo numa uninstall          # restore original DNS"
+echo "    sudo numa                    # start the DNS server"
+echo "    sudo numa install            # set as system DNS"
+echo "    sudo numa service start      # run as persistent service"
+echo "    open http://localhost:5380    # dashboard"
 echo ""

numa.service

@@ -5,7 +5,7 @@ Wants=network-online.target
 
 [Service]
 Type=simple
-ExecStart={{exe_path}}
+ExecStart=/usr/local/bin/numa
 Restart=always
 RestartSec=2
 StandardOutput=journal

numa.toml

@@ -2,12 +2,6 @@
 bind_addr = "0.0.0.0:53"
 api_port = 5380
 # api_bind_addr = "127.0.0.1"  # default; set to "0.0.0.0" for LAN dashboard access
-# data_dir = "/var/lib/numa"  # where numa stores TLS CA and cert material
-                              # Defaults: /var/lib/numa on linux (FHS),
-                              # /usr/local/var/numa on macos (homebrew prefix),
-                              # %PROGRAMDATA%\numa on windows. Override for
-                              # containerized deploys or tests that can't
-                              # write to the system path.
 
 # [upstream]
 # mode = "forward"                              # "forward" (default) — relay to upstream
@@ -60,7 +54,7 @@ enabled = true
 port = 80
 tls_port = 443
 tld = "numa"
-# bind_addr = "127.0.0.1"  # default; set to "0.0.0.0" for LAN access to .numa services
+# bind_addr = "127.0.0.1"  # default; auto 0.0.0.0 when [lan] enabled
 
 # Pre-configured services (numa.numa is always added automatically)
 # [[services]]
@@ -89,14 +83,6 @@ tld = "numa"
 # enabled = false             # opt-in: verify chain of trust from root KSK
 # strict = false              # true = SERVFAIL on bogus signatures
 
-# DNS-over-TLS listener (RFC 7858) — encrypted DNS on port 853
-# [dot]
-# enabled = false             # opt-in: accept DoT queries
-# port = 853                  # standard DoT port
-# bind_addr = "0.0.0.0"       # IPv4 or IPv6; unspecified binds all interfaces
-# cert_path = "/etc/numa/dot.crt"  # PEM cert; omit to use self-signed (proxy CA if available)
-# key_path = "/etc/numa/dot.key"   # PEM private key; must be set together with cert_path
-
 # LAN service discovery via mDNS (disabled by default — no network traffic unless enabled)
 # [lan]
 # enabled = true              # discover other Numa instances via mDNS (_numa._tcp.local)

scripts/update-homebrew-formula.py

@@ -1,57 +0,0 @@
-#!/usr/bin/env python3
-"""Rewrite a Homebrew formula in place: bump version, URL paths, and sha256 lines.
-
-Reads the formula path from argv[1], and the following env vars:
-  VERSION              e.g. "0.10.0" (no leading v)
-  SHA_MACOS_AARCH64
-  SHA_MACOS_X86_64
-  SHA_LINUX_AARCH64
-  SHA_LINUX_X86_64
-
-Assumptions about the formula:
-  - Has `version "X.Y.Z"` somewhere
-  - Has `url "...releases/download/vX.Y.Z/numa-<target>.tar.gz"` lines
-  - May or may not already have `sha256 "..."` lines immediately after each url
-"""
-import os
-import re
-import sys
-
-formula_path = sys.argv[1]
-version = os.environ["VERSION"].lstrip("v")
-shas = {
-    "macos-aarch64": os.environ["SHA_MACOS_AARCH64"],
-    "macos-x86_64": os.environ["SHA_MACOS_X86_64"],
-    "linux-aarch64": os.environ["SHA_LINUX_AARCH64"],
-    "linux-x86_64": os.environ["SHA_LINUX_X86_64"],
-}
-
-with open(formula_path) as f:
-    content = f.read()
-
-content = re.sub(r'version "[^"]*"', f'version "{version}"', content)
-content = re.sub(
-    r"releases/download/v[\d.]+/numa-",
-    f"releases/download/v{version}/numa-",
-    content,
-)
-content = re.sub(r'\n[ \t]*sha256 "[^"]*"', "", content)
-
-
-def add_sha(match: re.Match) -> str:
-    indent = match.group(1)
-    target = match.group(2)
-    if target not in shas:
-        return match.group(0)
-    return f'{match.group(0)}\n{indent}sha256 "{shas[target]}"'
-
-
-content = re.sub(
-    r'^([ \t]+)url "[^"]*numa-([\w-]+)\.tar\.gz"',
-    add_sha,
-    content,
-    flags=re.MULTILINE,
-)
-
-with open(formula_path, "w") as f:
-    f.write(content)

site/dashboard.html

@@ -101,7 +101,7 @@ body {
 /* Stat cards row */
 .stats-row {
   display: grid;
-  grid-template-columns: repeat(6, 1fr);
+  grid-template-columns: repeat(5, 1fr);
   gap: 1rem;
 }
 .stat-card {
@@ -125,8 +125,6 @@ body {
 .stat-card.blocked::before { background: var(--rose); }
 .stat-card.overrides::before { background: var(--violet); }
 .stat-card.uptime::before { background: var(--cyan); }
-.stat-card.memory::before { background: var(--text-dim); }
-.stat-card.memory .stat-value { color: var(--text-secondary); }
 
 .stat-label {
   font-size: 0.7rem;
@@ -470,74 +468,10 @@ body {
   display: none;
 }
 
-/* Memory sidebar panel */
-.memory-bar {
-  display: flex;
-  height: 18px;
-  border-radius: 4px;
-  overflow: hidden;
-  background: var(--bg-surface);
-  margin-bottom: 0.8rem;
-}
-.memory-bar-seg {
-  height: 100%;
-  min-width: 2px;
-  transition: width 0.6s ease;
-}
-.memory-bar-seg.cache { background: var(--teal); }
-.memory-bar-seg.blocklist { background: var(--rose); }
-.memory-bar-seg.querylog { background: var(--amber); }
-.memory-bar-seg.srtt { background: var(--cyan); }
-.memory-bar-seg.overrides { background: var(--violet); }
-.memory-row {
-  display: flex;
-  align-items: center;
-  padding: 0.3rem 0;
-  border-bottom: 1px solid var(--border);
-  font-family: var(--font-mono);
-  font-size: 0.72rem;
-}
-.memory-row:last-child { border-bottom: none; }
-.memory-row-dot {
-  width: 8px;
-  height: 8px;
-  border-radius: 2px;
-  flex-shrink: 0;
-  margin-right: 0.5rem;
-}
-.memory-row-label {
-  flex: 1;
-  color: var(--text-secondary);
-}
-.memory-row-size {
-  width: 65px;
-  text-align: right;
-  color: var(--text-primary);
-  font-weight: 500;
-}
-.memory-row-entries {
-  width: 90px;
-  text-align: right;
-  color: var(--text-dim);
-}
-.memory-rss {
-  margin-top: 0.5rem;
-  padding-top: 0.5rem;
-  border-top: 1px solid var(--border);
-  display: flex;
-  justify-content: space-between;
-  font-family: var(--font-mono);
-  font-size: 0.72rem;
-  color: var(--text-dim);
-}
-
 /* Responsive */
 @media (max-width: 1100px) {
   .main-grid { grid-template-columns: 1fr; }
 }
-@media (max-width: 900px) {
-  .stats-row { grid-template-columns: repeat(3, 1fr); }
-}
 @media (max-width: 700px) {
   .stats-row { grid-template-columns: repeat(2, 1fr); }
   .dashboard { padding: 1rem; }
@@ -590,11 +524,6 @@ body {
       <div class="stat-value" id="uptime">—</div>
       <div class="stat-sub" id="uptimeSub">&nbsp;</div>
     </div>
-    <div class="stat-card memory">
-      <div class="stat-label">Memory</div>
-      <div class="stat-value" id="memoryRss">—</div>
-      <div class="stat-sub" id="memorySub">&nbsp;</div>
-    </div>
   </div>
 
   <!-- Resolution paths -->
@@ -719,17 +648,6 @@ body {
         </div>
       </div>
 
-      <!-- Memory breakdown -->
-      <div class="panel" id="memoryPanel">
-        <div class="panel-header">
-          <span class="panel-title">Memory</span>
-          <span class="panel-title" id="memoryTotal" style="color: var(--text-dim)"></span>
-        </div>
-        <div class="panel-body" id="memoryBody">
-          <div class="empty-state">No memory data</div>
-        </div>
-      </div>
-
       <!-- Cache entries -->
       <div class="panel">
         <div class="panel-header">
@@ -794,69 +712,6 @@ function formatRemaining(secs) {
   return `${Math.floor(secs / 3600)}h ${Math.floor((secs % 3600) / 60)}m left`;
 }
 
-function formatBytes(bytes) {
-  if (bytes === 0) return '0 B';
-  if (bytes < 1024) return bytes + ' B';
-  if (bytes < 1048576) return (bytes / 1024).toFixed(1) + ' KB';
-  if (bytes < 1073741824) return (bytes / 1048576).toFixed(1) + ' MB';
-  return (bytes / 1073741824).toFixed(1) + ' GB';
-}
-
-const MEMORY_COMPONENTS = [
-  { key: 'cache',     label: 'Cache',     cls: 'cache',     color: 'var(--teal)' },
-  { key: 'blocklist', label: 'Blocklist',  cls: 'blocklist', color: 'var(--rose)' },
-  { key: 'query_log', label: 'Query Log',  cls: 'querylog',  color: 'var(--amber)' },
-  { key: 'srtt',      label: 'SRTT',       cls: 'srtt',      color: 'var(--cyan)' },
-  { key: 'overrides', label: 'Overrides',  cls: 'overrides', color: 'var(--violet)' },
-];
-
-function renderMemory(mem, stats) {
-  if (!mem) return;
-
-  // Stat card
-  document.getElementById('memoryRss').textContent = formatBytes(mem.process_memory_bytes);
-  document.getElementById('memorySub').textContent = 'est. ' + formatBytes(mem.total_estimated_bytes);
-
-  const entryCounts = {
-    cache: stats.cache.entries,
-    blocklist: stats.blocking.domains_loaded,
-    query_log: mem.query_log_entries,
-    srtt: mem.srtt_entries,
-    overrides: stats.overrides.active,
-  };
-
-  // Sidebar panel
-  const total = mem.total_estimated_bytes || 1;
-  document.getElementById('memoryTotal').textContent = formatBytes(total);
-
-  const barSegments = MEMORY_COMPONENTS.map(c => {
-    const bytes = mem[c.key + '_bytes'] || 0;
-    const pct = ((bytes / total) * 100).toFixed(1);
-    return `<div class="memory-bar-seg ${c.cls}" style="width:${pct}%" title="${c.label}: ${formatBytes(bytes)} (${pct}%)"></div>`;
-  }).join('');
-
-  const rows = MEMORY_COMPONENTS.map(c => {
-    const bytes = mem[c.key + '_bytes'] || 0;
-    const entries = entryCounts[c.key] || 0;
-    return `
-      <div class="memory-row">
-        <div class="memory-row-dot" style="background:${c.color}"></div>
-        <span class="memory-row-label">${c.label}</span>
-        <span class="memory-row-size">${formatBytes(bytes)}</span>
-        <span class="memory-row-entries">${formatNumber(entries)} entries</span>
-      </div>`;
-  }).join('');
-
-  document.getElementById('memoryBody').innerHTML = `
-    <div class="memory-bar">${barSegments}</div>
-    ${rows}
-    <div class="memory-rss">
-      <span>Process Footprint</span>
-      <span>${formatBytes(mem.process_memory_bytes)}</span>
-    </div>
-  `;
-}
-
 const PATH_DEFS = [
   { key: 'forwarded', label: 'Forward', cls: 'forward' },
   { key: 'recursive', label: 'Recursive', cls: 'recursive' },
@@ -1027,9 +882,6 @@ async function refresh() {
     document.getElementById('footerUpstream').textContent = stats.upstream || '';
     document.getElementById('footerConfig').textContent = stats.config_path || '';
     document.getElementById('footerData').textContent = stats.data_dir || '';
-    const modeEl = document.getElementById('footerMode');
-    modeEl.textContent = stats.mode || '—';
-    modeEl.style.color = stats.mode === 'recursive' ? 'var(--emerald)' : 'var(--amber)';
     document.getElementById('footerDnssec').textContent = stats.dnssec ? 'on' : 'off';
     document.getElementById('footerDnssec').style.color = stats.dnssec ? 'var(--emerald)' : 'var(--text-dim)';
     document.getElementById('footerSrtt').textContent = stats.srtt ? 'on' : 'off';
@@ -1093,7 +945,7 @@ async function refresh() {
     prevTime = now;
 
     // Cache hit rate
-    const answered = q.cached + q.forwarded + q.recursive + q.coalesced + q.local + q.overridden;
+    const answered = q.cached + q.forwarded + q.local + q.overridden;
     const hitRate = answered > 0 ? ((q.cached / answered) * 100).toFixed(1) : '0.0';
     document.getElementById('cacheRate').textContent = hitRate + '%';
 
@@ -1105,7 +957,6 @@ async function refresh() {
     renderServices(services);
     renderBlockingInfo(blockingInfo);
     renderAllowlist(allowlist);
-    renderMemory(stats.memory, stats);
 
   } catch (err) {
     document.getElementById('statusDot').className = 'status-dot error';
@@ -1385,7 +1236,6 @@ setInterval(refresh, 2000);
   Config: <span id="footerConfig" style="user-select:all;color:var(--emerald);"></span>
   · Data: <span id="footerData" style="user-select:all;color:var(--emerald);"></span>
   · Upstream: <span id="footerUpstream" style="user-select:all;color:var(--emerald);"></span>
-  · Mode: <span id="footerMode" style="color:var(--text-dim);">—</span>
   · DNSSEC: <span id="footerDnssec" style="color:var(--text-dim);">—</span>
   · SRTT: <span id="footerSrtt" style="color:var(--text-dim);">—</span>
   · Logs: <span style="user-select:all;color:var(--emerald);">macOS: /usr/local/var/log/numa.log · Linux: journalctl -u numa -f</span>

site/index.html

@@ -4,10 +4,10 @@
 <meta charset="UTF-8">
 <meta name="viewport" content="width=device-width, initial-scale=1.0">
 <title>Numa — DNS you own. Everywhere you go.</title>
-<meta name="description" content="DNS you own. Portable DNS resolver with caching, ad blocking, .numa local domains, developer overrides. Optional recursive resolution with full DNSSEC validation. Built from scratch in Rust.">
+<meta name="description" content="DNS you own. Recursive resolver with full DNSSEC validation, ad blocking, .numa local domains, developer overrides. A single portable binary built from scratch in Rust.">
 <link rel="canonical" href="https://numa.rs">
 <meta property="og:title" content="Numa — DNS you own. Everywhere you go.">
-<meta property="og:description" content="Portable DNS resolver with caching, ad blocking, .numa local domains, and developer overrides. Optional recursive resolution with full DNSSEC validation. Built from scratch in Rust.">
+<meta property="og:description" content="Recursive DNS resolver with full DNSSEC validation, ad blocking, .numa local domains, and developer overrides. Built from scratch in Rust.">
 <meta property="og:type" content="website">
 <meta property="og:url" content="https://numa.rs">
 <link rel="stylesheet" href="/fonts/fonts.css">
@@ -1232,17 +1232,17 @@ footer .closing {
     <div class="reveal">
       <div class="section-label">How It Works</div>
       <h2>What it does today</h2>
-      <p class="lead">A DNS resolver with caching, ad blocking, local service domains, and a REST API. Optional recursive resolution with DNSSEC. Everything runs in a single binary.</p>
+      <p class="lead">A recursive DNS resolver with DNSSEC validation, ad blocking, local service domains, and a REST API. Everything runs in a single binary.</p>
     </div>
     <div class="layers-grid">
       <div class="layer-card reveal reveal-delay-1">
         <div class="layer-badge">Layer 1</div>
         <h3>Resolve &amp; Protect</h3>
         <ul>
-          <li>Forward mode by default &mdash; transparent proxy to your existing DNS, with caching</li>
-          <li>Ad &amp; tracker blocking &mdash; 385K+ domains, zero config</li>
-          <li>Recursive resolution &mdash; opt-in, resolve from root nameservers, no upstream needed</li>
+          <li>Recursive resolution &mdash; resolve from root nameservers, no upstream needed</li>
           <li>DNSSEC validation &mdash; chain-of-trust + NSEC/NSEC3 denial proofs (RSA, ECDSA, Ed25519)</li>
+          <li>Ad &amp; tracker blocking &mdash; 385K+ domains, zero config</li>
+          <li>DNS-over-HTTPS &mdash; encrypted upstream as alternative to recursive mode</li>
           <li>TTL-aware caching (sub-ms lookups)</li>
           <li>Single binary, portable &mdash; macOS, Linux, and Windows</li>
         </ul>

src/api.rs

@@ -160,7 +160,6 @@ struct QueryLogResponse {
 struct StatsResponse {
     uptime_secs: u64,
     upstream: String,
-    mode: &'static str, // "recursive" or "forward" — never "auto" at runtime
     config_path: String,
     data_dir: String,
     dnssec: bool,
@@ -170,7 +169,6 @@ struct StatsResponse {
     overrides: OverrideStats,
     blocking: BlockingStatsResponse,
     lan: LanStatsResponse,
-    memory: MemoryStats,
 }
 
 #[derive(Serialize)]
@@ -211,19 +209,6 @@ struct BlockingStatsResponse {
     allowlist_size: usize,
 }
 
-#[derive(Serialize)]
-struct MemoryStats {
-    cache_bytes: usize,
-    blocklist_bytes: usize,
-    query_log_bytes: usize,
-    query_log_entries: usize,
-    srtt_bytes: usize,
-    srtt_entries: usize,
-    overrides_bytes: usize,
-    total_estimated_bytes: usize,
-    process_memory_bytes: usize,
-}
-
 #[derive(Serialize)]
 struct DiagnoseResponse {
     domain: String,
@@ -425,8 +410,14 @@ async fn forward_query_for_diagnose(
     timeout: std::time::Duration,
 ) -> (bool, String) {
     use crate::packet::DnsPacket;
+    use crate::question::DnsQuestion;
 
-    let query = DnsPacket::query(0xBEEF, domain, QueryType::A);
+    let mut query = DnsPacket::new();
+    query.header.id = 0xBEEF;
+    query.header.recursion_desired = true;
+    query
+        .questions
+        .push(DnsQuestion::new(domain.to_string(), QueryType::A));
 
     match forward_query(&query, upstream, timeout).await {
         Ok(resp) => (
@@ -485,29 +476,12 @@ async fn query_log(
 
 async fn stats(State(ctx): State<Arc<ServerCtx>>) -> Json<StatsResponse> {
     let snap = ctx.stats.lock().unwrap().snapshot();
-    let (cache_len, cache_max, cache_bytes) = {
+    let (cache_len, cache_max) = {
         let cache = ctx.cache.read().unwrap();
-        (cache.len(), cache.max_entries(), cache.heap_bytes())
+        (cache.len(), cache.max_entries())
     };
-    let (override_count, overrides_bytes) = {
-        let ov = ctx.overrides.read().unwrap();
-        (ov.active_count(), ov.heap_bytes())
-    };
-    let (bl_stats, blocklist_bytes) = {
-        let bl = ctx.blocklist.read().unwrap();
-        (bl.stats(), bl.heap_bytes())
-    };
-    let (query_log_bytes, query_log_entries) = {
-        let log = ctx.query_log.lock().unwrap();
-        (log.heap_bytes(), log.len())
-    };
-    let (srtt_bytes, srtt_entries, srtt_enabled) = {
-        let s = ctx.srtt.read().unwrap();
-        (s.heap_bytes(), s.len(), s.is_enabled())
-    };
-
-    let total_estimated =
-        cache_bytes + blocklist_bytes + query_log_bytes + srtt_bytes + overrides_bytes;
+    let override_count = ctx.overrides.read().unwrap().active_count();
+    let bl_stats = ctx.blocklist.read().unwrap().stats();
 
     let upstream = if ctx.upstream_mode == crate::config::UpstreamMode::Recursive {
         "recursive (root hints)".to_string()
@@ -518,11 +492,10 @@ async fn stats(State(ctx): State<Arc<ServerCtx>>) -> Json<StatsResponse> {
     Json(StatsResponse {
         uptime_secs: snap.uptime_secs,
         upstream,
-        mode: ctx.upstream_mode.as_str(),
         config_path: ctx.config_path.clone(),
         data_dir: ctx.data_dir.to_string_lossy().to_string(),
         dnssec: ctx.dnssec_enabled,
-        srtt: srtt_enabled,
+        srtt: ctx.srtt.read().unwrap().is_enabled(),
         queries: QueriesStats {
             total: snap.total,
             forwarded: snap.forwarded,
@@ -551,17 +524,6 @@ async fn stats(State(ctx): State<Arc<ServerCtx>>) -> Json<StatsResponse> {
             enabled: ctx.lan_enabled,
             peers: ctx.lan_peers.lock().unwrap().list().len(),
         },
-        memory: MemoryStats {
-            cache_bytes,
-            blocklist_bytes,
-            query_log_bytes,
-            query_log_entries,
-            srtt_bytes,
-            srtt_entries,
-            overrides_bytes,
-            total_estimated_bytes: total_estimated,
-            process_memory_bytes: crate::stats::process_memory_bytes(),
-        },
     })
 }
 
@@ -906,7 +868,7 @@ async fn remove_route(
 }
 
 async fn serve_ca(State(ctx): State<Arc<ServerCtx>>) -> Result<impl IntoResponse, StatusCode> {
-    let ca_path = ctx.data_dir.join(crate::tls::CA_FILE_NAME);
+    let ca_path = ctx.data_dir.join("ca.pem");
     let bytes = tokio::task::spawn_blocking(move || std::fs::read(ca_path))
         .await
         .map_err(|_| StatusCode::INTERNAL_SERVER_ERROR)?

src/blocklist.rs

@@ -183,15 +183,6 @@ impl BlocklistStore {
         self.allowlist.iter().cloned().collect()
     }
 
-    pub fn heap_bytes(&self) -> usize {
-        let per_slot_overhead = std::mem::size_of::<u64>() + std::mem::size_of::<String>() + 1;
-        let domains_table = self.domains.capacity() * per_slot_overhead;
-        let domains_heap: usize = self.domains.iter().map(|d| d.capacity()).sum();
-        let allow_table = self.allowlist.capacity() * per_slot_overhead;
-        let allow_heap: usize = self.allowlist.iter().map(|d| d.capacity()).sum();
-        domains_table + domains_heap + allow_table + allow_heap
-    }
-
     pub fn stats(&self) -> BlocklistStats {
         BlocklistStats {
             enabled: self.is_enabled(),
@@ -243,23 +234,6 @@ pub fn parse_blocklist(text: &str) -> HashSet<String> {
     domains
 }
 
-#[cfg(test)]
-mod tests {
-    use super::*;
-
-    #[test]
-    fn heap_bytes_grows_with_domains() {
-        let mut store = BlocklistStore::new();
-        let empty = store.heap_bytes();
-        let domains: HashSet<String> = ["example.com", "example.org", "test.net"]
-            .iter()
-            .map(|s| s.to_string())
-            .collect();
-        store.swap_domains(domains, vec![]);
-        assert!(store.heap_bytes() > empty);
-    }
-}
-
 pub async fn download_blocklists(lists: &[String]) -> Vec<(String, String)> {
     let client = reqwest::Client::builder()
         .timeout(std::time::Duration::from_secs(30))

src/buffer.rs

@@ -84,11 +84,6 @@ impl BytePacketBuffer {
 
     /// Read a qname, handling label compression (pointer jumps).
     /// Converts wire format like [3]www[6]google[3]com[0] into "www.google.com".
-    ///
-    /// Label bytes are escaped per RFC 1035 §5.1:
-    /// - literal `.` within a label → `\.`
-    /// - literal `\` → `\\`
-    /// - bytes outside `0x21..=0x7E` (excluding `.` and `\`) → `\DDD` (3-digit decimal)
     pub fn read_qname(&mut self, outstr: &mut String) -> Result<()> {
         let mut pos = self.pos();
         let mut jumped = false;
@@ -126,18 +121,7 @@ impl BytePacketBuffer {
 
                 let str_buffer = self.get_range(pos, len as usize)?;
                 for &b in str_buffer {
-                    let c = b.to_ascii_lowercase();
-                    match c {
-                        b'.' => outstr.push_str("\\."),
-                        b'\\' => outstr.push_str("\\\\"),
-                        0x21..=0x7E => outstr.push(c as char),
-                        _ => {
-                            outstr.push('\\');
-                            outstr.push((b'0' + c / 100) as char);
-                            outstr.push((b'0' + (c / 10) % 10) as char);
-                            outstr.push((b'0' + c % 10) as char);
-                        }
-                    }
+                    outstr.push(b.to_ascii_lowercase() as char);
                 }
 
                 delim = ".";
@@ -179,68 +163,24 @@ impl BytePacketBuffer {
         Ok(())
     }
 
-    /// Write a qname in wire format, parsing RFC 1035 §5.1 text escapes.
-    /// See `read_qname` for the escape grammar.
     pub fn write_qname(&mut self, qname: &str) -> Result<()> {
         if qname.is_empty() || qname == "." {
             self.write_u8(0)?;
             return Ok(());
         }
 
-        let bytes = qname.as_bytes();
-        let mut i = 0;
-        while i < bytes.len() {
-            let len_pos = self.pos;
-            self.write_u8(0)?; // placeholder length byte, backpatched below
-            let body_start = self.pos;
-
-            while i < bytes.len() && bytes[i] != b'.' {
-                let b = bytes[i];
-                if b == b'\\' {
-                    i += 1;
-                    let c1 = *bytes.get(i).ok_or("trailing backslash in qname")?;
-                    if c1.is_ascii_digit() {
-                        let c2 = *bytes
-                            .get(i + 1)
-                            .ok_or("invalid \\DDD escape: expected 3 digits")?;
-                        let c3 = *bytes
-                            .get(i + 2)
-                            .ok_or("invalid \\DDD escape: expected 3 digits")?;
-                        if !c2.is_ascii_digit() || !c3.is_ascii_digit() {
-                            return Err("invalid \\DDD escape: expected 3 digits".into());
-                        }
-                        let val =
-                            (c1 - b'0') as u16 * 100 + (c2 - b'0') as u16 * 10 + (c3 - b'0') as u16;
-                        if val > 255 {
-                            return Err(format!("\\DDD escape out of range: {}", val).into());
-                        }
-                        self.write_u8(val as u8)?;
-                        i += 3;
-                    } else {
-                        // \. \\ and any other \X → literal next byte
-                        self.write_u8(c1)?;
-                        i += 1;
-                    }
-                } else {
-                    self.write_u8(b)?;
-                    i += 1;
-                }
-
-                if self.pos - body_start > 0x3f {
-                    return Err("Single label exceeds 63 characters of length".into());
-                }
+        for label in qname.split('.') {
+            let len = label.len();
+            if len == 0 {
+                continue; // skip empty labels from trailing dot
+            }
+            if len > 0x3f {
+                return Err("Single label exceeds 63 characters of length".into());
             }
 
-            let label_len = self.pos - body_start;
-            if label_len == 0 && i < bytes.len() {
-                // Empty label from leading/consecutive dots — roll back the placeholder.
-                self.pos = len_pos;
-            } else {
-                self.set(len_pos, label_len as u8)?;
-            }
-
-            if i < bytes.len() && bytes[i] == b'.' {
-                i += 1;
+            self.write_u8(len as u8)?;
+            for b in label.as_bytes() {
+                self.write_u8(*b)?;
             }
         }
 
@@ -272,160 +212,3 @@ impl BytePacketBuffer {
         Ok(())
     }
 }
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-
-    fn roundtrip(wire: &[u8]) -> String {
-        let mut buf = BytePacketBuffer::from_bytes(wire);
-        let mut out = String::new();
-        buf.read_qname(&mut out).unwrap();
-        out
-    }
-
-    fn write_then_read(text: &str) -> String {
-        let mut buf = BytePacketBuffer::new();
-        buf.write_qname(text).unwrap();
-        let wire_end = buf.pos();
-        buf.seek(0).unwrap();
-        let mut out = String::new();
-        buf.read_qname(&mut out).unwrap();
-        assert_eq!(
-            buf.pos(),
-            wire_end,
-            "reader should consume exactly what writer wrote"
-        );
-        out
-    }
-
-    #[test]
-    fn read_plain_domain() {
-        // [3]www[6]google[3]com[0]
-        let wire = b"\x03www\x06google\x03com\x00";
-        assert_eq!(roundtrip(wire), "www.google.com");
-    }
-
-    #[test]
-    fn read_label_with_literal_dot_is_escaped() {
-        // fanf2's example: [8]exa.mple[3]com[0] — two labels, first contains 0x2E
-        let wire = b"\x08exa.mple\x03com\x00";
-        assert_eq!(roundtrip(wire), "exa\\.mple.com");
-    }
-
-    #[test]
-    fn read_label_with_backslash_is_escaped() {
-        // [4]a\bc[3]com[0]
-        let wire = b"\x04a\\bc\x03com\x00";
-        assert_eq!(roundtrip(wire), "a\\\\bc.com");
-    }
-
-    #[test]
-    fn read_label_with_nonprintable_byte_uses_decimal_escape() {
-        // [4]\x00foo[3]com[0] — null byte at label start
-        let wire = b"\x04\x00foo\x03com\x00";
-        assert_eq!(roundtrip(wire), "\\000foo.com");
-    }
-
-    #[test]
-    fn read_label_with_space_uses_decimal_escape() {
-        // Space (0x20) is outside 0x21..=0x7E, so it must be decimal-escaped.
-        let wire = b"\x05a b c\x00";
-        assert_eq!(roundtrip(wire), "a\\032b\\032c");
-    }
-
-    #[test]
-    fn write_plain_domain() {
-        let mut buf = BytePacketBuffer::new();
-        buf.write_qname("www.google.com").unwrap();
-        assert_eq!(&buf.buf[..buf.pos], b"\x03www\x06google\x03com\x00");
-    }
-
-    #[test]
-    fn write_escaped_dot_does_not_split_label() {
-        let mut buf = BytePacketBuffer::new();
-        buf.write_qname("exa\\.mple.com").unwrap();
-        assert_eq!(&buf.buf[..buf.pos], b"\x08exa.mple\x03com\x00");
-    }
-
-    #[test]
-    fn write_escaped_backslash() {
-        let mut buf = BytePacketBuffer::new();
-        buf.write_qname("a\\\\bc.com").unwrap();
-        assert_eq!(&buf.buf[..buf.pos], b"\x04a\\bc\x03com\x00");
-    }
-
-    #[test]
-    fn write_decimal_escape_yields_raw_byte() {
-        let mut buf = BytePacketBuffer::new();
-        buf.write_qname("\\000foo.com").unwrap();
-        assert_eq!(&buf.buf[..buf.pos], b"\x04\x00foo\x03com\x00");
-    }
-
-    #[test]
-    fn write_skips_empty_labels() {
-        // Leading dot — first (empty) label is rolled back.
-        let mut buf = BytePacketBuffer::new();
-        buf.write_qname(".foo.com").unwrap();
-        assert_eq!(&buf.buf[..buf.pos], b"\x03foo\x03com\x00");
-
-        // Consecutive dots — middle empty label is rolled back.
-        let mut buf = BytePacketBuffer::new();
-        buf.write_qname("foo..com").unwrap();
-        assert_eq!(&buf.buf[..buf.pos], b"\x03foo\x03com\x00");
-    }
-
-    #[test]
-    fn write_rejects_out_of_range_decimal_escape() {
-        let mut buf = BytePacketBuffer::new();
-        assert!(buf.write_qname("\\999foo.com").is_err());
-    }
-
-    #[test]
-    fn write_rejects_trailing_backslash() {
-        let mut buf = BytePacketBuffer::new();
-        assert!(buf.write_qname("foo\\").is_err());
-    }
-
-    #[test]
-    fn write_rejects_short_decimal_escape() {
-        let mut buf = BytePacketBuffer::new();
-        assert!(buf.write_qname("\\1").is_err());
-    }
-
-    #[test]
-    fn write_rejects_label_over_63_bytes() {
-        // 64 bytes exceeds the wire-format label cap.
-        let mut buf = BytePacketBuffer::new();
-        assert!(buf.write_qname(&"a".repeat(64)).is_err());
-
-        // 63 bytes is the maximum permitted label length.
-        let mut buf = BytePacketBuffer::new();
-        assert!(buf.write_qname(&"a".repeat(63)).is_ok());
-    }
-
-    #[test]
-    fn roundtrip_preserves_dot_in_label() {
-        assert_eq!(write_then_read("exa\\.mple.com"), "exa\\.mple.com");
-    }
-
-    #[test]
-    fn roundtrip_preserves_backslash_in_label() {
-        assert_eq!(write_then_read("a\\\\b.com"), "a\\\\b.com");
-    }
-
-    #[test]
-    fn roundtrip_preserves_nonprintable_byte() {
-        assert_eq!(write_then_read("\\000foo.com"), "\\000foo.com");
-    }
-
-    #[test]
-    fn root_name_empty_and_dot_both_produce_single_zero() {
-        let mut a = BytePacketBuffer::new();
-        a.write_qname("").unwrap();
-        let mut b = BytePacketBuffer::new();
-        b.write_qname(".").unwrap();
-        assert_eq!(&a.buf[..a.pos], b"\x00");
-        assert_eq!(&b.buf[..b.pos], b"\x00");
-    }
-}

src/cache.rs

@@ -142,26 +142,6 @@ impl DnsCache {
         self.entry_count = 0;
     }
 
-    pub fn heap_bytes(&self) -> usize {
-        let outer_slot = std::mem::size_of::<u64>()
-            + std::mem::size_of::<String>()
-            + std::mem::size_of::<HashMap<QueryType, CacheEntry>>()
-            + 1;
-        let mut total = self.entries.capacity() * outer_slot;
-        for (domain, type_map) in &self.entries {
-            total += domain.capacity();
-            let inner_slot = std::mem::size_of::<u64>()
-                + std::mem::size_of::<QueryType>()
-                + std::mem::size_of::<CacheEntry>()
-                + 1;
-            total += type_map.capacity() * inner_slot;
-            for entry in type_map.values() {
-                total += entry.packet.heap_bytes();
-            }
-        }
-        total
-    }
-
     pub fn remove(&mut self, domain: &str) {
         let domain_lower = domain.to_lowercase();
         if let Some(type_map) = self.entries.remove(&domain_lower) {
@@ -214,23 +194,3 @@ fn adjust_ttls(records: &mut [DnsRecord], new_ttl: u32) {
         record.set_ttl(new_ttl);
     }
 }
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-    use crate::packet::DnsPacket;
-
-    #[test]
-    fn heap_bytes_grows_with_entries() {
-        let mut cache = DnsCache::new(100, 1, 3600);
-        let empty = cache.heap_bytes();
-        let mut pkt = DnsPacket::new();
-        pkt.answers.push(DnsRecord::A {
-            domain: "example.com".into(),
-            addr: "1.2.3.4".parse().unwrap(),
-            ttl: 300,
-        });
-        cache.insert("example.com", QueryType::A, &pkt);
-        assert!(cache.heap_bytes() > empty);
-    }
-}

src/config.rs

@@ -1,7 +1,7 @@
 use std::collections::HashMap;
 use std::net::Ipv4Addr;
 use std::net::Ipv6Addr;
-use std::path::{Path, PathBuf};
+use std::path::Path;
 
 use serde::Deserialize;
 
@@ -29,8 +29,6 @@ pub struct Config {
     pub lan: LanConfig,
     #[serde(default)]
     pub dnssec: DnssecConfig,
-    #[serde(default)]
-    pub dot: DotConfig,
 }
 
 #[derive(Deserialize)]
@@ -41,10 +39,6 @@ pub struct ServerConfig {
     pub api_port: u16,
     #[serde(default = "default_api_bind_addr")]
     pub api_bind_addr: String,
-    /// Where numa writes TLS material (CA, leaf certs, regenerated state).
-    /// Defaults to `crate::data_dir()` (platform-specific system path) if unset.
-    #[serde(default)]
-    pub data_dir: Option<PathBuf>,
 }
 
 impl Default for ServerConfig {
@@ -53,7 +47,6 @@ impl Default for ServerConfig {
             bind_addr: default_bind_addr(),
             api_port: default_api_port(),
             api_bind_addr: default_api_bind_addr(),
-            data_dir: None,
         }
     }
 }
@@ -66,31 +59,18 @@ fn default_bind_addr() -> String {
     "0.0.0.0:53".to_string()
 }
 
-pub const DEFAULT_API_PORT: u16 = 5380;
-
 fn default_api_port() -> u16 {
-    DEFAULT_API_PORT
+    5380
 }
 
 #[derive(Deserialize, Default, PartialEq, Eq, Clone, Copy)]
 #[serde(rename_all = "lowercase")]
 pub enum UpstreamMode {
-    Auto,
     #[default]
     Forward,
     Recursive,
 }
 
-impl UpstreamMode {
-    pub fn as_str(&self) -> &'static str {
-        match self {
-            UpstreamMode::Auto => "auto",
-            UpstreamMode::Forward => "forward",
-            UpstreamMode::Recursive => "recursive",
-        }
-    }
-}
-
 #[derive(Deserialize)]
 pub struct UpstreamConfig {
     #[serde(default)]
@@ -123,12 +103,8 @@ impl Default for UpstreamConfig {
     }
 }
 
-fn default_true() -> bool {
-    true
-}
-
 fn default_srtt() -> bool {
-    default_true()
+    true
 }
 
 fn default_prime_tlds() -> Vec<String> {
@@ -377,41 +353,6 @@ pub struct DnssecConfig {
     pub strict: bool,
 }
 
-#[derive(Deserialize, Clone)]
-pub struct DotConfig {
-    #[serde(default)]
-    pub enabled: bool,
-    #[serde(default = "default_dot_port")]
-    pub port: u16,
-    #[serde(default = "default_dot_bind_addr")]
-    pub bind_addr: String,
-    /// Path to TLS certificate (PEM). If None, uses self-signed CA.
-    #[serde(default)]
-    pub cert_path: Option<PathBuf>,
-    /// Path to TLS private key (PEM). If None, uses self-signed CA.
-    #[serde(default)]
-    pub key_path: Option<PathBuf>,
-}
-
-impl Default for DotConfig {
-    fn default() -> Self {
-        DotConfig {
-            enabled: false,
-            port: default_dot_port(),
-            bind_addr: default_dot_bind_addr(),
-            cert_path: None,
-            key_path: None,
-        }
-    }
-}
-
-fn default_dot_port() -> u16 {
-    853
-}
-fn default_dot_bind_addr() -> String {
-    "0.0.0.0".to_string()
-}
-
 #[cfg(test)]
 mod tests {
     use super::*;

src/ctx.rs

@@ -62,21 +62,24 @@ pub struct ServerCtx {
     pub dnssec_strict: bool,
 }
 
-/// Transport-agnostic DNS resolution. Runs the full pipeline (overrides, blocklist,
-/// cache, upstream, DNSSEC) and returns the serialized response in a buffer.
-/// Callers use `.filled()` to get the response bytes without heap allocation.
-/// Callers are responsible for parsing the incoming buffer into a `DnsPacket`
-/// (and logging parse errors) before calling this function.
-pub async fn resolve_query(
-    query: DnsPacket,
+pub async fn handle_query(
+    mut buffer: BytePacketBuffer,
     src_addr: SocketAddr,
     ctx: &ServerCtx,
-) -> crate::Result<BytePacketBuffer> {
+) -> crate::Result<()> {
     let start = Instant::now();
 
+    let query = match DnsPacket::from_buffer(&mut buffer) {
+        Ok(packet) => packet,
+        Err(e) => {
+            warn!("{} | PARSE ERROR | {}", src_addr, e);
+            return Ok(());
+        }
+    };
+
     let (qname, qtype) = match query.questions.first() {
         Some(q) => (q.name.clone(), q.qtype),
-        None => return Err("empty question section".into()),
+        None => return Ok(()),
     };
 
     // Pipeline: overrides -> .tld interception -> blocklist -> local zones -> cache -> upstream
@@ -90,13 +93,18 @@ pub async fn resolve_query(
         } else if qname == "localhost" || qname.ends_with(".localhost") {
             // RFC 6761: .localhost always resolves to loopback
             let mut resp = DnsPacket::response_from(&query, ResultCode::NOERROR);
-            resp.answers.push(sinkhole_record(
-                &qname,
-                qtype,
-                std::net::Ipv4Addr::LOCALHOST,
-                std::net::Ipv6Addr::LOCALHOST,
-                300,
-            ));
+            match qtype {
+                QueryType::AAAA => resp.answers.push(DnsRecord::AAAA {
+                    domain: qname.clone(),
+                    addr: std::net::Ipv6Addr::LOCALHOST,
+                    ttl: 300,
+                }),
+                _ => resp.answers.push(DnsRecord::A {
+                    domain: qname.clone(),
+                    addr: std::net::Ipv4Addr::LOCALHOST,
+                    ttl: 300,
+                }),
+            }
             (resp, QueryPath::Local, DnssecStatus::Indeterminate)
         } else if is_special_use_domain(&qname) {
             // RFC 6761/8880: private PTR, DDR, NAT64 — answer locally
@@ -105,17 +113,12 @@ pub async fn resolve_query(
         } else if !ctx.proxy_tld_suffix.is_empty()
             && (qname.ends_with(&ctx.proxy_tld_suffix) || qname == ctx.proxy_tld)
         {
-            // Resolve .numa: remote clients get LAN IP (can't reach 127.0.0.1), local get loopback
+            // Resolve .numa: local services → 127.0.0.1, LAN peers → peer IP
             let service_name = qname.strip_suffix(&ctx.proxy_tld_suffix).unwrap_or(&qname);
-            let is_remote = !src_addr.ip().is_loopback();
             let resolve_ip = {
                 let local = ctx.services.lock().unwrap();
                 if local.lookup(service_name).is_some() {
-                    if is_remote {
-                        *ctx.lan_ip.lock().unwrap()
-                    } else {
-                        std::net::Ipv4Addr::LOCALHOST
-                    }
+                    std::net::Ipv4Addr::LOCALHOST
                 } else {
                     let mut peers = ctx.lan_peers.lock().unwrap();
                     peers
@@ -127,24 +130,38 @@ pub async fn resolve_query(
                         .unwrap_or(std::net::Ipv4Addr::LOCALHOST)
                 }
             };
-            let v6 = if resolve_ip == std::net::Ipv4Addr::LOCALHOST {
-                std::net::Ipv6Addr::LOCALHOST
-            } else {
-                resolve_ip.to_ipv6_mapped()
-            };
             let mut resp = DnsPacket::response_from(&query, ResultCode::NOERROR);
-            resp.answers
-                .push(sinkhole_record(&qname, qtype, resolve_ip, v6, 300));
+            match qtype {
+                QueryType::AAAA => resp.answers.push(DnsRecord::AAAA {
+                    domain: qname.clone(),
+                    addr: if resolve_ip == std::net::Ipv4Addr::LOCALHOST {
+                        std::net::Ipv6Addr::LOCALHOST
+                    } else {
+                        resolve_ip.to_ipv6_mapped()
+                    },
+                    ttl: 300,
+                }),
+                _ => resp.answers.push(DnsRecord::A {
+                    domain: qname.clone(),
+                    addr: resolve_ip,
+                    ttl: 300,
+                }),
+            }
             (resp, QueryPath::Local, DnssecStatus::Indeterminate)
         } else if ctx.blocklist.read().unwrap().is_blocked(&qname) {
             let mut resp = DnsPacket::response_from(&query, ResultCode::NOERROR);
-            resp.answers.push(sinkhole_record(
-                &qname,
-                qtype,
-                std::net::Ipv4Addr::UNSPECIFIED,
-                std::net::Ipv6Addr::UNSPECIFIED,
-                60,
-            ));
+            match qtype {
+                QueryType::AAAA => resp.answers.push(DnsRecord::AAAA {
+                    domain: qname.clone(),
+                    addr: std::net::Ipv6Addr::UNSPECIFIED,
+                    ttl: 60,
+                }),
+                _ => resp.answers.push(DnsRecord::A {
+                    domain: qname.clone(),
+                    addr: std::net::Ipv4Addr::UNSPECIFIED,
+                    ttl: 60,
+                }),
+            }
             (resp, QueryPath::Blocked, DnssecStatus::Indeterminate)
         } else if let Some(records) = ctx.zone_map.get(qname.as_str()).and_then(|m| m.get(&qtype)) {
             let mut resp = DnsPacket::response_from(&query, ResultCode::NOERROR);
@@ -159,54 +176,64 @@ pub async fn resolve_query(
                     resp.header.authed_data = true;
                 }
                 (resp, QueryPath::Cached, cached_dnssec)
-            } else if let Some(fwd_addr) =
-                crate::system_dns::match_forwarding_rule(&qname, &ctx.forwarding_rules)
-            {
-                // Conditional forwarding takes priority over recursive mode
-                // (e.g. Tailscale .ts.net, VPC private zones)
-                let upstream = Upstream::Udp(fwd_addr);
-                match forward_query(&query, &upstream, ctx.timeout).await {
-                    Ok(resp) => {
-                        ctx.cache.write().unwrap().insert(&qname, qtype, &resp);
-                        (resp, QueryPath::Forwarded, DnssecStatus::Indeterminate)
-                    }
-                    Err(e) => {
-                        error!(
-                            "{} | {:?} {} | FORWARD ERROR | {}",
-                            src_addr, qtype, qname, e
-                        );
-                        (
-                            DnsPacket::response_from(&query, ResultCode::SERVFAIL),
-                            QueryPath::UpstreamError,
-                            DnssecStatus::Indeterminate,
-                        )
-                    }
-                }
             } else if ctx.upstream_mode == UpstreamMode::Recursive {
                 let key = (qname.clone(), qtype);
-                let (resp, path, err) = resolve_coalesced(&ctx.inflight, key, &query, || {
-                    crate::recursive::resolve_recursive(
-                        &qname,
-                        qtype,
-                        &ctx.cache,
-                        &query,
-                        &ctx.root_hints,
-                        &ctx.srtt,
-                    )
-                })
-                .await;
-                if path == QueryPath::Coalesced {
-                    debug!("{} | {:?} {} | COALESCED", src_addr, qtype, qname);
-                } else if path == QueryPath::UpstreamError {
-                    error!(
-                        "{} | {:?} {} | RECURSIVE ERROR | {}",
-                        src_addr,
-                        qtype,
-                        qname,
-                        err.as_deref().unwrap_or("leader failed")
-                    );
+                let disposition = acquire_inflight(&ctx.inflight, key.clone());
+
+                match disposition {
+                    Disposition::Follower(mut rx) => {
+                        debug!("{} | {:?} {} | COALESCED", src_addr, qtype, qname);
+                        match rx.recv().await {
+                            Ok(Some(mut resp)) => {
+                                resp.header.id = query.header.id;
+                                (resp, QueryPath::Coalesced, DnssecStatus::Indeterminate)
+                            }
+                            _ => (
+                                DnsPacket::response_from(&query, ResultCode::SERVFAIL),
+                                QueryPath::UpstreamError,
+                                DnssecStatus::Indeterminate,
+                            ),
+                        }
+                    }
+                    Disposition::Leader(tx) => {
+                        // Drop guard: remove inflight entry even on panic/cancellation
+                        let guard = InflightGuard {
+                            inflight: &ctx.inflight,
+                            key: key.clone(),
+                        };
+
+                        let result = crate::recursive::resolve_recursive(
+                            &qname,
+                            qtype,
+                            &ctx.cache,
+                            &query,
+                            &ctx.root_hints,
+                            &ctx.srtt,
+                        )
+                        .await;
+
+                        drop(guard);
+
+                        match result {
+                            Ok(resp) => {
+                                let _ = tx.send(Some(resp.clone()));
+                                (resp, QueryPath::Recursive, DnssecStatus::Indeterminate)
+                            }
+                            Err(e) => {
+                                let _ = tx.send(None);
+                                error!(
+                                    "{} | {:?} {} | RECURSIVE ERROR | {}",
+                                    src_addr, qtype, qname, e
+                                );
+                                (
+                                    DnsPacket::response_from(&query, ResultCode::SERVFAIL),
+                                    QueryPath::UpstreamError,
+                                    DnssecStatus::Indeterminate,
+                                )
+                            }
+                        }
+                    }
                 }
-                (resp, path, DnssecStatus::Indeterminate)
             } else {
                 let upstream =
                     match crate::system_dns::match_forwarding_rule(&qname, &ctx.forwarding_rules) {
@@ -303,17 +330,17 @@ pub async fn resolve_query(
         response.resources.len(),
     );
 
-    // Serialize response
-    // TODO: TC bit is UDP-specific; DoT connections could carry up to 65535 bytes.
-    // Once BytePacketBuffer supports larger buffers, skip truncation for TCP/TLS.
     let mut resp_buffer = BytePacketBuffer::new();
     if response.write(&mut resp_buffer).is_err() {
-        // Response too large — set TC bit and send header + question only
+        // Response too large for UDP — set TC bit and send header + question only
         debug!("response too large, setting TC bit for {}", qname);
         let mut tc_response = DnsPacket::response_from(&query, response.header.rescode);
         tc_response.header.truncated_message = true;
-        resp_buffer = BytePacketBuffer::new();
-        tc_response.write(&mut resp_buffer)?;
+        let mut tc_buffer = BytePacketBuffer::new();
+        tc_response.write(&mut tc_buffer)?;
+        ctx.socket.send_to(tc_buffer.filled(), src_addr).await?;
+    } else {
+        ctx.socket.send_to(resp_buffer.filled(), src_addr).await?;
     }
 
     // Record stats and query log
@@ -336,30 +363,6 @@ pub async fn resolve_query(
         dnssec,
     });
 
-    Ok(resp_buffer)
-}
-
-/// Handle a DNS query received over UDP. Thin wrapper around resolve_query.
-pub async fn handle_query(
-    mut buffer: BytePacketBuffer,
-    src_addr: SocketAddr,
-    ctx: &ServerCtx,
-) -> crate::Result<()> {
-    let query = match DnsPacket::from_buffer(&mut buffer) {
-        Ok(packet) => packet,
-        Err(e) => {
-            warn!("{} | PARSE ERROR | {}", src_addr, e);
-            return Ok(());
-        }
-    };
-    match resolve_query(query, src_addr, ctx).await {
-        Ok(resp_buffer) => {
-            ctx.socket.send_to(resp_buffer.filled(), src_addr).await?;
-        }
-        Err(e) => {
-            warn!("{} | RESOLVE ERROR | {}", src_addr, e);
-        }
-    }
     Ok(())
 }
 
@@ -413,27 +416,6 @@ fn is_special_use_domain(qname: &str) -> bool {
     qname == "local" || qname.ends_with(".local")
 }
 
-fn sinkhole_record(
-    domain: &str,
-    qtype: QueryType,
-    v4: std::net::Ipv4Addr,
-    v6: std::net::Ipv6Addr,
-    ttl: u32,
-) -> DnsRecord {
-    match qtype {
-        QueryType::AAAA => DnsRecord::AAAA {
-            domain: domain.to_string(),
-            addr: v6,
-            ttl,
-        },
-        _ => DnsRecord::A {
-            domain: domain.to_string(),
-            addr: v4,
-            ttl,
-        },
-    }
-}
-
 enum Disposition {
     Leader(broadcast::Sender<Option<DnsPacket>>),
     Follower(broadcast::Receiver<Option<DnsPacket>>),
@@ -450,57 +432,6 @@ fn acquire_inflight(inflight: &Mutex<InflightMap>, key: (String, QueryType)) ->
     }
 }
 
-/// Run a resolve function with in-flight coalescing. Multiple concurrent calls
-/// for the same key share a single resolution — the first caller (leader)
-/// executes `resolve_fn`, and followers wait for the broadcast result.
-async fn resolve_coalesced<F, Fut>(
-    inflight: &Mutex<InflightMap>,
-    key: (String, QueryType),
-    query: &DnsPacket,
-    resolve_fn: F,
-) -> (DnsPacket, QueryPath, Option<String>)
-where
-    F: FnOnce() -> Fut,
-    Fut: std::future::Future<Output = crate::Result<DnsPacket>>,
-{
-    let disposition = acquire_inflight(inflight, key.clone());
-
-    match disposition {
-        Disposition::Follower(mut rx) => match rx.recv().await {
-            Ok(Some(mut resp)) => {
-                resp.header.id = query.header.id;
-                (resp, QueryPath::Coalesced, None)
-            }
-            _ => (
-                DnsPacket::response_from(query, ResultCode::SERVFAIL),
-                QueryPath::UpstreamError,
-                None,
-            ),
-        },
-        Disposition::Leader(tx) => {
-            let guard = InflightGuard { inflight, key };
-            let result = resolve_fn().await;
-            drop(guard);
-
-            match result {
-                Ok(resp) => {
-                    let _ = tx.send(Some(resp.clone()));
-                    (resp, QueryPath::Recursive, None)
-                }
-                Err(e) => {
-                    let _ = tx.send(None);
-                    let err_msg = e.to_string();
-                    (
-                        DnsPacket::response_from(query, ResultCode::SERVFAIL),
-                        QueryPath::UpstreamError,
-                        Some(err_msg),
-                    )
-                }
-            }
-        }
-    }
-}
-
 struct InflightGuard<'a> {
     inflight: &'a Mutex<InflightMap>,
     key: (String, QueryType),
@@ -512,6 +443,20 @@ impl Drop for InflightGuard<'_> {
     }
 }
 
+/// Build a wire-format DNS query packet for the given domain and type.
+#[cfg(test)]
+fn build_wire_query(id: u16, domain: &str, qtype: QueryType) -> BytePacketBuffer {
+    let mut pkt = DnsPacket::new();
+    pkt.header.id = id;
+    pkt.header.recursion_desired = true;
+    pkt.header.questions = 1;
+    pkt.questions
+        .push(crate::question::DnsQuestion::new(domain.to_string(), qtype));
+    let mut buf = BytePacketBuffer::new();
+    pkt.write(&mut buf).unwrap();
+    BytePacketBuffer::from_bytes(buf.filled())
+}
+
 fn special_use_response(query: &DnsPacket, qname: &str, qtype: QueryType) -> DnsPacket {
     use std::net::{Ipv4Addr, Ipv6Addr};
     if qname == "ipv4only.arpa" {
@@ -550,8 +495,8 @@ fn special_use_response(query: &DnsPacket, qname: &str, qtype: QueryType) -> Dns
 mod tests {
     use super::*;
     use std::collections::HashMap;
-    use std::net::Ipv4Addr;
-    use std::sync::{Arc, Mutex};
+    use std::net::{Ipv4Addr, SocketAddr};
+    use std::sync::{Arc, Mutex, RwLock};
     use tokio::sync::broadcast;
 
     // ---- InflightGuard unit tests ----
@@ -724,212 +669,189 @@ mod tests {
         }
     }
 
-    // ---- Integration: resolve_coalesced with mock futures ----
+    // ---- Integration: concurrent handle_query coalescing ----
 
-    fn mock_response(domain: &str) -> DnsPacket {
-        let mut resp = DnsPacket::new();
-        resp.header.response = true;
-        resp.header.rescode = ResultCode::NOERROR;
-        resp.answers.push(DnsRecord::A {
-            domain: domain.to_string(),
-            addr: Ipv4Addr::new(10, 0, 0, 1),
-            ttl: 300,
+    use tokio::io::{AsyncReadExt, AsyncWriteExt};
+    use tokio::net::TcpListener;
+
+    /// Spawn a slow TCP DNS server that delays `delay` before responding.
+    /// Returns (addr, query_count) where query_count is an Arc<AtomicU32>
+    /// tracking how many queries were actually resolved (not coalesced).
+    async fn spawn_slow_dns_server(
+        delay: Duration,
+    ) -> (SocketAddr, Arc<std::sync::atomic::AtomicU32>) {
+        let listener = TcpListener::bind("127.0.0.1:0").await.unwrap();
+        let addr = listener.local_addr().unwrap();
+        let count = Arc::new(std::sync::atomic::AtomicU32::new(0));
+        let count_clone = count.clone();
+
+        tokio::spawn(async move {
+            loop {
+                let (mut stream, _) = match listener.accept().await {
+                    Ok(c) => c,
+                    Err(_) => break,
+                };
+                let count = count_clone.clone();
+                let delay = delay;
+                tokio::spawn(async move {
+                    let mut len_buf = [0u8; 2];
+                    if stream.read_exact(&mut len_buf).await.is_err() {
+                        return;
+                    }
+                    let len = u16::from_be_bytes(len_buf) as usize;
+                    let mut data = vec![0u8; len];
+                    if stream.read_exact(&mut data).await.is_err() {
+                        return;
+                    }
+
+                    let mut buf = BytePacketBuffer::from_bytes(&data);
+                    let query = match DnsPacket::from_buffer(&mut buf) {
+                        Ok(q) => q,
+                        Err(_) => return,
+                    };
+
+                    count.fetch_add(1, std::sync::atomic::Ordering::Relaxed);
+
+                    // Deliberate delay to create coalescing window
+                    tokio::time::sleep(delay).await;
+
+                    let mut resp = DnsPacket::response_from(&query, ResultCode::NOERROR);
+                    resp.header.authoritative_answer = true;
+                    if let Some(q) = query.questions.first() {
+                        resp.answers.push(DnsRecord::A {
+                            domain: q.name.clone(),
+                            addr: Ipv4Addr::new(10, 0, 0, 1),
+                            ttl: 300,
+                        });
+                    }
+
+                    let mut resp_buf = BytePacketBuffer::new();
+                    if resp.write(&mut resp_buf).is_err() {
+                        return;
+                    }
+                    let resp_bytes = resp_buf.filled();
+                    let mut out = Vec::with_capacity(2 + resp_bytes.len());
+                    out.extend_from_slice(&(resp_bytes.len() as u16).to_be_bytes());
+                    out.extend_from_slice(resp_bytes);
+                    let _ = stream.write_all(&out).await;
+                });
+            }
         });
-        resp
+        (addr, count)
+    }
+
+    async fn test_recursive_ctx(root_hint: SocketAddr) -> Arc<ServerCtx> {
+        let socket = tokio::net::UdpSocket::bind("127.0.0.1:0").await.unwrap();
+        Arc::new(ServerCtx {
+            socket,
+            zone_map: HashMap::new(),
+            cache: RwLock::new(crate::cache::DnsCache::new(100, 60, 86400)),
+            stats: Mutex::new(crate::stats::ServerStats::new()),
+            overrides: RwLock::new(crate::override_store::OverrideStore::new()),
+            blocklist: RwLock::new(crate::blocklist::BlocklistStore::new()),
+            query_log: Mutex::new(crate::query_log::QueryLog::new(100)),
+            services: Mutex::new(crate::service_store::ServiceStore::new()),
+            lan_peers: Mutex::new(crate::lan::PeerStore::new(90)),
+            forwarding_rules: Vec::new(),
+            upstream: Mutex::new(crate::forward::Upstream::Udp(
+                "127.0.0.1:53".parse().unwrap(),
+            )),
+            upstream_auto: false,
+            upstream_port: 53,
+            lan_ip: Mutex::new(Ipv4Addr::LOCALHOST),
+            timeout: Duration::from_secs(3),
+            proxy_tld: "numa".to_string(),
+            proxy_tld_suffix: ".numa".to_string(),
+            lan_enabled: false,
+            config_path: "/tmp/test-numa.toml".to_string(),
+            config_found: false,
+            config_dir: std::path::PathBuf::from("/tmp"),
+            data_dir: std::path::PathBuf::from("/tmp"),
+            tls_config: None,
+            upstream_mode: crate::config::UpstreamMode::Recursive,
+            root_hints: vec![root_hint],
+            srtt: RwLock::new(crate::srtt::SrttCache::new(true)),
+            inflight: Mutex::new(HashMap::new()),
+            dnssec_enabled: false,
+            dnssec_strict: false,
+        })
     }
 
     #[tokio::test]
     async fn concurrent_queries_coalesce_to_single_resolution() {
-        let inflight = Arc::new(Mutex::new(HashMap::new()));
-        let resolve_count = Arc::new(std::sync::atomic::AtomicU32::new(0));
+        // Force TCP-only so mock server works
+        crate::recursive::UDP_DISABLED.store(true, std::sync::atomic::Ordering::Release);
 
+        let (server_addr, query_count) = spawn_slow_dns_server(Duration::from_millis(200)).await;
+        let ctx = test_recursive_ctx(server_addr).await;
+        let src: SocketAddr = "127.0.0.1:9999".parse().unwrap();
+
+        // Fire 5 concurrent queries for the same (domain, A)
         let mut handles = Vec::new();
         for i in 0..5u16 {
-            let count = resolve_count.clone();
-            let inf = inflight.clone();
-            let key = ("coalesce.test".to_string(), QueryType::A);
-            let query = DnsPacket::query(100 + i, "coalesce.test", QueryType::A);
-            handles.push(tokio::spawn(async move {
-                resolve_coalesced(&inf, key, &query, || async {
-                    count.fetch_add(1, std::sync::atomic::Ordering::Relaxed);
-                    tokio::time::sleep(Duration::from_millis(200)).await;
-                    Ok(mock_response("coalesce.test"))
-                })
-                .await
-            }));
+            let ctx = ctx.clone();
+            let buf = build_wire_query(100 + i, "coalesce-test.example.com", QueryType::A);
+            handles.push(tokio::spawn(
+                async move { handle_query(buf, src, &ctx).await },
+            ));
         }
 
-        let mut paths = Vec::new();
         for h in handles {
-            let (_, path, _) = h.await.unwrap();
-            paths.push(path);
+            h.await.unwrap().unwrap();
         }
 
-        let actual = resolve_count.load(std::sync::atomic::Ordering::Relaxed);
-        assert_eq!(actual, 1, "expected 1 resolution, got {}", actual);
+        // Only 1 resolution should have reached the upstream server
+        let actual = query_count.load(std::sync::atomic::Ordering::Relaxed);
+        assert_eq!(actual, 1, "expected 1 upstream query, got {}", actual);
 
-        let recursive = paths.iter().filter(|p| **p == QueryPath::Recursive).count();
-        let coalesced = paths.iter().filter(|p| **p == QueryPath::Coalesced).count();
-        assert_eq!(recursive, 1, "expected 1 RECURSIVE, got {}", recursive);
-        assert_eq!(coalesced, 4, "expected 4 COALESCED, got {}", coalesced);
+        // Inflight map must be empty after all queries complete
+        assert!(ctx.inflight.lock().unwrap().is_empty());
 
-        assert!(inflight.lock().unwrap().is_empty());
+        crate::recursive::reset_udp_state();
     }
 
     #[tokio::test]
     async fn different_qtypes_not_coalesced() {
-        let inflight = Arc::new(Mutex::new(HashMap::new()));
-        let resolve_count = Arc::new(std::sync::atomic::AtomicU32::new(0));
+        crate::recursive::UDP_DISABLED.store(true, std::sync::atomic::Ordering::Release);
 
-        let inf1 = inflight.clone();
-        let inf2 = inflight.clone();
-        let count1 = resolve_count.clone();
-        let count2 = resolve_count.clone();
+        let (server_addr, query_count) = spawn_slow_dns_server(Duration::from_millis(100)).await;
+        let ctx = test_recursive_ctx(server_addr).await;
+        let src: SocketAddr = "127.0.0.1:9999".parse().unwrap();
 
-        let query_a = DnsPacket::query(200, "same.domain", QueryType::A);
-        let query_aaaa = DnsPacket::query(201, "same.domain", QueryType::AAAA);
+        // Fire A and AAAA concurrently — should NOT coalesce
+        let ctx_ref = ctx.clone();
+        let ctx_ref2 = ctx.clone();
+        let buf_a = build_wire_query(200, "different-qt.example.com", QueryType::A);
+        let buf_aaaa = build_wire_query(201, "different-qt.example.com", QueryType::AAAA);
 
-        let h1 = tokio::spawn(async move {
-            resolve_coalesced(
-                &inf1,
-                ("same.domain".to_string(), QueryType::A),
-                &query_a,
-                || async {
-                    count1.fetch_add(1, std::sync::atomic::Ordering::Relaxed);
-                    tokio::time::sleep(Duration::from_millis(100)).await;
-                    Ok(mock_response("same.domain"))
-                },
-            )
-            .await
-        });
-        let h2 = tokio::spawn(async move {
-            resolve_coalesced(
-                &inf2,
-                ("same.domain".to_string(), QueryType::AAAA),
-                &query_aaaa,
-                || async {
-                    count2.fetch_add(1, std::sync::atomic::Ordering::Relaxed);
-                    tokio::time::sleep(Duration::from_millis(100)).await;
-                    Ok(mock_response("same.domain"))
-                },
-            )
-            .await
-        });
+        let h1 = tokio::spawn(async move { handle_query(buf_a, src, &ctx_ref).await });
+        let h2 = tokio::spawn(async move { handle_query(buf_aaaa, src, &ctx_ref2).await });
 
-        let (_, path1, _) = h1.await.unwrap();
-        let (_, path2, _) = h2.await.unwrap();
+        h1.await.unwrap().unwrap();
+        h2.await.unwrap().unwrap();
 
-        let actual = resolve_count.load(std::sync::atomic::Ordering::Relaxed);
-        assert_eq!(actual, 2, "A and AAAA should each resolve, got {}", actual);
-        assert_eq!(path1, QueryPath::Recursive);
-        assert_eq!(path2, QueryPath::Recursive);
-
-        assert!(inflight.lock().unwrap().is_empty());
-    }
-
-    #[tokio::test]
-    async fn inflight_map_cleaned_after_error() {
-        let inflight: Mutex<InflightMap> = Mutex::new(HashMap::new());
-        let query = DnsPacket::query(300, "will-fail.test", QueryType::A);
-
-        let (_, path, _) = resolve_coalesced(
-            &inflight,
-            ("will-fail.test".to_string(), QueryType::A),
-            &query,
-            || async { Err::<DnsPacket, _>("upstream timeout".into()) },
-        )
-        .await;
-
-        assert_eq!(path, QueryPath::UpstreamError);
-        assert!(inflight.lock().unwrap().is_empty());
-    }
-
-    #[tokio::test]
-    async fn follower_gets_servfail_when_leader_fails() {
-        let inflight = Arc::new(Mutex::new(HashMap::new()));
-
-        let mut handles = Vec::new();
-        for i in 0..3u16 {
-            let inf = inflight.clone();
-            let query = DnsPacket::query(400 + i, "fail.test", QueryType::A);
-            handles.push(tokio::spawn(async move {
-                resolve_coalesced(
-                    &inf,
-                    ("fail.test".to_string(), QueryType::A),
-                    &query,
-                    || async {
-                        tokio::time::sleep(Duration::from_millis(200)).await;
-                        Err::<DnsPacket, _>("upstream error".into())
-                    },
-                )
-                .await
-            }));
-        }
-
-        let mut paths = Vec::new();
-        for h in handles {
-            let (resp, path, _) = h.await.unwrap();
-            assert_eq!(resp.header.rescode, ResultCode::SERVFAIL);
-            assert_eq!(
-                resp.questions.len(),
-                1,
-                "SERVFAIL must echo question section"
-            );
-            assert_eq!(resp.questions[0].name, "fail.test");
-            paths.push(path);
-        }
-
-        let errors = paths
-            .iter()
-            .filter(|p| **p == QueryPath::UpstreamError)
-            .count();
-        assert_eq!(errors, 3, "all 3 should be UpstreamError, got {}", errors);
-
-        assert!(inflight.lock().unwrap().is_empty());
-    }
-
-    #[tokio::test]
-    async fn servfail_leader_includes_question_section() {
-        let inflight: Mutex<InflightMap> = Mutex::new(HashMap::new());
-        let query = DnsPacket::query(500, "question.test", QueryType::A);
-
-        let (resp, _, _) = resolve_coalesced(
-            &inflight,
-            ("question.test".to_string(), QueryType::A),
-            &query,
-            || async { Err::<DnsPacket, _>("fail".into()) },
-        )
-        .await;
-
-        assert_eq!(resp.header.rescode, ResultCode::SERVFAIL);
-        assert_eq!(
-            resp.questions.len(),
-            1,
-            "SERVFAIL must echo question section"
+        let actual = query_count.load(std::sync::atomic::Ordering::Relaxed);
+        assert!(
+            actual >= 2,
+            "A and AAAA should resolve independently, got {}",
+            actual
         );
-        assert_eq!(resp.questions[0].name, "question.test");
-        assert_eq!(resp.questions[0].qtype, QueryType::A);
-        assert_eq!(resp.header.id, 500);
+        assert!(ctx.inflight.lock().unwrap().is_empty());
+
+        crate::recursive::reset_udp_state();
     }
 
     #[tokio::test]
-    async fn leader_error_preserves_message() {
-        let inflight: Mutex<InflightMap> = Mutex::new(HashMap::new());
-        let query = DnsPacket::query(700, "err-msg.test", QueryType::A);
+    async fn inflight_map_cleaned_after_upstream_error() {
+        // Server that rejects everything — no server running at all
+        let bogus_addr: SocketAddr = "127.0.0.1:1".parse().unwrap();
+        let ctx = test_recursive_ctx(bogus_addr).await;
+        let src: SocketAddr = "127.0.0.1:9999".parse().unwrap();
 
-        let (_, path, err) = resolve_coalesced(
-            &inflight,
-            ("err-msg.test".to_string(), QueryType::A),
-            &query,
-            || async { Err::<DnsPacket, _>("connection refused by upstream".into()) },
-        )
-        .await;
+        let buf = build_wire_query(300, "will-fail.example.com", QueryType::A);
+        let _ = handle_query(buf, src, &ctx).await;
 
-        assert_eq!(path, QueryPath::UpstreamError);
-        assert_eq!(
-            err.as_deref(),
-            Some("connection refused by upstream"),
-            "error message must be preserved for logging"
-        );
+        // Map must be clean even after error
+        assert!(ctx.inflight.lock().unwrap().is_empty());
     }
 }

src/dnssec.rs

@@ -5,7 +5,6 @@ use log::{debug, trace};
 use ring::digest;
 use ring::signature;
 
-use crate::buffer::BytePacketBuffer;
 use crate::cache::{DnsCache, DnssecStatus};
 use crate::packet::DnsPacket;
 use crate::question::QueryType;
@@ -721,29 +720,22 @@ pub fn verify_ds(ds: &DnsRecord, dnskey: &DnsRecord, owner: &str) -> bool {
 
 // -- Canonical wire format --
 
-/// Encode a DNS name in canonical wire form per RFC 4034 §6.2:
-/// uncompressed, with ASCII letters lowercased.
-///
-/// Lowercasing happens *after* escape resolution because `\065` yields
-/// `'A'`, which canonical form must convert to `'a'`.
 pub fn name_to_wire(name: &str) -> Vec<u8> {
-    let mut buf = BytePacketBuffer::new();
-    buf.write_qname(name)
-        .expect("name_to_wire: input must parse as a valid DNS name");
-    let mut wire = buf.filled().to_vec();
-
-    let mut i = 0;
-    while i < wire.len() {
-        let label_len = wire[i] as usize;
-        if label_len == 0 {
-            break;
-        }
-        i += 1;
-        let end = i + label_len;
-        wire[i..end].make_ascii_lowercase();
-        i = end;
+    let mut wire = Vec::with_capacity(name.len() + 2);
+    if name == "." || name.is_empty() {
+        wire.push(0);
+        return wire;
     }
-
+    for label in name.split('.') {
+        if label.is_empty() {
+            continue;
+        }
+        wire.push(label.len() as u8);
+        for &b in label.as_bytes() {
+            wire.push(b.to_ascii_lowercase());
+        }
+    }
+    wire.push(0);
     wire
 }
 
@@ -1483,23 +1475,6 @@ mod tests {
         );
     }
 
-    #[test]
-    fn name_to_wire_escaped_dot_in_label_is_not_a_separator() {
-        // `exa\.mple.com` is two labels: `exa.mple` (8 bytes including the 0x2E) and `com`.
-        let wire = name_to_wire("exa\\.mple.com");
-        assert_eq!(
-            wire,
-            vec![8, b'e', b'x', b'a', b'.', b'm', b'p', b'l', b'e', 3, b'c', b'o', b'm', 0]
-        );
-    }
-
-    #[test]
-    fn name_to_wire_decimal_escape_is_lowercased() {
-        // \065 = 'A', must become 'a' in canonical form.
-        let wire = name_to_wire("\\065bc.com");
-        assert_eq!(wire, vec![3, b'a', b'b', b'c', 3, b'c', b'o', b'm', 0]);
-    }
-
     #[test]
     fn parent_zone_cases() {
         assert_eq!(parent_zone("example.com"), "com");

src/dot.rs

@@ -1,542 +0,0 @@
-use std::net::{IpAddr, SocketAddr};
-use std::path::Path;
-use std::sync::Arc;
-use std::time::Duration;
-
-use log::{debug, error, info, warn};
-use rustls::ServerConfig;
-use tokio::io::{AsyncReadExt, AsyncWriteExt};
-use tokio::net::TcpListener;
-use tokio::sync::Semaphore;
-use tokio_rustls::TlsAcceptor;
-
-use crate::buffer::BytePacketBuffer;
-use crate::config::DotConfig;
-use crate::ctx::{resolve_query, ServerCtx};
-use crate::header::ResultCode;
-use crate::packet::DnsPacket;
-
-const MAX_CONNECTIONS: usize = 512;
-const IDLE_TIMEOUT: Duration = Duration::from_secs(30);
-const HANDSHAKE_TIMEOUT: Duration = Duration::from_secs(10);
-const WRITE_TIMEOUT: Duration = Duration::from_secs(10);
-// Matches BytePacketBuffer::BUF_SIZE — RFC 7858 allows up to 65535 but our
-// buffer would silently truncate anything larger.
-const MAX_MSG_LEN: usize = 4096;
-
-fn dot_alpn() -> Vec<Vec<u8>> {
-    vec![b"dot".to_vec()]
-}
-
-/// Build a TLS ServerConfig for DoT from user-provided cert/key PEM files.
-fn load_tls_config(cert_path: &Path, key_path: &Path) -> crate::Result<Arc<ServerConfig>> {
-    // rustls needs a CryptoProvider installed before ServerConfig::builder().
-    // The proxy's build_tls_config also does this; we repeat it here because
-    // running DoT with user-provided certs while the proxy is disabled would
-    // otherwise panic on first handshake (no default provider).
-    let _ = rustls::crypto::ring::default_provider().install_default();
-
-    let cert_pem = std::fs::read(cert_path)?;
-    let key_pem = std::fs::read(key_path)?;
-
-    let certs: Vec<_> = rustls_pemfile::certs(&mut &cert_pem[..]).collect::<Result<_, _>>()?;
-    let key = rustls_pemfile::private_key(&mut &key_pem[..])?
-        .ok_or("no private key found in key file")?;
-
-    let mut config = ServerConfig::builder()
-        .with_no_client_auth()
-        .with_single_cert(certs, key)?;
-    config.alpn_protocols = dot_alpn();
-
-    Ok(Arc::new(config))
-}
-
-/// Build a self-signed DoT TLS config. Can't reuse `ctx.tls_config` (the
-/// proxy's shared config) because DoT needs its own ALPN advertisement.
-///
-/// Pass `proxy_tld` itself as a service name so the cert gets an explicit
-/// `{tld}.{tld}` SAN (e.g. "numa.numa") matching the ServerName that
-/// setup-phone's mobileconfig sends as SNI. The `*.{tld}` wildcard alone
-/// is rejected by strict TLS clients under single-label TLDs (per the
-/// note in tls.rs::generate_service_cert).
-fn self_signed_tls(ctx: &ServerCtx) -> Option<Arc<ServerConfig>> {
-    let service_names = [ctx.proxy_tld.clone()];
-    match crate::tls::build_tls_config(&ctx.proxy_tld, &service_names, dot_alpn(), &ctx.data_dir) {
-        Ok(cfg) => Some(cfg),
-        Err(e) => {
-            warn!(
-                "DoT: failed to generate self-signed TLS: {} — DoT disabled",
-                e
-            );
-            None
-        }
-    }
-}
-
-/// Start the DNS-over-TLS listener (RFC 7858).
-pub async fn start_dot(ctx: Arc<ServerCtx>, config: &DotConfig) {
-    let tls_config = match (&config.cert_path, &config.key_path) {
-        (Some(cert), Some(key)) => match load_tls_config(cert, key) {
-            Ok(cfg) => cfg,
-            Err(e) => {
-                warn!("DoT: failed to load TLS cert/key: {} — DoT disabled", e);
-                return;
-            }
-        },
-        _ => match self_signed_tls(&ctx) {
-            Some(cfg) => cfg,
-            None => return,
-        },
-    };
-
-    let bind_addr: IpAddr = config
-        .bind_addr
-        .parse()
-        .unwrap_or(IpAddr::V4(std::net::Ipv4Addr::UNSPECIFIED));
-    let addr = SocketAddr::new(bind_addr, config.port);
-    let listener = match TcpListener::bind(addr).await {
-        Ok(l) => l,
-        Err(e) => {
-            warn!("DoT: could not bind {} ({}) — DoT disabled", addr, e);
-            return;
-        }
-    };
-    info!("DoT listening on {}", addr);
-
-    accept_loop(listener, TlsAcceptor::from(tls_config), ctx).await;
-}
-
-async fn accept_loop(listener: TcpListener, acceptor: TlsAcceptor, ctx: Arc<ServerCtx>) {
-    let semaphore = Arc::new(Semaphore::new(MAX_CONNECTIONS));
-
-    loop {
-        let (tcp_stream, remote_addr) = match listener.accept().await {
-            Ok(conn) => conn,
-            Err(e) => {
-                error!("DoT: TCP accept error: {}", e);
-                // Back off to avoid tight-looping on persistent failures (e.g. fd exhaustion).
-                tokio::time::sleep(Duration::from_millis(100)).await;
-                continue;
-            }
-        };
-
-        let permit = match semaphore.clone().try_acquire_owned() {
-            Ok(p) => p,
-            Err(_) => {
-                debug!("DoT: connection limit reached, rejecting {}", remote_addr);
-                continue;
-            }
-        };
-        let acceptor = acceptor.clone();
-        let ctx = Arc::clone(&ctx);
-
-        tokio::spawn(async move {
-            let _permit = permit; // held until task exits
-
-            let tls_stream =
-                match tokio::time::timeout(HANDSHAKE_TIMEOUT, acceptor.accept(tcp_stream)).await {
-                    Ok(Ok(s)) => s,
-                    Ok(Err(e)) => {
-                        debug!("DoT: TLS handshake failed from {}: {}", remote_addr, e);
-                        return;
-                    }
-                    Err(_) => {
-                        debug!("DoT: TLS handshake timeout from {}", remote_addr);
-                        return;
-                    }
-                };
-
-            handle_dot_connection(tls_stream, remote_addr, &ctx).await;
-        });
-    }
-}
-
-/// Handle a single persistent DoT connection (RFC 7858).
-/// Reads length-prefixed DNS queries until EOF, idle timeout, or error.
-async fn handle_dot_connection<S>(mut stream: S, remote_addr: SocketAddr, ctx: &ServerCtx)
-where
-    S: AsyncReadExt + AsyncWriteExt + Unpin,
-{
-    loop {
-        // Read 2-byte length prefix (RFC 1035 §4.2.2) with idle timeout
-        let mut len_buf = [0u8; 2];
-        let Ok(Ok(_)) = tokio::time::timeout(IDLE_TIMEOUT, stream.read_exact(&mut len_buf)).await
-        else {
-            break;
-        };
-        let msg_len = u16::from_be_bytes(len_buf) as usize;
-        if msg_len > MAX_MSG_LEN {
-            debug!("DoT: oversized message {} from {}", msg_len, remote_addr);
-            break;
-        }
-
-        let mut buffer = BytePacketBuffer::new();
-        let Ok(Ok(_)) =
-            tokio::time::timeout(IDLE_TIMEOUT, stream.read_exact(&mut buffer.buf[..msg_len])).await
-        else {
-            break;
-        };
-
-        // Parse query up-front so we can echo its question section in SERVFAIL
-        // responses when resolve_query fails.
-        let query = match DnsPacket::from_buffer(&mut buffer) {
-            Ok(q) => q,
-            Err(e) => {
-                warn!("{} | PARSE ERROR | {}", remote_addr, e);
-                // BytePacketBuffer is zero-initialized, so buf[0..2] reads as 0x0000
-                // for sub-2-byte messages — harmless FORMERR with id=0.
-                let query_id = u16::from_be_bytes([buffer.buf[0], buffer.buf[1]]);
-                let mut resp = DnsPacket::new();
-                resp.header.id = query_id;
-                resp.header.response = true;
-                resp.header.rescode = ResultCode::FORMERR;
-                if send_response(&mut stream, &resp, remote_addr)
-                    .await
-                    .is_err()
-                {
-                    break;
-                }
-                continue;
-            }
-        };
-
-        match resolve_query(query.clone(), remote_addr, ctx).await {
-            Ok(resp_buffer) => {
-                if write_framed(&mut stream, resp_buffer.filled())
-                    .await
-                    .is_err()
-                {
-                    break;
-                }
-            }
-            Err(e) => {
-                warn!("{} | RESOLVE ERROR | {}", remote_addr, e);
-                // SERVFAIL that echoes the original question section.
-                let resp = DnsPacket::response_from(&query, ResultCode::SERVFAIL);
-                if send_response(&mut stream, &resp, remote_addr)
-                    .await
-                    .is_err()
-                {
-                    break;
-                }
-            }
-        }
-    }
-}
-
-/// Serialize a DNS response and send it framed. Logs serialization failures
-/// and returns Err so the caller can tear down the connection.
-async fn send_response<S>(
-    stream: &mut S,
-    resp: &DnsPacket,
-    remote_addr: SocketAddr,
-) -> std::io::Result<()>
-where
-    S: AsyncWriteExt + Unpin,
-{
-    let mut out_buf = BytePacketBuffer::new();
-    if resp.write(&mut out_buf).is_err() {
-        debug!(
-            "DoT: failed to serialize {:?} response for {}",
-            resp.header.rescode, remote_addr
-        );
-        return Err(std::io::Error::other("serialize failed"));
-    }
-    write_framed(stream, out_buf.filled()).await
-}
-
-/// Write a DNS message with its 2-byte length prefix, coalesced into one syscall.
-/// Bounded by WRITE_TIMEOUT so a stalled reader can't indefinitely hold a worker.
-async fn write_framed<S>(stream: &mut S, msg: &[u8]) -> std::io::Result<()>
-where
-    S: AsyncWriteExt + Unpin,
-{
-    let mut out = Vec::with_capacity(2 + msg.len());
-    out.extend_from_slice(&(msg.len() as u16).to_be_bytes());
-    out.extend_from_slice(msg);
-    match tokio::time::timeout(WRITE_TIMEOUT, async {
-        stream.write_all(&out).await?;
-        stream.flush().await
-    })
-    .await
-    {
-        Ok(result) => result,
-        Err(_) => Err(std::io::Error::other("write timeout")),
-    }
-}
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-    use std::collections::HashMap;
-    use std::sync::{Mutex, RwLock};
-
-    use rcgen::{CertificateParams, DnType, KeyPair};
-    use rustls::pki_types::{CertificateDer, PrivateKeyDer, PrivatePkcs8KeyDer, ServerName};
-    use tokio::io::{AsyncReadExt, AsyncWriteExt};
-
-    use crate::buffer::BytePacketBuffer;
-    use crate::header::ResultCode;
-    use crate::packet::DnsPacket;
-    use crate::question::QueryType;
-    use crate::record::DnsRecord;
-
-    /// Generate a self-signed DoT server config and return its leaf cert DER
-    /// so callers can build matching client configs with arbitrary ALPN.
-    fn test_tls_configs() -> (Arc<ServerConfig>, CertificateDer<'static>) {
-        let _ = rustls::crypto::ring::default_provider().install_default();
-
-        // Mirror production self_signed_tls SAN shape: *.numa wildcard plus
-        // explicit numa.numa apex (the ServerName setup-phone uses as SNI).
-        let key_pair = KeyPair::generate().unwrap();
-        let mut params = CertificateParams::default();
-        params
-            .distinguished_name
-            .push(DnType::CommonName, "Numa .numa services");
-        params.subject_alt_names = vec![
-            rcgen::SanType::DnsName("*.numa".try_into().unwrap()),
-            rcgen::SanType::DnsName("numa.numa".try_into().unwrap()),
-        ];
-        let cert = params.self_signed(&key_pair).unwrap();
-
-        let cert_der = CertificateDer::from(cert.der().to_vec());
-        let key_der = PrivateKeyDer::Pkcs8(PrivatePkcs8KeyDer::from(key_pair.serialize_der()));
-
-        let mut server_config = ServerConfig::builder()
-            .with_no_client_auth()
-            .with_single_cert(vec![cert_der.clone()], key_der)
-            .unwrap();
-        server_config.alpn_protocols = dot_alpn();
-
-        (Arc::new(server_config), cert_der)
-    }
-
-    /// Build a TLS client config that trusts `cert_der` and advertises the
-    /// given ALPN protocols. Used by tests to vary ALPN per test case.
-    fn dot_client(
-        cert_der: &CertificateDer<'static>,
-        alpn: Vec<Vec<u8>>,
-    ) -> Arc<rustls::ClientConfig> {
-        let mut root_store = rustls::RootCertStore::empty();
-        root_store.add(cert_der.clone()).unwrap();
-        let mut config = rustls::ClientConfig::builder()
-            .with_root_certificates(root_store)
-            .with_no_client_auth();
-        config.alpn_protocols = alpn;
-        Arc::new(config)
-    }
-
-    /// Spin up a DoT listener with a test TLS config. Returns the bind addr
-    /// and the leaf cert DER so callers can build clients with arbitrary ALPN.
-    /// The upstream is pointed at a bound-but-unresponsive UDP socket we own, so
-    /// any query that escapes to the upstream path times out deterministically
-    /// (SERVFAIL) regardless of what the host has running on port 53.
-    async fn spawn_dot_server() -> (SocketAddr, CertificateDer<'static>) {
-        let (server_tls, cert_der) = test_tls_configs();
-
-        let socket = tokio::net::UdpSocket::bind("127.0.0.1:0").await.unwrap();
-        // Bind an unresponsive upstream and leak it so it lives for the test duration.
-        let blackhole = Box::leak(Box::new(std::net::UdpSocket::bind("127.0.0.1:0").unwrap()));
-        let upstream_addr = blackhole.local_addr().unwrap();
-        let ctx = Arc::new(ServerCtx {
-            socket,
-            zone_map: {
-                let mut m = HashMap::new();
-                let mut inner = HashMap::new();
-                inner.insert(
-                    QueryType::A,
-                    vec![DnsRecord::A {
-                        domain: "dot-test.example".to_string(),
-                        addr: std::net::Ipv4Addr::new(10, 0, 0, 1),
-                        ttl: 300,
-                    }],
-                );
-                m.insert("dot-test.example".to_string(), inner);
-                m
-            },
-            cache: RwLock::new(crate::cache::DnsCache::new(100, 60, 86400)),
-            stats: Mutex::new(crate::stats::ServerStats::new()),
-            overrides: RwLock::new(crate::override_store::OverrideStore::new()),
-            blocklist: RwLock::new(crate::blocklist::BlocklistStore::new()),
-            query_log: Mutex::new(crate::query_log::QueryLog::new(100)),
-            services: Mutex::new(crate::service_store::ServiceStore::new()),
-            lan_peers: Mutex::new(crate::lan::PeerStore::new(90)),
-            forwarding_rules: Vec::new(),
-            upstream: Mutex::new(crate::forward::Upstream::Udp(upstream_addr)),
-            upstream_auto: false,
-            upstream_port: 53,
-            lan_ip: Mutex::new(std::net::Ipv4Addr::LOCALHOST),
-            timeout: Duration::from_millis(200),
-            proxy_tld: "numa".to_string(),
-            proxy_tld_suffix: ".numa".to_string(),
-            lan_enabled: false,
-            config_path: String::new(),
-            config_found: false,
-            config_dir: std::path::PathBuf::from("/tmp"),
-            data_dir: std::path::PathBuf::from("/tmp"),
-            tls_config: Some(arc_swap::ArcSwap::from(server_tls)),
-            upstream_mode: crate::config::UpstreamMode::Forward,
-            root_hints: Vec::new(),
-            srtt: RwLock::new(crate::srtt::SrttCache::new(true)),
-            inflight: Mutex::new(HashMap::new()),
-            dnssec_enabled: false,
-            dnssec_strict: false,
-        });
-
-        let listener = TcpListener::bind("127.0.0.1:0").await.unwrap();
-        let addr = listener.local_addr().unwrap();
-
-        let tls_config = Arc::clone(&*ctx.tls_config.as_ref().unwrap().load());
-        let acceptor = TlsAcceptor::from(tls_config);
-
-        tokio::spawn(accept_loop(listener, acceptor, ctx));
-
-        (addr, cert_der)
-    }
-
-    /// Open a TLS connection to the DoT server and return the stream.
-    /// Uses SNI "numa.numa" to mirror what setup-phone's mobileconfig sends.
-    async fn dot_connect(
-        addr: SocketAddr,
-        client_config: &Arc<rustls::ClientConfig>,
-    ) -> tokio_rustls::client::TlsStream<tokio::net::TcpStream> {
-        let connector = tokio_rustls::TlsConnector::from(Arc::clone(client_config));
-        let tcp = tokio::net::TcpStream::connect(addr).await.unwrap();
-        connector
-            .connect(ServerName::try_from("numa.numa").unwrap(), tcp)
-            .await
-            .unwrap()
-    }
-
-    /// Send a DNS query over a DoT stream and read the response.
-    async fn dot_exchange(
-        stream: &mut tokio_rustls::client::TlsStream<tokio::net::TcpStream>,
-        query: &DnsPacket,
-    ) -> DnsPacket {
-        let mut buf = BytePacketBuffer::new();
-        query.write(&mut buf).unwrap();
-        let msg = buf.filled();
-
-        let mut out = Vec::with_capacity(2 + msg.len());
-        out.extend_from_slice(&(msg.len() as u16).to_be_bytes());
-        out.extend_from_slice(msg);
-        stream.write_all(&out).await.unwrap();
-
-        let mut len_buf = [0u8; 2];
-        stream.read_exact(&mut len_buf).await.unwrap();
-        let resp_len = u16::from_be_bytes(len_buf) as usize;
-
-        let mut data = vec![0u8; resp_len];
-        stream.read_exact(&mut data).await.unwrap();
-
-        let mut resp_buf = BytePacketBuffer::from_bytes(&data);
-        DnsPacket::from_buffer(&mut resp_buf).unwrap()
-    }
-
-    #[tokio::test]
-    async fn dot_resolves_local_zone() {
-        let (addr, cert_der) = spawn_dot_server().await;
-        let client_config = dot_client(&cert_der, dot_alpn());
-        let mut stream = dot_connect(addr, &client_config).await;
-
-        let query = DnsPacket::query(0x1234, "dot-test.example", QueryType::A);
-        let resp = dot_exchange(&mut stream, &query).await;
-
-        assert_eq!(resp.header.id, 0x1234);
-        assert!(resp.header.response);
-        assert_eq!(resp.header.rescode, ResultCode::NOERROR);
-        assert_eq!(resp.answers.len(), 1);
-        match &resp.answers[0] {
-            DnsRecord::A { domain, addr, ttl } => {
-                assert_eq!(domain, "dot-test.example");
-                assert_eq!(*addr, std::net::Ipv4Addr::new(10, 0, 0, 1));
-                assert_eq!(*ttl, 300);
-            }
-            other => panic!("expected A record, got {:?}", other),
-        }
-    }
-
-    #[tokio::test]
-    async fn dot_multiple_queries_on_persistent_connection() {
-        let (addr, cert_der) = spawn_dot_server().await;
-        let client_config = dot_client(&cert_der, dot_alpn());
-        let mut stream = dot_connect(addr, &client_config).await;
-
-        for i in 0..3u16 {
-            let query = DnsPacket::query(0xA000 + i, "dot-test.example", QueryType::A);
-            let resp = dot_exchange(&mut stream, &query).await;
-            assert_eq!(resp.header.id, 0xA000 + i);
-            assert_eq!(resp.header.rescode, ResultCode::NOERROR);
-            assert_eq!(resp.answers.len(), 1);
-        }
-    }
-
-    #[tokio::test]
-    async fn dot_nxdomain_for_unknown() {
-        let (addr, cert_der) = spawn_dot_server().await;
-        let client_config = dot_client(&cert_der, dot_alpn());
-        let mut stream = dot_connect(addr, &client_config).await;
-
-        let query = DnsPacket::query(0xBEEF, "nonexistent.test", QueryType::A);
-        let resp = dot_exchange(&mut stream, &query).await;
-
-        assert_eq!(resp.header.id, 0xBEEF);
-        assert!(resp.header.response);
-        // Query goes to the blackhole upstream which never replies → SERVFAIL.
-        // The SERVFAIL response echoes the question section.
-        assert_eq!(resp.header.rescode, ResultCode::SERVFAIL);
-        assert_eq!(resp.questions.len(), 1);
-        assert_eq!(resp.questions[0].name, "nonexistent.test");
-    }
-
-    #[tokio::test]
-    async fn dot_negotiates_alpn() {
-        let (addr, cert_der) = spawn_dot_server().await;
-        let client_config = dot_client(&cert_der, dot_alpn());
-        let stream = dot_connect(addr, &client_config).await;
-        let (_io, conn) = stream.get_ref();
-        assert_eq!(conn.alpn_protocol(), Some(&b"dot"[..]));
-    }
-
-    #[tokio::test]
-    async fn dot_rejects_non_dot_alpn() {
-        // Cross-protocol confusion defense: a client that only offers "h2"
-        // (e.g. an HTTP/2 client mistakenly hitting :853) must not complete
-        // a TLS handshake with the DoT server. Verifies the rustls server
-        // sends `no_application_protocol` rather than silently negotiating.
-        let (addr, cert_der) = spawn_dot_server().await;
-        let client_config = dot_client(&cert_der, vec![b"h2".to_vec()]);
-        let connector = tokio_rustls::TlsConnector::from(client_config);
-        let tcp = tokio::net::TcpStream::connect(addr).await.unwrap();
-        let result = connector
-            .connect(ServerName::try_from("numa.numa").unwrap(), tcp)
-            .await;
-        assert!(
-            result.is_err(),
-            "DoT server must reject ALPN that doesn't include \"dot\""
-        );
-    }
-
-    #[tokio::test]
-    async fn dot_concurrent_connections() {
-        let (addr, cert_der) = spawn_dot_server().await;
-        let client_config = dot_client(&cert_der, dot_alpn());
-
-        let mut handles = Vec::new();
-        for i in 0..5u16 {
-            let cfg = Arc::clone(&client_config);
-            handles.push(tokio::spawn(async move {
-                let mut stream = dot_connect(addr, &cfg).await;
-                let query = DnsPacket::query(0xC000 + i, "dot-test.example", QueryType::A);
-                let resp = dot_exchange(&mut stream, &query).await;
-                assert_eq!(resp.header.id, 0xC000 + i);
-                assert_eq!(resp.header.rescode, ResultCode::NOERROR);
-                assert_eq!(resp.answers.len(), 1);
-            }));
-        }
-
-        for h in handles {
-            h.await.unwrap();
-        }
-    }
-}

src/forward.rs

@@ -141,7 +141,7 @@ mod tests {
     use std::future::IntoFuture;
 
     use crate::header::ResultCode;
-    use crate::question::QueryType;
+    use crate::question::{DnsQuestion, QueryType};
     use crate::record::DnsRecord;
 
     #[test]
@@ -160,7 +160,12 @@ mod tests {
     }
 
     fn make_query() -> DnsPacket {
-        DnsPacket::query(0xABCD, "example.com", QueryType::A)
+        let mut q = DnsPacket::new();
+        q.header.id = 0xABCD;
+        q.header.recursion_desired = true;
+        q.questions
+            .push(DnsQuestion::new("example.com".to_string(), QueryType::A));
+        q
     }
 
     fn make_response(query: &DnsPacket) -> DnsPacket {

src/lib.rs

@@ -5,7 +5,6 @@ pub mod cache;
 pub mod config;
 pub mod ctx;
 pub mod dnssec;
-pub mod dot;
 pub mod forward;
 pub mod header;
 pub mod lan;
@@ -26,10 +25,7 @@ pub type Error = Box<dyn std::error::Error + Send + Sync>;
 pub type Result<T> = std::result::Result<T, Error>;
 
 /// Shared config directory for persistent data (services.json, etc).
-/// Unix users: ~/.config/numa/
-/// Linux root daemon: /var/lib/numa (FHS) — falls back to /usr/local/var/numa
-///                    if a pre-v0.10.1 install already lives there.
-/// macOS root daemon: /usr/local/var/numa (Homebrew prefix)
+/// Unix: ~/.config/numa/ (or /usr/local/var/numa/ when running as root daemon)
 /// Windows: %APPDATA%\numa
 pub fn config_dir() -> std::path::PathBuf {
     #[cfg(windows)]
@@ -66,15 +62,11 @@ fn config_dir_unix() -> std::path::PathBuf {
     }
 
     // Running as root daemon (launchd/systemd) — use system-wide path
-    daemon_data_dir()
+    std::path::PathBuf::from("/usr/local/var/numa")
 }
 
-/// Default system-wide data directory for TLS certs. Overridable via
-/// `[server] data_dir = "..."` in numa.toml — this function only provides
-/// the fallback when the config doesn't set it.
-/// Linux: /var/lib/numa (FHS) — falls back to /usr/local/var/numa if a
-///        pre-v0.10.1 install already has data there.
-/// macOS: /usr/local/var/numa (Homebrew prefix)
+/// System-wide data directory for TLS certs.
+/// Unix: /usr/local/var/numa
 /// Windows: %PROGRAMDATA%\numa
 pub fn data_dir() -> std::path::PathBuf {
     #[cfg(windows)]
@@ -86,62 +78,6 @@ pub fn data_dir() -> std::path::PathBuf {
     }
     #[cfg(not(windows))]
     {
-        daemon_data_dir()
-    }
-}
-
-/// Resolve the system-wide data directory for the running platform.
-/// Honors backwards compatibility with pre-v0.10.1 installs that still
-/// have their CA cert + services.json under `/usr/local/var/numa`.
-#[cfg(not(windows))]
-fn daemon_data_dir() -> std::path::PathBuf {
-    #[cfg(target_os = "linux")]
-    {
-        std::path::PathBuf::from(resolve_linux_data_dir(
-            std::path::Path::new("/usr/local/var/numa").exists(),
-            std::path::Path::new("/var/lib/numa").exists(),
-        ))
-    }
-    #[cfg(target_os = "macos")]
-    {
-        // macOS uses the Homebrew prefix convention; no FHS migration needed.
         std::path::PathBuf::from("/usr/local/var/numa")
     }
 }
-
-/// Extracted as a pure function so the migration logic is unit-testable
-/// without touching the real filesystem.
-#[cfg(any(target_os = "linux", test))]
-fn resolve_linux_data_dir(legacy_exists: bool, fhs_exists: bool) -> &'static str {
-    if legacy_exists && !fhs_exists {
-        "/usr/local/var/numa"
-    } else {
-        "/var/lib/numa"
-    }
-}
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-
-    #[test]
-    fn linux_data_dir_fresh_install_uses_fhs() {
-        assert_eq!(resolve_linux_data_dir(false, false), "/var/lib/numa");
-    }
-
-    #[test]
-    fn linux_data_dir_upgrading_install_keeps_legacy() {
-        // Migration must keep legacy so the user doesn't lose their CA on upgrade.
-        assert_eq!(resolve_linux_data_dir(true, false), "/usr/local/var/numa");
-    }
-
-    #[test]
-    fn linux_data_dir_after_migration_uses_fhs() {
-        assert_eq!(resolve_linux_data_dir(true, true), "/var/lib/numa");
-    }
-
-    #[test]
-    fn linux_data_dir_only_fhs_uses_fhs() {
-        assert_eq!(resolve_linux_data_dir(false, true), "/var/lib/numa");
-    }
-}

src/main.rs

@@ -17,12 +17,10 @@ use numa::query_log::QueryLog;
 use numa::service_store::ServiceStore;
 use numa::stats::ServerStats;
 use numa::system_dns::{
-    discover_system_dns, install_service, restart_service, service_status, uninstall_service,
+    discover_system_dns, install_service, install_system_dns, restart_service, service_status,
+    uninstall_service, uninstall_system_dns,
 };
 
-const QUAD9_IP: &str = "9.9.9.9";
-const DOH_FALLBACK: &str = "https://9.9.9.9/dns-query";
-
 #[tokio::main]
 async fn main() -> numa::Result<()> {
     env_logger::Builder::from_env(env_logger::Env::default().default_filter_or("info"))
@@ -33,12 +31,12 @@ async fn main() -> numa::Result<()> {
     let arg1 = std::env::args().nth(1).unwrap_or_default();
     match arg1.as_str() {
         "install" => {
-            eprintln!("\x1b[1;38;2;192;98;58mNuma\x1b[0m — installing\n");
-            return install_service().map_err(|e| e.into());
+            eprintln!("\x1b[1;38;2;192;98;58mNuma\x1b[0m — configuring system DNS\n");
+            return install_system_dns().map_err(|e| e.into());
         }
         "uninstall" => {
-            eprintln!("\x1b[1;38;2;192;98;58mNuma\x1b[0m — uninstalling\n");
-            return uninstall_service().map_err(|e| e.into());
+            eprintln!("\x1b[1;38;2;192;98;58mNuma\x1b[0m — restoring system DNS\n");
+            return uninstall_system_dns().map_err(|e| e.into());
         }
         "service" => {
             let sub = std::env::args().nth(2).unwrap_or_default();
@@ -109,81 +107,32 @@ async fn main() -> numa::Result<()> {
     // Discover system DNS in a single pass (upstream + forwarding rules)
     let system_dns = discover_system_dns();
 
-    let root_hints = numa::recursive::parse_root_hints(&config.upstream.root_hints);
-
-    let (resolved_mode, upstream_auto, upstream, upstream_label) = match config.upstream.mode {
-        numa::config::UpstreamMode::Auto => {
-            info!("auto mode: probing recursive resolution...");
-            if numa::recursive::probe_recursive(&root_hints).await {
-                info!("recursive probe succeeded — self-sovereign mode");
-                let dummy = Upstream::Udp("0.0.0.0:0".parse().unwrap());
-                (
-                    numa::config::UpstreamMode::Recursive,
-                    false,
-                    dummy,
-                    "recursive (root hints)".to_string(),
-                )
-            } else {
-                log::warn!("recursive probe failed — falling back to Quad9 DoH");
-                let client = reqwest::Client::builder()
-                    .use_rustls_tls()
-                    .build()
-                    .unwrap_or_default();
-                let url = DOH_FALLBACK.to_string();
-                let label = url.clone();
-                (
-                    numa::config::UpstreamMode::Forward,
-                    false,
-                    Upstream::Doh { url, client },
-                    label,
-                )
-            }
-        }
-        numa::config::UpstreamMode::Recursive => {
-            let dummy = Upstream::Udp("0.0.0.0:0".parse().unwrap());
-            (
-                numa::config::UpstreamMode::Recursive,
-                false,
-                dummy,
-                "recursive (root hints)".to_string(),
-            )
-        }
-        numa::config::UpstreamMode::Forward => {
-            let upstream_addr = if config.upstream.address.is_empty() {
-                system_dns
-                    .default_upstream
-                    .or_else(numa::system_dns::detect_dhcp_dns)
-                    .unwrap_or_else(|| {
-                        info!("could not detect system DNS, falling back to Quad9 DoH");
-                        DOH_FALLBACK.to_string()
-                    })
-            } else {
-                config.upstream.address.clone()
-            };
-
-            let upstream: Upstream = if upstream_addr.starts_with("https://") {
-                let client = reqwest::Client::builder()
-                    .use_rustls_tls()
-                    .build()
-                    .unwrap_or_default();
-                Upstream::Doh {
-                    url: upstream_addr,
-                    client,
-                }
-            } else {
-                let addr: SocketAddr =
-                    format!("{}:{}", upstream_addr, config.upstream.port).parse()?;
-                Upstream::Udp(addr)
-            };
-            let label = upstream.to_string();
-            (
-                numa::config::UpstreamMode::Forward,
-                config.upstream.address.is_empty(),
-                upstream,
-                label,
-            )
-        }
+    let upstream_addr = if config.upstream.address.is_empty() {
+        system_dns
+            .default_upstream
+            .or_else(numa::system_dns::detect_dhcp_dns)
+            .unwrap_or_else(|| {
+                info!("could not detect system DNS, falling back to Quad9 DoH");
+                "https://dns.quad9.net/dns-query".to_string()
+            })
+    } else {
+        config.upstream.address.clone()
     };
+
+    let upstream: Upstream = if upstream_addr.starts_with("https://") {
+        let client = reqwest::Client::builder()
+            .use_rustls_tls()
+            .build()
+            .unwrap_or_default();
+        Upstream::Doh {
+            url: upstream_addr,
+            client,
+        }
+    } else {
+        let addr: SocketAddr = format!("{}:{}", upstream_addr, config.upstream.port).parse()?;
+        Upstream::Udp(addr)
+    };
+    let upstream_label = upstream.to_string();
     let api_port = config.server.api_port;
 
     let mut blocklist = BlocklistStore::new();
@@ -204,30 +153,13 @@ async fn main() -> numa::Result<()> {
 
     let forwarding_rules = system_dns.forwarding_rules;
 
-    // Resolve data_dir from config, falling back to the platform default.
-    // Used for TLS CA storage below and stored on ServerCtx for runtime use.
-    let resolved_data_dir = config
-        .server
-        .data_dir
-        .clone()
-        .unwrap_or_else(numa::data_dir);
-
     // Build initial TLS config before ServerCtx (so ArcSwap is ready at construction)
     let initial_tls = if config.proxy.enabled && config.proxy.tls_port > 0 {
         let service_names = service_store.names();
-        match numa::tls::build_tls_config(
-            &config.proxy.tld,
-            &service_names,
-            Vec::new(),
-            &resolved_data_dir,
-        ) {
+        match numa::tls::build_tls_config(&config.proxy.tld, &service_names) {
             Ok(tls_config) => Some(ArcSwap::from(tls_config)),
             Err(e) => {
-                if let Some(advisory) = numa::tls::try_data_dir_advisory(&e, &resolved_data_dir) {
-                    eprint!("{}", advisory);
-                } else {
-                    log::warn!("TLS setup failed, HTTPS proxy disabled: {}", e);
-                }
+                log::warn!("TLS setup failed, HTTPS proxy disabled: {}", e);
                 None
             }
         }
@@ -235,21 +167,8 @@ async fn main() -> numa::Result<()> {
         None
     };
 
-    let socket = match UdpSocket::bind(&config.server.bind_addr).await {
-        Ok(s) => s,
-        Err(e) => {
-            if let Some(advisory) =
-                numa::system_dns::try_port53_advisory(&config.server.bind_addr, &e)
-            {
-                eprint!("{}", advisory);
-                std::process::exit(1);
-            }
-            return Err(e.into());
-        }
-    };
-
     let ctx = Arc::new(ServerCtx {
-        socket,
+        socket: UdpSocket::bind(&config.server.bind_addr).await?,
         zone_map: build_zone_map(&config.zones)?,
         cache: RwLock::new(DnsCache::new(
             config.cache.max_entries,
@@ -264,7 +183,7 @@ async fn main() -> numa::Result<()> {
         lan_peers: Mutex::new(numa::lan::PeerStore::new(config.lan.peer_timeout_secs)),
         forwarding_rules,
         upstream: Mutex::new(upstream),
-        upstream_auto,
+        upstream_auto: config.upstream.address.is_empty(),
         upstream_port: config.upstream.port,
         lan_ip: Mutex::new(numa::lan::detect_lan_ip().unwrap_or(std::net::Ipv4Addr::LOCALHOST)),
         timeout: Duration::from_millis(config.upstream.timeout_ms),
@@ -278,10 +197,10 @@ async fn main() -> numa::Result<()> {
         config_path: resolved_config_path,
         config_found,
         config_dir: numa::config_dir(),
-        data_dir: resolved_data_dir,
+        data_dir: numa::data_dir(),
         tls_config: initial_tls,
-        upstream_mode: resolved_mode,
-        root_hints,
+        upstream_mode: config.upstream.mode,
+        root_hints: numa::recursive::parse_root_hints(&config.upstream.root_hints),
         srtt: std::sync::RwLock::new(numa::srtt::SrttCache::new(config.upstream.srtt)),
         inflight: std::sync::Mutex::new(std::collections::HashMap::new()),
         dnssec_enabled: config.dnssec.enabled,
@@ -289,6 +208,7 @@ async fn main() -> numa::Result<()> {
     });
 
     let zone_count: usize = ctx.zone_map.values().map(|m| m.len()).sum();
+
     // Build banner rows, then size the box to fit the longest value
     let api_url = format!("http://localhost:{}", api_port);
     let proxy_label = if config.proxy.enabled {
@@ -388,20 +308,6 @@ async fn main() -> numa::Result<()> {
     );
     if let Some(ref label) = proxy_label {
         row("Proxy", g, label);
-        if config.proxy.bind_addr == "127.0.0.1" {
-            let y = "\x1b[38;2;204;176;59m"; // yellow
-            row(
-                "",
-                y,
-                &format!(
-                    "⚠ proxy on 127.0.0.1 — .{} not LAN reachable",
-                    config.proxy.tld
-                ),
-            );
-        }
-    }
-    if config.dot.enabled {
-        row("DoT", g, &format!("tls://:{}", config.dot.port));
     }
     if config.lan.enabled {
         row("LAN", g, "mDNS (_numa._tcp.local)");
@@ -469,11 +375,16 @@ async fn main() -> numa::Result<()> {
         axum::serve(listener, app).await.unwrap();
     });
 
-    let proxy_bind: std::net::Ipv4Addr = config
-        .proxy
-        .bind_addr
-        .parse()
-        .unwrap_or(std::net::Ipv4Addr::LOCALHOST);
+    // Proxy binds 0.0.0.0 when LAN is enabled (cross-machine access), otherwise config value
+    let proxy_bind: std::net::Ipv4Addr = if config.lan.enabled {
+        std::net::Ipv4Addr::UNSPECIFIED
+    } else {
+        config
+            .proxy
+            .bind_addr
+            .parse()
+            .unwrap_or(std::net::Ipv4Addr::LOCALHOST)
+    };
 
     // Spawn HTTP reverse proxy for .numa domains
     if config.proxy.enabled {
@@ -510,27 +421,11 @@ async fn main() -> numa::Result<()> {
         });
     }
 
-    // Spawn DNS-over-TLS listener (RFC 7858)
-    if config.dot.enabled {
-        let dot_ctx = Arc::clone(&ctx);
-        let dot_config = config.dot.clone();
-        tokio::spawn(async move {
-            numa::dot::start_dot(dot_ctx, &dot_config).await;
-        });
-    }
-
     // UDP DNS listener
     #[allow(clippy::infinite_loop)]
     loop {
         let mut buffer = BytePacketBuffer::new();
-        let (_, src_addr) = match ctx.socket.recv_from(&mut buffer.buf).await {
-            Ok(r) => r,
-            Err(e) if e.kind() == std::io::ErrorKind::ConnectionReset => {
-                // Windows delivers ICMP port-unreachable as ConnectionReset on UDP sockets
-                continue;
-            }
-            Err(e) => return Err(e.into()),
-        };
+        let (_, src_addr) = ctx.socket.recv_from(&mut buffer.buf).await?;
 
         let ctx = Arc::clone(&ctx);
         tokio::spawn(async move {
@@ -573,7 +468,7 @@ async fn network_watch_loop(ctx: Arc<numa::ctx::ServerCtx>) {
             let new_addr = dns_info
                 .default_upstream
                 .or_else(numa::system_dns::detect_dhcp_dns)
-                .unwrap_or_else(|| QUAD9_IP.to_string());
+                .unwrap_or_else(|| "9.9.9.9".to_string());
             if let Ok(new_sock) =
                 format!("{}:{}", new_addr, ctx.upstream_port).parse::<SocketAddr>()
             {

src/override_store.rs

@@ -117,22 +117,6 @@ impl OverrideStore {
         self.entries.clear();
     }
 
-    pub fn heap_bytes(&self) -> usize {
-        let per_slot = std::mem::size_of::<u64>()
-            + std::mem::size_of::<String>()
-            + std::mem::size_of::<OverrideEntry>()
-            + 1;
-        let table = self.entries.capacity() * per_slot;
-        let heap: usize = self
-            .entries
-            .iter()
-            .map(|(k, v)| {
-                k.capacity() + v.domain.capacity() + v.target.capacity() + v.record.heap_bytes()
-            })
-            .sum();
-        table + heap
-    }
-
     pub fn active_count(&self) -> usize {
         self.entries.values().filter(|e| !e.is_expired()).count()
     }
@@ -170,16 +154,3 @@ fn parse_target(domain: &str, target: &str, ttl: u32) -> Result<(QueryType, DnsR
         },
     ))
 }
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-
-    #[test]
-    fn heap_bytes_grows_with_entries() {
-        let mut store = OverrideStore::new();
-        let empty = store.heap_bytes();
-        store.insert("example.com", "1.2.3.4", 300, None).unwrap();
-        assert!(store.heap_bytes() > empty);
-    }
-}

src/packet.rs

@@ -57,34 +57,6 @@ impl DnsPacket {
         }
     }
 
-    pub fn query(id: u16, domain: &str, qtype: crate::question::QueryType) -> DnsPacket {
-        let mut pkt = DnsPacket::new();
-        pkt.header.id = id;
-        pkt.header.recursion_desired = true;
-        pkt.questions
-            .push(crate::question::DnsQuestion::new(domain.to_string(), qtype));
-        pkt
-    }
-
-    pub fn heap_bytes(&self) -> usize {
-        fn records_heap(records: &[DnsRecord]) -> usize {
-            records
-                .iter()
-                .map(|r| std::mem::size_of::<DnsRecord>() + r.heap_bytes())
-                .sum::<usize>()
-        }
-        let questions: usize = self
-            .questions
-            .iter()
-            .map(|q| std::mem::size_of::<DnsQuestion>() + q.name.capacity())
-            .sum();
-        questions
-            + records_heap(&self.answers)
-            + records_heap(&self.authorities)
-            + records_heap(&self.resources)
-            + self.edns.as_ref().map_or(0, |e| e.options.capacity())
-    }
-
     pub fn response_from(query: &DnsPacket, rescode: crate::header::ResultCode) -> DnsPacket {
         let mut resp = DnsPacket::new();
         resp.header.id = query.header.id;
@@ -610,16 +582,4 @@ mod tests {
             panic!("expected DNSKEY");
         }
     }
-
-    #[test]
-    fn heap_bytes_accounts_for_records() {
-        let mut pkt = DnsPacket::new();
-        let empty = pkt.heap_bytes();
-        pkt.answers.push(DnsRecord::A {
-            domain: "example.com".into(),
-            addr: "1.2.3.4".parse().unwrap(),
-            ttl: 300,
-        });
-        assert!(pkt.heap_bytes() > empty);
-    }
 }

src/query_log.rs

@@ -38,21 +38,6 @@ impl QueryLog {
         self.entries.push_back(entry);
     }
 
-    pub fn len(&self) -> usize {
-        self.entries.len()
-    }
-
-    pub fn is_empty(&self) -> bool {
-        self.entries.is_empty()
-    }
-
-    pub fn heap_bytes(&self) -> usize {
-        self.entries
-            .iter()
-            .map(|e| std::mem::size_of::<QueryLogEntry>() + e.domain.capacity())
-            .sum()
-    }
-
     pub fn query(&self, filter: &QueryLogFilter) -> Vec<&QueryLogEntry> {
         self.entries
             .iter()
@@ -92,25 +77,3 @@ pub struct QueryLogFilter {
     pub since: Option<SystemTime>,
     pub limit: Option<usize>,
 }
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-
-    #[test]
-    fn heap_bytes_grows_with_entries() {
-        let mut log = QueryLog::new(100);
-        let empty = log.heap_bytes();
-        log.push(QueryLogEntry {
-            timestamp: SystemTime::now(),
-            src_addr: "127.0.0.1:1234".parse().unwrap(),
-            domain: "example.com".into(),
-            query_type: QueryType::A,
-            path: QueryPath::Forwarded,
-            rescode: ResultCode::NOERROR,
-            latency_us: 500,
-            dnssec: DnssecStatus::Indeterminate,
-        });
-        assert!(log.heap_bytes() > empty);
-    }
-}

src/record.rs

@@ -136,46 +136,6 @@ impl DnsRecord {
         }
     }
 
-    pub fn heap_bytes(&self) -> usize {
-        match self {
-            DnsRecord::A { domain, .. } => domain.capacity(),
-            DnsRecord::NS { domain, host, .. } | DnsRecord::CNAME { domain, host, .. } => {
-                domain.capacity() + host.capacity()
-            }
-            DnsRecord::MX { domain, host, .. } => domain.capacity() + host.capacity(),
-            DnsRecord::AAAA { domain, .. } => domain.capacity(),
-            DnsRecord::DNSKEY {
-                domain, public_key, ..
-            } => domain.capacity() + public_key.capacity(),
-            DnsRecord::DS { domain, digest, .. } => domain.capacity() + digest.capacity(),
-            DnsRecord::RRSIG {
-                domain,
-                signer_name,
-                signature,
-                ..
-            } => domain.capacity() + signer_name.capacity() + signature.capacity(),
-            DnsRecord::NSEC {
-                domain,
-                next_domain,
-                type_bitmap,
-                ..
-            } => domain.capacity() + next_domain.capacity() + type_bitmap.capacity(),
-            DnsRecord::NSEC3 {
-                domain,
-                salt,
-                next_hashed_owner,
-                type_bitmap,
-                ..
-            } => {
-                domain.capacity()
-                    + salt.capacity()
-                    + next_hashed_owner.capacity()
-                    + type_bitmap.capacity()
-            }
-            DnsRecord::UNKNOWN { domain, data, .. } => domain.capacity() + data.capacity(),
-        }
-    }
-
     pub fn set_ttl(&mut self, new_ttl: u32) {
         match self {
             DnsRecord::A { ttl, .. }
@@ -690,14 +650,4 @@ mod tests {
         let parsed = round_trip(&rec);
         assert_eq!(rec, parsed);
     }
-
-    #[test]
-    fn heap_bytes_reflects_string_capacity() {
-        let rec = DnsRecord::CNAME {
-            domain: "a]".repeat(100),
-            host: "b".repeat(200),
-            ttl: 60,
-        };
-        assert!(rec.heap_bytes() >= 300);
-    }
 }

src/recursive.rs

@@ -9,7 +9,7 @@ use crate::cache::DnsCache;
 use crate::forward::forward_udp;
 use crate::header::ResultCode;
 use crate::packet::DnsPacket;
-use crate::question::QueryType;
+use crate::question::{DnsQuestion, QueryType};
 use crate::record::DnsRecord;
 use crate::srtt::SrttCache;
 
@@ -32,14 +32,6 @@ fn dns_addr(ip: impl Into<IpAddr>) -> SocketAddr {
     SocketAddr::new(ip.into(), 53)
 }
 
-fn record_to_addr(rec: &DnsRecord) -> Option<SocketAddr> {
-    match rec {
-        DnsRecord::A { addr, .. } => Some(dns_addr(*addr)),
-        DnsRecord::AAAA { addr, .. } => Some(dns_addr(*addr)),
-        _ => None,
-    }
-}
-
 pub fn reset_udp_state() {
     UDP_DISABLED.store(false, Ordering::Release);
     UDP_FAILURES.store(0, Ordering::Release);
@@ -54,8 +46,11 @@ pub async fn probe_udp(root_hints: &[SocketAddr]) {
         Some(h) => *h,
         None => return,
     };
-    let mut probe = DnsPacket::query(next_id(), ".", QueryType::NS);
-    probe.header.recursion_desired = false;
+    let mut probe = DnsPacket::new();
+    probe.header.id = next_id();
+    probe
+        .questions
+        .push(DnsQuestion::new(".".to_string(), QueryType::NS));
     if forward_udp(&probe, hint, Duration::from_millis(1500))
         .await
         .is_ok()
@@ -65,21 +60,6 @@ pub async fn probe_udp(root_hints: &[SocketAddr]) {
     }
 }
 
-/// Probe whether recursive resolution works by querying root servers.
-/// Tries up to 3 hints before declaring failure.
-pub async fn probe_recursive(root_hints: &[SocketAddr]) -> bool {
-    let mut probe = DnsPacket::query(next_id(), ".", QueryType::NS);
-    probe.header.recursion_desired = false;
-    for hint in root_hints.iter().take(3) {
-        if let Ok(resp) = forward_udp(&probe, *hint, Duration::from_secs(3)).await {
-            if !resp.answers.is_empty() || !resp.authorities.is_empty() {
-                return true;
-            }
-        }
-    }
-    false
-}
-
 pub async fn prime_tld_cache(
     cache: &RwLock<DnsCache>,
     root_hints: &[SocketAddr],
@@ -316,8 +296,17 @@ pub(crate) fn resolve_iterative<'a>(
                             )
                             .await
                             {
-                                new_ns_addrs
-                                    .extend(ns_resp.answers.iter().filter_map(record_to_addr));
+                                for rec in &ns_resp.answers {
+                                    match rec {
+                                        DnsRecord::A { addr, .. } => {
+                                            new_ns_addrs.push(dns_addr(*addr));
+                                        }
+                                        DnsRecord::AAAA { addr, .. } => {
+                                            new_ns_addrs.push(dns_addr(*addr));
+                                        }
+                                        _ => {}
+                                    }
+                                }
                             }
                             if !new_ns_addrs.is_empty() {
                                 break;
@@ -371,7 +360,13 @@ fn find_closest_ns(
                 if let DnsRecord::NS { host, .. } = ns_rec {
                     for qt in [QueryType::A, QueryType::AAAA] {
                         if let Some(resp) = guard.lookup(host, qt) {
-                            addrs.extend(resp.answers.iter().filter_map(record_to_addr));
+                            for rec in &resp.answers {
+                                match rec {
+                                    DnsRecord::A { addr, .. } => addrs.push(dns_addr(*addr)),
+                                    DnsRecord::AAAA { addr, .. } => addrs.push(dns_addr(*addr)),
+                                    _ => {}
+                                }
+                            }
                         }
                     }
                 }
@@ -457,7 +452,13 @@ fn addrs_from_cache(cache: &RwLock<DnsCache>, name: &str) -> Vec<SocketAddr> {
     let mut addrs = Vec::new();
     for qt in [QueryType::A, QueryType::AAAA] {
         if let Some(pkt) = guard.lookup(name, qt) {
-            addrs.extend(pkt.answers.iter().filter_map(record_to_addr));
+            for rec in &pkt.answers {
+                match rec {
+                    DnsRecord::A { addr, .. } => addrs.push(dns_addr(*addr)),
+                    DnsRecord::AAAA { addr, .. } => addrs.push(dns_addr(*addr)),
+                    _ => {}
+                }
+            }
         }
     }
     addrs
@@ -467,13 +468,15 @@ fn glue_addrs_for(response: &DnsPacket, ns_name: &str) -> Vec<SocketAddr> {
     response
         .resources
         .iter()
-        .filter(|r| match r {
-            DnsRecord::A { domain, .. } | DnsRecord::AAAA { domain, .. } => {
-                domain.eq_ignore_ascii_case(ns_name)
+        .filter_map(|r| match r {
+            DnsRecord::A { domain, addr, .. } if domain.eq_ignore_ascii_case(ns_name) => {
+                Some(dns_addr(*addr))
             }
-            _ => false,
+            DnsRecord::AAAA { domain, addr, .. } if domain.eq_ignore_ascii_case(ns_name) => {
+                Some(dns_addr(*addr))
+            }
+            _ => None,
         })
-        .filter_map(record_to_addr)
         .collect()
 }
 
@@ -593,8 +596,12 @@ async fn send_query(
     server: SocketAddr,
     srtt: &RwLock<SrttCache>,
 ) -> crate::Result<DnsPacket> {
-    let mut query = DnsPacket::query(next_id(), qname, qtype);
+    let mut query = DnsPacket::new();
+    query.header.id = next_id();
     query.header.recursion_desired = false;
+    query
+        .questions
+        .push(DnsQuestion::new(qname.to_string(), qtype));
     query.edns = Some(crate::packet::EdnsOpt {
         do_bit: true,
         ..Default::default()
@@ -870,25 +877,14 @@ mod tests {
                 };
                 let handler = handler.clone();
                 tokio::spawn(async move {
-                    let timeout = std::time::Duration::from_secs(5);
                     // Read length-prefixed DNS query
                     let mut len_buf = [0u8; 2];
-                    if tokio::time::timeout(timeout, stream.read_exact(&mut len_buf))
-                        .await
-                        .ok()
-                        .and_then(|r| r.ok())
-                        .is_none()
-                    {
+                    if stream.read_exact(&mut len_buf).await.is_err() {
                         return;
                     }
                     let len = u16::from_be_bytes(len_buf) as usize;
                     let mut data = vec![0u8; len];
-                    if tokio::time::timeout(timeout, stream.read_exact(&mut data))
-                        .await
-                        .ok()
-                        .and_then(|r| r.ok())
-                        .is_none()
-                    {
+                    if stream.read_exact(&mut data).await.is_err() {
                         return;
                     }
 
@@ -1060,7 +1056,11 @@ mod tests {
         })
         .await;
 
-        let query = DnsPacket::query(0xBEEF, "test.com", QueryType::A);
+        let mut query = DnsPacket::new();
+        query.header.id = 0xBEEF;
+        query
+            .questions
+            .push(DnsQuestion::new("test.com".to_string(), QueryType::A));
 
         let resp = crate::forward::forward_tcp(&query, server_addr, Duration::from_secs(2))
             .await
@@ -1120,7 +1120,11 @@ mod tests {
                 .unwrap();
         });
 
-        let query = DnsPacket::query(0xCAFE, "strict.test", QueryType::A);
+        let mut query = DnsPacket::new();
+        query.header.id = 0xCAFE;
+        query
+            .questions
+            .push(DnsQuestion::new("strict.test".to_string(), QueryType::A));
 
         let resp = crate::forward::forward_tcp(&query, addr, Duration::from_secs(2))
             .await

src/srtt.rs

@@ -47,19 +47,16 @@ impl SrttCache {
 
     /// Apply time-based decay: each DECAY_AFTER_SECS period halves distance to INITIAL.
     fn decayed_srtt(entry: &SrttEntry) -> u64 {
-        Self::decay_for_age(entry.srtt_ms, entry.updated_at.elapsed().as_secs())
-    }
-
-    fn decay_for_age(srtt_ms: u64, age_secs: u64) -> u64 {
+        let age_secs = entry.updated_at.elapsed().as_secs();
         if age_secs > DECAY_AFTER_SECS {
             let periods = (age_secs / DECAY_AFTER_SECS).min(8);
-            let mut srtt = srtt_ms;
+            let mut srtt = entry.srtt_ms;
             for _ in 0..periods {
                 srtt = (srtt + INITIAL_SRTT_MS) / 2;
             }
             srtt
         } else {
-            srtt_ms
+            entry.srtt_ms
         }
     }
 
@@ -103,14 +100,6 @@ impl SrttCache {
         addrs.sort_by_key(|a| self.get(a.ip()));
     }
 
-    pub fn heap_bytes(&self) -> usize {
-        let per_slot = std::mem::size_of::<u64>()
-            + std::mem::size_of::<IpAddr>()
-            + std::mem::size_of::<SrttEntry>()
-            + 1;
-        self.entries.capacity() * per_slot
-    }
-
     pub fn len(&self) -> usize {
         self.entries.len()
     }
@@ -119,6 +108,13 @@ impl SrttCache {
         self.entries.is_empty()
     }
 
+    #[cfg(test)]
+    fn set_updated_at(&mut self, ip: IpAddr, at: Instant) {
+        if let Some(entry) = self.entries.get_mut(&ip) {
+            entry.updated_at = at;
+        }
+    }
+
     fn maybe_evict(&mut self) {
         if self.entries.len() < MAX_ENTRIES {
             return;
@@ -214,41 +210,63 @@ mod tests {
         assert_eq!(addrs, original);
     }
 
+    fn age(secs: u64) -> Instant {
+        Instant::now() - std::time::Duration::from_secs(secs)
+    }
+
+    /// Cache with ip(1) saturated at FAILURE_PENALTY_MS
+    fn saturated_penalty_cache() -> SrttCache {
+        let mut cache = SrttCache::new(true);
+        for _ in 0..30 {
+            cache.record_rtt(ip(1), FAILURE_PENALTY_MS, false);
+        }
+        cache
+    }
+
     #[test]
     fn no_decay_within_threshold() {
-        // At exactly DECAY_AFTER_SECS, no decay applied
-        let result = SrttCache::decay_for_age(FAILURE_PENALTY_MS, DECAY_AFTER_SECS);
-        assert_eq!(result, FAILURE_PENALTY_MS);
+        let mut cache = SrttCache::new(true);
+        cache.record_rtt(ip(1), 5000, false);
+        cache.set_updated_at(ip(1), age(DECAY_AFTER_SECS));
+        assert_eq!(cache.get(ip(1)), cache.entries[&ip(1)].srtt_ms);
     }
 
     #[test]
     fn one_decay_period() {
-        let result = SrttCache::decay_for_age(FAILURE_PENALTY_MS, DECAY_AFTER_SECS + 1);
-        let expected = (FAILURE_PENALTY_MS + INITIAL_SRTT_MS) / 2;
-        assert_eq!(result, expected);
+        let mut cache = saturated_penalty_cache();
+        let raw = cache.entries[&ip(1)].srtt_ms;
+        cache.set_updated_at(ip(1), age(DECAY_AFTER_SECS + 1));
+        let expected = (raw + INITIAL_SRTT_MS) / 2;
+        assert_eq!(cache.get(ip(1)), expected);
     }
 
     #[test]
     fn multiple_decay_periods() {
-        let result = SrttCache::decay_for_age(FAILURE_PENALTY_MS, DECAY_AFTER_SECS * 4 + 1);
-        let mut expected = FAILURE_PENALTY_MS;
+        let mut cache = saturated_penalty_cache();
+        let raw = cache.entries[&ip(1)].srtt_ms;
+        cache.set_updated_at(ip(1), age(DECAY_AFTER_SECS * 4 + 1));
+        let mut expected = raw;
         for _ in 0..4 {
             expected = (expected + INITIAL_SRTT_MS) / 2;
         }
-        assert_eq!(result, expected);
+        assert_eq!(cache.get(ip(1)), expected);
     }
 
     #[test]
     fn decay_caps_at_8_periods() {
         // 9 periods and 100 periods should produce the same result (capped at 8)
-        let a = SrttCache::decay_for_age(FAILURE_PENALTY_MS, DECAY_AFTER_SECS * 9 + 1);
-        let b = SrttCache::decay_for_age(FAILURE_PENALTY_MS, DECAY_AFTER_SECS * 100);
-        assert_eq!(a, b);
+        let mut cache_a = saturated_penalty_cache();
+        let mut cache_b = saturated_penalty_cache();
+        cache_a.set_updated_at(ip(1), age(DECAY_AFTER_SECS * 9 + 1));
+        cache_b.set_updated_at(ip(1), age(DECAY_AFTER_SECS * 100));
+        assert_eq!(cache_a.get(ip(1)), cache_b.get(ip(1)));
     }
 
     #[test]
     fn decay_converges_toward_initial() {
-        let decayed = SrttCache::decay_for_age(FAILURE_PENALTY_MS, DECAY_AFTER_SECS * 100);
+        let mut cache = saturated_penalty_cache();
+        cache.set_updated_at(ip(1), age(DECAY_AFTER_SECS * 100));
+        let decayed = cache.get(ip(1));
         let diff = decayed.abs_diff(INITIAL_SRTT_MS);
         assert!(
             diff < 25,
@@ -260,38 +278,29 @@ mod tests {
 
     #[test]
     fn record_rtt_applies_decay_before_ewma() {
-        // Verify decay is applied before EWMA in record_rtt by checking
-        // that a saturated penalty + long age + new sample produces a low SRTT
-        let decayed = SrttCache::decay_for_age(FAILURE_PENALTY_MS, DECAY_AFTER_SECS * 8);
-        // EWMA: (decayed * 7 + 50) / 8
-        let after_ewma = (decayed * 7 + 50) / 8;
-        assert!(
-            after_ewma < 500,
-            "expected decay before EWMA, got srtt={}",
-            after_ewma
-        );
+        let mut cache = saturated_penalty_cache();
+        cache.set_updated_at(ip(1), age(DECAY_AFTER_SECS * 8));
+        cache.record_rtt(ip(1), 50, false);
+        let srtt = cache.get(ip(1));
+        // Without decay-before-EWMA, result would be ~(5000*7+50)/8 ≈ 4381
+        assert!(srtt < 500, "expected decay before EWMA, got srtt={}", srtt);
     }
 
     #[test]
     fn decay_reranks_stale_failures() {
-        // After enough decay, a failed server (5000ms) converges toward
-        // INITIAL (200ms), which is below a stable server at 300ms
-        let decayed = SrttCache::decay_for_age(FAILURE_PENALTY_MS, DECAY_AFTER_SECS * 100);
-        assert!(
-            decayed < 300,
-            "expected decayed penalty ({}) < 300ms",
-            decayed
-        );
-    }
-
-    #[test]
-    fn heap_bytes_grows_with_entries() {
-        let mut cache = SrttCache::new(true);
-        let empty = cache.heap_bytes();
-        for i in 1..=10u8 {
-            cache.record_rtt(ip(i), 100, false);
+        let mut cache = saturated_penalty_cache();
+        for _ in 0..30 {
+            cache.record_rtt(ip(2), 300, false);
         }
-        assert!(cache.heap_bytes() > empty);
+        let mut addrs = vec![sock(1), sock(2)];
+        cache.sort_by_rtt(&mut addrs);
+        assert_eq!(addrs, vec![sock(2), sock(1)]);
+
+        // Age server 1 so it decays toward INITIAL (200ms) — below server 2's 300ms
+        cache.set_updated_at(ip(1), age(DECAY_AFTER_SECS * 100));
+        let mut addrs = vec![sock(1), sock(2)];
+        cache.sort_by_rtt(&mut addrs);
+        assert_eq!(addrs, vec![sock(1), sock(2)]);
     }
 
     #[test]

src/stats.rs

@@ -1,92 +1,5 @@
 use std::time::Instant;
 
-/// Returns the process memory footprint in bytes, or 0 if unavailable.
-/// macOS: phys_footprint (matches Activity Monitor). Linux: RSS from /proc/self/statm.
-pub fn process_memory_bytes() -> usize {
-    #[cfg(target_os = "macos")]
-    {
-        macos_rss()
-    }
-    #[cfg(target_os = "linux")]
-    {
-        linux_rss()
-    }
-    #[cfg(not(any(target_os = "macos", target_os = "linux")))]
-    {
-        0
-    }
-}
-
-#[cfg(target_os = "macos")]
-fn macos_rss() -> usize {
-    use std::mem;
-    extern "C" {
-        fn mach_task_self() -> u32;
-        fn task_info(
-            target_task: u32,
-            flavor: u32,
-            task_info_out: *mut TaskVmInfo,
-            task_info_count: *mut u32,
-        ) -> i32;
-    }
-    // Partial task_vm_info_data_t — only fields up to phys_footprint.
-    #[repr(C)]
-    struct TaskVmInfo {
-        virtual_size: u64,
-        region_count: i32,
-        page_size: i32,
-        resident_size: u64,
-        resident_size_peak: u64,
-        device: u64,
-        device_peak: u64,
-        internal: u64,
-        internal_peak: u64,
-        external: u64,
-        external_peak: u64,
-        reusable: u64,
-        reusable_peak: u64,
-        purgeable_volatile_pmap: u64,
-        purgeable_volatile_resident: u64,
-        purgeable_volatile_virtual: u64,
-        compressed: u64,
-        compressed_peak: u64,
-        compressed_lifetime: u64,
-        phys_footprint: u64,
-    }
-    const TASK_VM_INFO: u32 = 22;
-    let mut info: TaskVmInfo = unsafe { mem::zeroed() };
-    let mut count = (mem::size_of::<TaskVmInfo>() / mem::size_of::<u32>()) as u32;
-    let kr = unsafe { task_info(mach_task_self(), TASK_VM_INFO, &mut info, &mut count) };
-    if kr == 0 {
-        info.phys_footprint as usize
-    } else {
-        0
-    }
-}
-
-#[cfg(target_os = "linux")]
-fn linux_rss() -> usize {
-    extern "C" {
-        fn sysconf(name: i32) -> i64;
-    }
-    const SC_PAGESIZE: i32 = 30; // x86_64 + aarch64; differs on mips (28), sparc (29)
-    let page_size = unsafe { sysconf(SC_PAGESIZE) };
-    let page_size = if page_size > 0 {
-        page_size as usize
-    } else {
-        4096
-    };
-
-    if let Ok(statm) = std::fs::read_to_string("/proc/self/statm") {
-        if let Some(rss_pages) = statm.split_whitespace().nth(1) {
-            if let Ok(pages) = rss_pages.parse::<usize>() {
-                return pages * page_size;
-            }
-        }
-    }
-    0
-}
-
 pub struct ServerStats {
     queries_total: u64,
     queries_forwarded: u64,
@@ -100,7 +13,7 @@ pub struct ServerStats {
     started_at: Instant,
 }
 
-#[derive(Clone, Copy, Debug, PartialEq, Eq)]
+#[derive(Clone, Copy, PartialEq, Eq)]
 pub enum QueryPath {
     Local,
     Cached,

src/tls.rs

@@ -5,9 +5,7 @@ use std::sync::Arc;
 use log::{info, warn};
 
 use crate::ctx::ServerCtx;
-use rcgen::{
-    BasicConstraints, CertificateParams, DnType, IsCa, Issuer, KeyPair, KeyUsagePurpose, SanType,
-};
+use rcgen::{BasicConstraints, CertificateParams, DnType, IsCa, KeyPair, KeyUsagePurpose, SanType};
 use rustls::pki_types::{CertificateDer, PrivateKeyDer, PrivatePkcs8KeyDer};
 use rustls::ServerConfig;
 use time::{Duration, OffsetDateTime};
@@ -15,13 +13,6 @@ use time::{Duration, OffsetDateTime};
 const CA_VALIDITY_DAYS: i64 = 3650; // 10 years
 const CERT_VALIDITY_DAYS: i64 = 365; // 1 year
 
-/// Common Name on Numa's local CA. Referenced by trust-store helpers
-/// (`security`, `certutil`) when locating the cert for removal.
-pub const CA_COMMON_NAME: &str = "Numa Local CA";
-
-/// Filename of the CA certificate inside the data dir.
-pub const CA_FILE_NAME: &str = "ca.pem";
-
 /// Collect all service + LAN peer names and regenerate the TLS cert.
 pub fn regenerate_tls(ctx: &ServerCtx) {
     let tls = match &ctx.tls_config {
@@ -33,7 +24,7 @@ pub fn regenerate_tls(ctx: &ServerCtx) {
     names.extend(ctx.lan_peers.lock().unwrap().names());
     let names: Vec<String> = names.into_iter().collect();
 
-    match build_tls_config(&ctx.proxy_tld, &names, Vec::new(), &ctx.data_dir) {
+    match build_tls_config(&ctx.proxy_tld, &names) {
         Ok(new_config) => {
             tls.store(new_config);
             info!("TLS cert regenerated for {} services", names.len());
@@ -42,63 +33,20 @@ pub fn regenerate_tls(ctx: &ServerCtx) {
     }
 }
 
-/// Advisory for TLS-setup failures caused by a non-writable data dir;
-/// `None` if not applicable so the caller can fall back to the raw error.
-pub fn try_data_dir_advisory(err: &crate::Error, data_dir: &Path) -> Option<String> {
-    let io_err = err.downcast_ref::<std::io::Error>()?;
-    if io_err.kind() != std::io::ErrorKind::PermissionDenied {
-        return None;
-    }
-    let o = "\x1b[1;38;2;192;98;58m";
-    let r = "\x1b[0m";
-    Some(format!(
-        "
-{o}Numa{r} — HTTPS proxy disabled: cannot write TLS CA to {}.
-
-  The data directory is not writable by the current user. Numa needs
-  to persist a local Certificate Authority there to serve .numa over
-  HTTPS. DNS resolution and plain-HTTP proxy continue to work.
-
-  Fix — pick one:
-
-    1. Install Numa as the system resolver (sets up a writable data dir):
-
-         sudo numa install       (on Windows, run as Administrator)
-
-    2. Point data_dir at a path you can write.
-       Create ~/.config/numa/numa.toml with:
-
-         [server]
-         data_dir = \"/path/you/can/write\"
-
-",
-        data_dir.display()
-    ))
-}
-
 /// Build a TLS config with a cert covering all provided service names.
 /// Wildcards under single-label TLDs (*.numa) are rejected by browsers,
 /// so we list each service explicitly as a SAN.
-/// `alpn` is advertised in the TLS ServerHello — pass empty for the proxy
-/// (which accepts any ALPN), or `[b"dot"]` for DoT (RFC 7858 §3.2).
-/// `data_dir` is where the CA material is stored — taken from
-/// `[server] data_dir` in numa.toml (defaults to `crate::data_dir()`).
-pub fn build_tls_config(
-    tld: &str,
-    service_names: &[String],
-    alpn: Vec<Vec<u8>>,
-    data_dir: &Path,
-) -> crate::Result<Arc<ServerConfig>> {
-    let (ca_der, issuer) = ensure_ca(data_dir)?;
-    let (cert_chain, key) = generate_service_cert(&ca_der, &issuer, tld, service_names)?;
+pub fn build_tls_config(tld: &str, service_names: &[String]) -> crate::Result<Arc<ServerConfig>> {
+    let dir = crate::data_dir();
+    let (ca_cert, ca_key) = ensure_ca(&dir)?;
+    let (cert_chain, key) = generate_service_cert(&ca_cert, &ca_key, tld, service_names)?;
 
     // Ensure a crypto provider is installed (rustls needs one)
     let _ = rustls::crypto::ring::default_provider().install_default();
 
-    let mut config = ServerConfig::builder()
+    let config = ServerConfig::builder()
         .with_no_client_auth()
         .with_single_cert(cert_chain, key)?;
-    config.alpn_protocols = alpn;
 
     info!(
         "TLS configured for {} .{} domains",
@@ -108,20 +56,18 @@ pub fn build_tls_config(
     Ok(Arc::new(config))
 }
 
-fn ensure_ca(dir: &Path) -> crate::Result<(CertificateDer<'static>, Issuer<'static, KeyPair>)> {
+fn ensure_ca(dir: &Path) -> crate::Result<(rcgen::Certificate, KeyPair)> {
     let ca_key_path = dir.join("ca.key");
-    let ca_cert_path = dir.join(CA_FILE_NAME);
+    let ca_cert_path = dir.join("ca.pem");
 
     if ca_key_path.exists() && ca_cert_path.exists() {
         let key_pem = std::fs::read_to_string(&ca_key_path)?;
         let cert_pem = std::fs::read_to_string(&ca_cert_path)?;
         let key_pair = KeyPair::from_pem(&key_pem)?;
-        let ca_der = rustls_pemfile::certs(&mut cert_pem.as_bytes())
-            .next()
-            .ok_or("empty CA PEM file")??;
-        let issuer = Issuer::from_ca_cert_der(&ca_der, key_pair)?;
+        let params = CertificateParams::from_ca_cert_pem(&cert_pem)?;
+        let cert = params.self_signed(&key_pair)?;
         info!("loaded CA from {:?}", ca_cert_path);
-        return Ok((ca_der, issuer));
+        return Ok((cert, key_pair));
     }
 
     // Generate new CA
@@ -131,7 +77,7 @@ fn ensure_ca(dir: &Path) -> crate::Result<(CertificateDer<'static>, Issuer<'stat
     let mut params = CertificateParams::default();
     params
         .distinguished_name
-        .push(DnType::CommonName, CA_COMMON_NAME);
+        .push(DnType::CommonName, "Numa Local CA");
     params.is_ca = IsCa::Ca(BasicConstraints::Unconstrained);
     params.key_usages = vec![KeyUsagePurpose::KeyCertSign, KeyUsagePurpose::CrlSign];
     params.not_before = OffsetDateTime::now_utc();
@@ -149,16 +95,14 @@ fn ensure_ca(dir: &Path) -> crate::Result<(CertificateDer<'static>, Issuer<'stat
     }
 
     info!("generated CA at {:?}", ca_cert_path);
-    let ca_der = cert.der().clone();
-    let issuer = Issuer::new(params, key_pair);
-    Ok((ca_der, issuer))
+    Ok((cert, key_pair))
 }
 
 /// Generate a cert with explicit SANs for each service name.
 /// Always regenerated at startup (~5ms) — no disk caching needed.
 fn generate_service_cert(
-    ca_der: &CertificateDer<'static>,
-    issuer: &Issuer<'_, KeyPair>,
+    ca_cert: &rcgen::Certificate,
+    ca_key: &KeyPair,
     tld: &str,
     service_names: &[String],
 ) -> crate::Result<(Vec<CertificateDer<'static>>, PrivateKeyDer<'static>)> {
@@ -193,7 +137,7 @@ fn generate_service_cert(
     params.not_before = OffsetDateTime::now_utc();
     params.not_after = OffsetDateTime::now_utc() + Duration::days(CERT_VALIDITY_DAYS);
 
-    let cert = params.signed_by(&key_pair, issuer)?;
+    let cert = params.signed_by(&key_pair, ca_cert, ca_key)?;
 
     info!(
         "generated TLS cert for: {}",
@@ -204,39 +148,9 @@ fn generate_service_cert(
             .join(", ")
     );
 
-    let cert_der = cert.der().clone();
-    let ca_cert_der = ca_der.clone();
+    let cert_der = CertificateDer::from(cert.der().to_vec());
+    let ca_der = CertificateDer::from(ca_cert.der().to_vec());
     let key_der = PrivateKeyDer::Pkcs8(PrivatePkcs8KeyDer::from(key_pair.serialize_der()));
 
-    Ok((vec![cert_der, ca_cert_der], key_der))
-}
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-    use std::path::PathBuf;
-
-    #[test]
-    fn try_data_dir_advisory_permission_denied() {
-        let err: crate::Error =
-            Box::new(std::io::Error::from(std::io::ErrorKind::PermissionDenied));
-        let path = PathBuf::from("/usr/local/var/numa");
-        let msg = try_data_dir_advisory(&err, &path).expect("should advise");
-        assert!(msg.contains("HTTPS proxy disabled"));
-        assert!(msg.contains("/usr/local/var/numa"));
-        assert!(msg.contains("numa install"));
-        assert!(msg.contains("data_dir"));
-    }
-
-    #[test]
-    fn try_data_dir_advisory_skips_other_io_kinds() {
-        let err: crate::Error = Box::new(std::io::Error::from(std::io::ErrorKind::NotFound));
-        assert!(try_data_dir_advisory(&err, &PathBuf::from("/x")).is_none());
-    }
-
-    #[test]
-    fn try_data_dir_advisory_skips_non_io_errors() {
-        let err: crate::Error = "rcgen failure".into();
-        assert!(try_data_dir_advisory(&err, &PathBuf::from("/x")).is_none());
-    }
+    Ok((vec![cert_der, ca_der], key_der))
 }

tests/docker/install-trust.sh

@@ -1,123 +0,0 @@
-#!/usr/bin/env bash
-#
-# Cross-distro CA trust contract test for issue #35.
-#
-# Runs the exact shell commands `src/system_dns.rs::trust_ca_linux` would run
-# on each Linux trust-store family (Debian, Fedora pki, Arch p11-kit), and
-# asserts the certificate ends up in (and is removed from) the system bundle.
-#
-# This is a contract test, not an integration test: it doesn't drive the Rust
-# code (that would need systemd-in-container). It verifies the assumptions in
-# `LINUX_TRUST_STORES` against the real distro behavior. If you change that
-# table in src/system_dns.rs, update the per-distro cases below to match.
-#
-# Requirements: docker, openssl (host).
-# Usage:        ./tests/docker/install-trust.sh
-
-set -euo pipefail
-
-cd "$(dirname "$0")/../.."
-
-GREEN="\033[32m"; RED="\033[31m"; RESET="\033[0m"
-
-# Self-signed CA fixture, mounted into each container as ca.pem.
-# basicConstraints=CA:TRUE is required — without it, Debian's
-# update-ca-certificates silently skips the cert during bundle build.
-FIXTURE_DIR=$(mktemp -d)
-trap 'rm -rf "$FIXTURE_DIR"' EXIT
-openssl req -x509 -newkey rsa:2048 -nodes -days 1 \
-    -keyout "$FIXTURE_DIR/ca.key" \
-    -out    "$FIXTURE_DIR/ca.pem" \
-    -subj   "/CN=Numa Local CA Test $(date +%s)" \
-    -addext "basicConstraints=critical,CA:TRUE" \
-    -addext "keyUsage=critical,keyCertSign,cRLSign" >/dev/null 2>&1
-
-# Distro bundles store certs differently — Debian writes raw PEM only,
-# Fedora prepends "# CN" comment headers, Arch via extract-compat is
-# raw PEM. To detect cert presence uniformly we grep for a deterministic
-# substring of the base64 body (first base64 line is unique per cert).
-CERT_TAG=$(sed -n '2p' "$FIXTURE_DIR/ca.pem")
-
-PASSED=0; FAILED=0
-
-run_case() {
-    local distro="$1"; shift
-    local image="$1"; shift
-    local platform="$1"; shift
-    local script="$1"
-
-    printf "── %s (%s) ──\n" "$distro" "$image"
-    if docker run --rm \
-        --platform "$platform" \
-        --security-opt seccomp=unconfined \
-        -e CERT_TAG="$CERT_TAG" \
-        -e DEBIAN_FRONTEND=noninteractive \
-        -v "$FIXTURE_DIR/ca.pem:/fixture/ca.pem:ro" \
-        "$image" bash -c "$script"; then
-        printf "${GREEN}✓${RESET} %s\n\n" "$distro"
-        PASSED=$((PASSED + 1))
-    else
-        printf "${RED}✗${RESET} %s\n\n" "$distro"
-        FAILED=$((FAILED + 1))
-    fi
-}
-
-# Debian / Ubuntu / Mint — anchor: /usr/local/share/ca-certificates/*.crt
-run_case "debian" "debian:stable" "linux/amd64" '
-    set -e
-    apt-get update -qq
-    apt-get install -qq -y ca-certificates >/dev/null
-    install -m 0644 /fixture/ca.pem /usr/local/share/ca-certificates/numa-local-ca.crt
-    update-ca-certificates >/dev/null 2>&1
-    grep -q "$CERT_TAG" /etc/ssl/certs/ca-certificates.crt
-    echo "  install: cert present in bundle"
-    rm /usr/local/share/ca-certificates/numa-local-ca.crt
-    update-ca-certificates --fresh >/dev/null 2>&1
-    if grep -q "$CERT_TAG" /etc/ssl/certs/ca-certificates.crt; then
-        echo "  uninstall: cert STILL present (regression)" >&2
-        exit 1
-    fi
-    echo "  uninstall: cert removed from bundle"
-'
-
-# Fedora / RHEL / CentOS / SUSE — anchor: /etc/pki/ca-trust/source/anchors/*.pem
-run_case "fedora" "fedora:latest" "linux/amd64" '
-    set -e
-    dnf install -q -y ca-certificates >/dev/null
-    install -m 0644 /fixture/ca.pem /etc/pki/ca-trust/source/anchors/numa-local-ca.pem
-    update-ca-trust extract
-    grep -q "$CERT_TAG" /etc/pki/ca-trust/extracted/pem/tls-ca-bundle.pem
-    echo "  install: cert present in bundle"
-    rm /etc/pki/ca-trust/source/anchors/numa-local-ca.pem
-    update-ca-trust extract
-    if grep -q "$CERT_TAG" /etc/pki/ca-trust/extracted/pem/tls-ca-bundle.pem; then
-        echo "  uninstall: cert STILL present (regression)" >&2
-        exit 1
-    fi
-    echo "  uninstall: cert removed from bundle"
-'
-
-# Arch / Manjaro — anchor: /etc/ca-certificates/trust-source/anchors/*.pem
-# archlinux:latest is x86_64-only; --platform forces emulation on Apple Silicon.
-run_case "arch" "archlinux:latest" "linux/amd64" '
-    set -e
-    # pacman 7+ filters syscalls in its own sandbox; disable for Rosetta/qemu emulation.
-    sed -i "s/^#DisableSandboxSyscalls/DisableSandboxSyscalls/" /etc/pacman.conf
-    pacman -Sy --noconfirm --needed ca-certificates p11-kit >/dev/null 2>&1
-    install -m 0644 /fixture/ca.pem /etc/ca-certificates/trust-source/anchors/numa-local-ca.pem
-    trust extract-compat
-    grep -q "$CERT_TAG" /etc/ssl/certs/ca-certificates.crt
-    echo "  install: cert present in bundle"
-    rm /etc/ca-certificates/trust-source/anchors/numa-local-ca.pem
-    trust extract-compat
-    if grep -q "$CERT_TAG" /etc/ssl/certs/ca-certificates.crt; then
-        echo "  uninstall: cert STILL present (regression)" >&2
-        exit 1
-    fi
-    echo "  uninstall: cert removed from bundle"
-'
-
-printf "── summary ──\n"
-printf "  ${GREEN}passed${RESET}: %d\n" "$PASSED"
-printf "  ${RED}failed${RESET}: %d\n" "$FAILED"
-[ "$FAILED" -eq 0 ]

tests/docker/smoke-arch.sh

@@ -1,147 +0,0 @@
-#!/usr/bin/env bash
-#
-# Arch Linux compatibility smoke test.
-#
-# Builds numa from source inside an archlinux:latest container, runs it
-# in forward mode on port 5354, and verifies a single DNS query returns
-# an A record. Validates the "Arch compatible" claim end-to-end before
-# release announcements.
-#
-# Dogfooding: the test numa forwards to the host's running numa via
-# host.docker.internal (Docker Desktop's host gateway). This avoids the
-# Docker NAT/UDP issues with public resolvers and exercises the realistic
-# numa-on-numa shape. Requires the host to be running numa on port 53.
-#
-# First run is slow (~8-12 min): image pull + pacman + cold cargo build.
-# No caching across runs.
-#
-# Requirements: docker, host running numa on 0.0.0.0:53
-# Usage:        ./tests/docker/smoke-arch.sh
-
-set -euo pipefail
-
-cd "$(dirname "$0")/../.."
-
-GREEN="\033[32m"; RED="\033[31m"; RESET="\033[0m"
-
-# Precondition: the test numa-on-arch forwards to the host numa as its
-# upstream (dogfood pattern). Fail fast with a clear error if there is
-# no working DNS on the host, rather than letting the dig inside the
-# container time out with "deadline has elapsed".
-if ! dig @127.0.0.1 google.com A +short +time=1 +tries=1 >/dev/null 2>&1; then
-    printf "${RED}error:${RESET} host numa is not answering on 127.0.0.1:53\n" >&2
-    echo "  This test forwards to the host numa via host.docker.internal." >&2
-    echo "  Start numa on the host first (sudo numa install), then rerun." >&2
-    exit 1
-fi
-
-echo "── building + running numa on archlinux:latest ──"
-echo "  (first run is slow: image pull + pacman + cold cargo build, ~8-12 min)"
-echo
-
-docker run --rm \
-    --platform linux/amd64 \
-    --security-opt seccomp=unconfined \
-    -v "$PWD:/src:ro" \
-    -v numa-arch-cargo:/root/.cargo \
-    -v numa-arch-target:/work/target \
-    archlinux:latest bash -c '
-    set -e
-
-    # pacman 7+ filters syscalls in its own sandbox; disable for Rosetta/qemu
-    sed -i "s/^#DisableSandboxSyscalls/DisableSandboxSyscalls/" /etc/pacman.conf
-
-    echo "── pacman: installing build + runtime deps ──"
-    pacman -Sy --noconfirm --needed rust gcc pkgconf cmake make perl bind 2>&1 | tail -3
-    echo
-
-    # Copy source to a writable workdir, skipping target/ + .git so we
-    # do not pull in the host (macOS) build artifacts.
-    mkdir -p /work
-    tar -C /src --exclude=./target --exclude=./.git -cf - . | tar -C /work -xf -
-    cd /work
-
-    echo "── cargo build --release --locked ──"
-    cargo build --release --locked 2>&1 | tail -5
-    echo
-
-    # Dogfood: forward to the host numa via host.docker.internal.
-    # numa parses upstream.address as a literal SocketAddr, so we resolve
-    # the hostname to an IPv4 address first (force v4 — getent hosts may
-    # return IPv6 first, and IPv6 addresses need bracketed addr:port form).
-    HOST_IP=$(getent ahostsv4 host.docker.internal | awk "/STREAM/ {print \$1; exit}")
-    if [ -z "$HOST_IP" ]; then
-        echo "  ✗ could not resolve host.docker.internal to IPv4 (not on Docker Desktop?)"
-        exit 1
-    fi
-    echo "── starting numa on :5354 (forward to host numa at $HOST_IP:53) ──"
-    # Intentionally NOT setting [server] data_dir — we want to exercise the
-    # default code path (data_dir() → daemon_data_dir() → /var/lib/numa) so
-    # the FHS-path assertion below verifies the live wiring, not just the
-    # unit-tested helper.
-    cat > /tmp/numa.toml <<EOF
-[server]
-bind_addr = "127.0.0.1:5354"
-api_port = 5381
-
-[upstream]
-mode = "forward"
-address = "$HOST_IP"
-port = 53
-EOF
-
-    ./target/release/numa /tmp/numa.toml > /tmp/numa.log 2>&1 &
-    NUMA_PID=$!
-
-    # Poll for readiness — numa is ready when it answers a query
-    READY=0
-    for i in 1 2 3 4 5 6 7 8; do
-        sleep 1
-        if dig @127.0.0.1 -p 5354 google.com A +short +time=1 +tries=1 2>/dev/null \
-            | grep -qE "^[0-9]+\.[0-9]+\.[0-9]+\.[0-9]+$"; then
-            READY=1
-            break
-        fi
-    done
-
-    if [ "$READY" -ne 1 ]; then
-        echo "  ✗ numa did not return an A record after 8s"
-        echo "  numa log:"
-        cat /tmp/numa.log
-        kill $NUMA_PID 2>/dev/null || true
-        exit 1
-    fi
-
-    echo "── dig @127.0.0.1 -p 5354 google.com A ──"
-    ANSWER=$(dig @127.0.0.1 -p 5354 google.com A +short +time=2 +tries=1)
-    echo "$ANSWER" | sed "s/^/  /"
-
-    kill $NUMA_PID 2>/dev/null || true
-
-    # FHS path assertion: the default data dir on Linux must be /var/lib/numa
-    # (not the legacy /usr/local/var/numa). The CA cert generated at startup
-    # is the canonical proof that numa wrote to the right place.
-    echo
-    echo "── FHS path check ──"
-    if [ -f /var/lib/numa/ca.pem ]; then
-        echo "  ✓ CA cert at /var/lib/numa/ca.pem (FHS path)"
-    else
-        echo "  ✗ CA cert NOT at /var/lib/numa/ca.pem"
-        echo "  ls /var/lib/numa/:"
-        ls -la /var/lib/numa/ 2>&1 | sed "s/^/    /"
-        echo "  ls /usr/local/var/numa/:"
-        ls -la /usr/local/var/numa/ 2>&1 | sed "s/^/    /"
-        exit 1
-    fi
-    if [ -e /usr/local/var/numa ]; then
-        echo "  ✗ legacy path /usr/local/var/numa unexpectedly exists on a fresh container"
-        exit 1
-    fi
-    echo "  ✓ legacy path /usr/local/var/numa absent (fresh install used FHS)"
-
-    echo
-    echo "  ✓ numa built, ran, answered a forward query, and used the FHS data dir on Arch"
-'
-
-echo
-printf "${GREEN}── smoke-arch passed ──${RESET}\n"

tests/docker/smoke-port53.sh

@@ -1,138 +0,0 @@
-#!/usr/bin/env bash
-#
-# Port-53 conflict advisory integration test.
-#
-# Builds numa from source inside a debian:bookworm container, pre-binds
-# port 53 with a UDP socket, then runs numa bare (default bind_addr
-# 0.0.0.0:53). Verifies:
-#   - process exits with code 1
-#   - stderr contains the advisory ("cannot bind to")
-#   - stderr contains both fix suggestions ("numa install", "bind_addr")
-#
-# This is the end-to-end test for the fix in:
-#   src/main.rs — AddrInUse match arm → eprint advisory + process::exit(1)
-#
-# No systemd-resolved needed — the conflict is simulated by a Python
-# UDP socket held open before numa starts.
-#
-# Requirements: docker
-# Usage:        ./tests/docker/smoke-port53.sh
-
-set -euo pipefail
-
-cd "$(dirname "$0")/../.."
-
-GREEN="\033[32m"; RED="\033[31m"; RESET="\033[0m"
-
-pass() { printf "  ${GREEN}✓${RESET} %s\n" "$1"; }
-fail() { printf "  ${RED}✗${RESET} %s\n" "$1"; printf "    %s\n" "$2"; FAILED=$((FAILED+1)); }
-FAILED=0
-
-echo "── smoke-port53: building + testing numa on debian:bookworm ──"
-echo "  (first run is slow: image pull + cold cargo build, ~5-8 min)"
-echo
-
-OUTPUT=$(docker run --rm \
-    --platform linux/amd64 \
-    -v "$PWD:/src:ro" \
-    -v numa-port53-cargo:/root/.cargo \
-    -v numa-port53-target:/work/target \
-    debian:bookworm bash -c '
-set -e
-
-apt-get update -qq && apt-get install -y -qq curl build-essential python3 2>&1 | tail -3
-
-# Install rustup if not already in the cargo cache volume
-if ! command -v cargo &>/dev/null; then
-    curl -sSf https://sh.rustup.rs | sh -s -- -y --profile minimal --quiet
-fi
-. "$HOME/.cargo/env"
-
-# Copy source to a writable workdir
-mkdir -p /work
-tar -C /src --exclude=./target --exclude=./.git -cf - . | tar -C /work -xf -
-cd /work
-
-echo "── cargo build --release --locked ──"
-cargo build --release --locked 2>&1 | tail -5
-echo
-
-# Write the holder script to a file to avoid quoting hell.
-# Holds port 53 until killed — no sleep race.
-cat > /tmp/hold53.py << '"'"'PYEOF'"'"'
-import socket, signal
-s = socket.socket(socket.AF_INET, socket.SOCK_DGRAM)
-s.setsockopt(socket.SOL_SOCKET, socket.SO_REUSEADDR, 0)
-s.bind(("", 53))
-signal.pause()
-PYEOF
-
-python3 /tmp/hold53.py &
-HOLDER_PID=$!
-
-# Verify the holder is actually up before proceeding
-sleep 0.3
-if ! kill -0 $HOLDER_PID 2>/dev/null; then
-    echo "holder_failed=1"
-    exit 1
-fi
-
-echo "── running numa with port 53 already bound ──"
-# timeout 5: guards against numa not exiting (advisory not fired, bug present)
-# Capture stderr to a file so the exit code is not clobbered by || or $()
-set +e
-timeout 5 ./target/release/numa > /tmp/numa-stderr.txt 2>&1
-EXIT_CODE=$?
-set -e
-STDERR=$(cat /tmp/numa-stderr.txt)
-
-kill $HOLDER_PID 2>/dev/null || true
-
-echo "exit_code=$EXIT_CODE"
-printf "%s" "$STDERR" | sed "s/^/  numa: /"
-' 2>&1)
-
-echo "$OUTPUT"
-
-echo
-echo "── assertions ──"
-
-if echo "$OUTPUT" | grep -q "holder_failed=1"; then
-    echo "  SETUP FAILED: could not pre-bind port 53 inside container"
-    exit 1
-fi
-
-EXIT_CODE=$(echo "$OUTPUT" | grep '^exit_code=' | cut -d= -f2)
-
-if [ "${EXIT_CODE:-}" = "1" ]; then
-    pass "exits with code 1"
-else
-    fail "exits with code 1" "got: exit_code=${EXIT_CODE:-<missing>}"
-fi
-
-if echo "$OUTPUT" | grep -q "cannot bind to"; then
-    pass "advisory printed to stderr"
-else
-    fail "advisory printed to stderr" "stderr did not contain 'cannot bind to'"
-fi
-
-if echo "$OUTPUT" | grep -q "numa install"; then
-    pass "advisory offers 'sudo numa install'"
-else
-    fail "advisory offers 'sudo numa install'" "not found in output"
-fi
-
-if echo "$OUTPUT" | grep -q "bind_addr"; then
-    pass "advisory offers non-privileged port alternative"
-else
-    fail "advisory offers non-privileged port alternative" "'bind_addr' not found in output"
-fi
-
-echo
-if [ "$FAILED" -eq 0 ]; then
-    printf "${GREEN}── smoke-port53 passed ──${RESET}\n"
-    exit 0
-else
-    printf "${RED}── smoke-port53 failed ($FAILED assertion(s)) ──${RESET}\n"
-    exit 1
-fi

tests/integration.sh

@@ -404,241 +404,6 @@ check "Cache flushed" \
 
 kill "$NUMA_PID" 2>/dev/null || true
 wait "$NUMA_PID" 2>/dev/null || true
-sleep 1
-
-# ---- Suite 5: DNS-over-TLS (RFC 7858) ----
-echo ""
-echo "╔══════════════════════════════════════════╗"
-echo "║  Suite 5: DNS-over-TLS (RFC 7858)        ║"
-echo "╚══════════════════════════════════════════╝"
-
-if ! command -v kdig >/dev/null 2>&1; then
-    printf "  ${DIM}skipped — install 'knot' for kdig${RESET}\n"
-elif ! command -v openssl >/dev/null 2>&1; then
-    printf "  ${DIM}skipped — openssl not found${RESET}\n"
-else
-    DOT_PORT=8853
-    DOT_CERT=/tmp/numa-integration-dot.crt
-    DOT_KEY=/tmp/numa-integration-dot.key
-
-    # Generate a test cert mirroring production self_signed_tls SAN shape
-    # (*.numa wildcard + explicit numa.numa apex).
-    openssl req -x509 -newkey rsa:2048 -nodes -days 1 \
-        -keyout "$DOT_KEY" -out "$DOT_CERT" \
-        -subj "/CN=Numa .numa services" \
-        -addext "subjectAltName=DNS:*.numa,DNS:numa.numa" \
-        >/dev/null 2>&1
-
-    # Suite 5 uses a local zone so it's upstream-independent — the point is
-    # to exercise the DoT transport layer (handshake, ALPN, framing,
-    # persistent connections), not re-test recursive resolution.
-    cat > "$CONFIG" << CONF
-[server]
-bind_addr = "127.0.0.1:$PORT"
-api_port = $API_PORT
-
-[upstream]
-mode = "forward"
-address = "127.0.0.1"
-port = 65535
-
-[cache]
-max_entries = 10000
-
-[blocking]
-enabled = false
-
-[proxy]
-enabled = false
-
-[dot]
-enabled = true
-port = $DOT_PORT
-bind_addr = "127.0.0.1"
-cert_path = "$DOT_CERT"
-key_path = "$DOT_KEY"
-
-[[zones]]
-domain = "dot-test.example"
-record_type = "A"
-value = "10.0.0.1"
-ttl = 60
-CONF
-
-    RUST_LOG=info "$BINARY" "$CONFIG" > "$LOG" 2>&1 &
-    NUMA_PID=$!
-    sleep 4
-
-    if ! kill -0 "$NUMA_PID" 2>/dev/null; then
-        FAILED=$((FAILED + 1))
-        printf "  ${RED}✗${RESET} DoT startup\n"
-        printf "    ${DIM}%s${RESET}\n" "$(tail -5 "$LOG")"
-    else
-        echo ""
-        echo "=== Listener ==="
-
-        check "DoT bound on 127.0.0.1:$DOT_PORT" \
-            "DoT listening on 127.0.0.1:$DOT_PORT" \
-            "$(grep 'DoT listening' "$LOG")"
-
-        KDIG="kdig @127.0.0.1 -p $DOT_PORT +tls +tls-ca=$DOT_CERT +tls-hostname=numa.numa +time=5 +retry=0"
-
-        echo ""
-        echo "=== Queries over DoT ==="
-
-        check "DoT local zone A record" \
-            "10.0.0.1" \
-            "$($KDIG +short dot-test.example A 2>/dev/null)"
-
-        # +keepopen reuses one TLS connection for multiple queries — tests
-        # persistent connection handling. kdig applies options left-to-right,
-        # so +short and +keepopen must come before the query specs.
-        check "DoT persistent connection (3 queries, 1 handshake)" \
-            "10.0.0.1" \
-            "$($KDIG +keepopen +short dot-test.example A dot-test.example A dot-test.example A 2>/dev/null | head -1)"
-
-        echo ""
-        echo "=== ALPN ==="
-
-        # Positive case: client offers "dot", server picks it.
-        ALPN_OK=$(echo "" | openssl s_client -connect "127.0.0.1:$DOT_PORT" \
-            -servername numa.numa -alpn dot -CAfile "$DOT_CERT" 2>&1 </dev/null || true)
-        check "DoT negotiates ALPN \"dot\"" \
-            "ALPN protocol: dot" \
-            "$ALPN_OK"
-
-        # Negative case: client offers only "h2", server must reject the
-        # handshake with no_application_protocol alert (cross-protocol
-        # confusion defense, RFC 7858bis §3.2).
-        if echo "" | openssl s_client -connect "127.0.0.1:$DOT_PORT" \
-            -servername numa.numa -alpn h2 -CAfile "$DOT_CERT" \
-            </dev/null >/dev/null 2>&1; then
-            ALPN_MISMATCH="handshake unexpectedly succeeded"
-        else
-            ALPN_MISMATCH="rejected"
-        fi
-        check "DoT rejects non-dot ALPN" \
-            "rejected" \
-            "$ALPN_MISMATCH"
-    fi
-
-    kill "$NUMA_PID" 2>/dev/null || true
-    wait "$NUMA_PID" 2>/dev/null || true
-    rm -f "$DOT_CERT" "$DOT_KEY"
-fi
-sleep 1
-
-# ---- Suite 6: Proxy + DoT coexistence ----
-echo ""
-echo "╔══════════════════════════════════════════╗"
-echo "║  Suite 6: Proxy + DoT Coexistence        ║"
-echo "╚══════════════════════════════════════════╝"
-
-if ! command -v kdig >/dev/null 2>&1 || ! command -v openssl >/dev/null 2>&1; then
-    printf "  ${DIM}skipped — needs kdig + openssl${RESET}\n"
-else
-    DOT_PORT=8853
-    PROXY_HTTP_PORT=8080
-    PROXY_HTTPS_PORT=8443
-    NUMA_DATA=/tmp/numa-integration-data
-
-    # Fresh data dir so we generate a fresh CA for this suite. Path is set
-    # via [server] data_dir in the TOML below, not an env var — numa treats
-    # its config file as the single source of truth for all knobs.
-    rm -rf "$NUMA_DATA"
-    mkdir -p "$NUMA_DATA"
-
-    cat > "$CONFIG" << CONF
-[server]
-bind_addr = "127.0.0.1:$PORT"
-api_port = $API_PORT
-data_dir = "$NUMA_DATA"
-
-[upstream]
-mode = "forward"
-address = "127.0.0.1"
-port = 65535
-
-[cache]
-max_entries = 10000
-
-[blocking]
-enabled = false
-
-[proxy]
-enabled = true
-port = $PROXY_HTTP_PORT
-tls_port = $PROXY_HTTPS_PORT
-tld = "numa"
-bind_addr = "127.0.0.1"
-
-[dot]
-enabled = true
-port = $DOT_PORT
-bind_addr = "127.0.0.1"
-
-[[zones]]
-domain = "dot-test.example"
-record_type = "A"
-value = "10.0.0.1"
-ttl = 60
-CONF
-
-    RUST_LOG=info "$BINARY" "$CONFIG" > "$LOG" 2>&1 &
-    NUMA_PID=$!
-    sleep 4
-
-    if ! kill -0 "$NUMA_PID" 2>/dev/null; then
-        FAILED=$((FAILED + 1))
-        printf "  ${RED}✗${RESET} Startup with proxy + DoT\n"
-        printf "    ${DIM}%s${RESET}\n" "$(tail -5 "$LOG")"
-    else
-        echo ""
-        echo "=== Both listeners ==="
-
-        check "DoT listener bound" \
-            "DoT listening on 127.0.0.1:$DOT_PORT" \
-            "$(grep 'DoT listening' "$LOG")"
-
-        check "HTTPS proxy listener bound" \
-            "HTTPS proxy listening on 127.0.0.1:$PROXY_HTTPS_PORT" \
-            "$(grep 'HTTPS proxy listening' "$LOG")"
-
-        PANIC_COUNT=$(grep -c 'panicked' "$LOG" 2>/dev/null || echo 0)
-        check "No startup panics in log" \
-            "^0$" \
-            "$PANIC_COUNT"
-
-        echo ""
-        echo "=== DoT works with proxy enabled ==="
-
-        # Proxy's build_tls_config runs first and creates the CA in
-        # $NUMA_DATA_DIR. DoT self_signed_tls then loads the same CA and
-        # issues its own leaf cert. One CA trusts both listeners.
-        CA="$NUMA_DATA/ca.pem"
-        KDIG="kdig @127.0.0.1 -p $DOT_PORT +tls +tls-ca=$CA +tls-hostname=numa.numa +time=5 +retry=0"
-
-        check "DoT local zone A (with proxy on)" \
-            "10.0.0.1" \
-            "$($KDIG +short dot-test.example A 2>/dev/null)"
-
-        echo ""
-        echo "=== Proxy TLS works with DoT enabled ==="
-
-        # Proxy cert has SAN numa.numa (auto-added "numa" service). A
-        # successful handshake validates that the proxy's separate
-        # ServerConfig wasn't disturbed by DoT's own cert generation.
-        PROXY_TLS=$(echo "" | openssl s_client -connect "127.0.0.1:$PROXY_HTTPS_PORT" \
-            -servername numa.numa -CAfile "$CA" 2>&1 </dev/null || true)
-        check "Proxy HTTPS TLS handshake succeeds" \
-            "Verify return code: 0 (ok)" \
-            "$PROXY_TLS"
-    fi
-
-    kill "$NUMA_PID" 2>/dev/null || true
-    wait "$NUMA_PID" 2>/dev/null || true
-    rm -rf "$NUMA_DATA"
-fi
 
 # Summary
 echo ""

tests/manual/install-trust-macos.sh

@@ -1,94 +0,0 @@
-#!/usr/bin/env bash
-#
-# Manual macOS CA trust contract test.
-#
-# Mirrors src/system_dns.rs::trust_ca_macos / untrust_ca_macos by running
-# the same `security` shell commands against a fixture cert with a unique
-# CN. Safe to run alongside a production numa install:
-#
-#   - Test cert CN = "Numa Local CA Test <pid-ts>", always strictly longer
-#     than the production CN "Numa Local CA". `security find-certificate -c`
-#     does substring matching, so the test's search for $TEST_CN can never
-#     match the production cert (the search term is longer than the prod CN).
-#   - All deletes use `delete-certificate -Z <hash>`, which only touches the
-#     cert with that exact hash. Production and test certs have different
-#     hashes by construction (different key material), so the delete cannot
-#     reach the production cert even if a CN search somehow returned both.
-#
-# Mutates the System keychain (briefly). Cleans up on success or interrupt.
-# Requires sudo for `security add-trusted-cert` and `delete-certificate`.
-#
-# Usage: ./tests/manual/install-trust-macos.sh
-
-set -euo pipefail
-
-if [[ "$OSTYPE" != darwin* ]]; then
-    echo "This test is macOS-only." >&2
-    exit 1
-fi
-
-GREEN="\033[32m"; RED="\033[31m"; RESET="\033[0m"
-
-# Production constant from src/tls.rs::CA_COMMON_NAME — keep in sync.
-PROD_CN="Numa Local CA"
-KEYCHAIN="/Library/Keychains/System.keychain"
-
-# Notice if production numa is already installed. We proceed regardless —
-# see header for why coexistence is safe (unique CN + by-hash deletion).
-if security find-certificate -c "$PROD_CN" "$KEYCHAIN" >/dev/null 2>&1; then
-    echo "  note: production '$PROD_CN' detected — proceeding alongside (test cert can't touch it)"
-    echo
-fi
-
-# Unique CN ensures the test cert can never collide with production.
-TEST_CN="Numa Local CA Test $$-$(date +%s)"
-FIXTURE_DIR=$(mktemp -d)
-
-cleanup() {
-    # Best-effort: remove any test certs by hash if still present.
-    if security find-certificate -c "$TEST_CN" "$KEYCHAIN" >/dev/null 2>&1; then
-        echo "  cleanup: removing leftover test cert"
-        security find-certificate -c "$TEST_CN" -a -Z "$KEYCHAIN" 2>/dev/null \
-            | awk '/^SHA-1 hash:/ {print $NF}' \
-            | while read -r hash; do
-                sudo security delete-certificate -Z "$hash" "$KEYCHAIN" >/dev/null 2>&1 || true
-            done
-    fi
-    rm -rf "$FIXTURE_DIR"
-}
-trap cleanup EXIT
-
-echo "── generating fixture CA ──"
-openssl req -x509 -newkey rsa:2048 -nodes -days 1 \
-    -keyout "$FIXTURE_DIR/ca.key" \
-    -out    "$FIXTURE_DIR/ca.pem" \
-    -subj   "/CN=$TEST_CN" \
-    -addext "basicConstraints=critical,CA:TRUE" \
-    -addext "keyUsage=critical,keyCertSign,cRLSign" >/dev/null 2>&1
-echo "  CN: $TEST_CN"
-echo
-
-echo "── trust step (mirrors trust_ca_macos) ──"
-sudo security add-trusted-cert -d -r trustRoot -k "$KEYCHAIN" "$FIXTURE_DIR/ca.pem"
-if security find-certificate -c "$TEST_CN" "$KEYCHAIN" >/dev/null 2>&1; then
-    printf "  ${GREEN}✓${RESET} test cert found in keychain\n"
-else
-    printf "  ${RED}✗${RESET} test cert NOT found after add-trusted-cert\n"
-    exit 1
-fi
-echo
-
-echo "── untrust step (mirrors untrust_ca_macos) ──"
-security find-certificate -c "$TEST_CN" -a -Z "$KEYCHAIN" 2>/dev/null \
-    | awk '/^SHA-1 hash:/ {print $NF}' \
-    | while read -r hash; do
-        sudo security delete-certificate -Z "$hash" "$KEYCHAIN" >/dev/null
-    done
-if security find-certificate -c "$TEST_CN" "$KEYCHAIN" >/dev/null 2>&1; then
-    printf "  ${RED}✗${RESET} test cert STILL present after delete (regression)\n"
-    exit 1
-fi
-printf "  ${GREEN}✓${RESET} test cert removed from keychain\n"
-echo
-
-printf "${GREEN}all checks passed${RESET}\n"