rm-hull / dot-block

51%

LAST BUILD ON BRANCH main

branch: SELECT

CHANGE BRANCH

x

Sync Branches

• No branch selected
• chore/correct-package-name
• chore/slog
• chore/update-deps
• dependabot/github_actions/actions/cache-5
• dependabot/github_actions/actions/checkout-6
• dependabot/github_actions/actions/upload-artifact-6
• dependabot/github_actions/golangci/golangci-lint-action-9
• dependabot/github_actions/mikepenz/action-junit-report-6
• dependabot/go_modules/github.com/axiomhq/hyperloglog-0.2.6
• dependabot/go_modules/github.com/getsentry/sentry-go-0.37.0
• dependabot/go_modules/github.com/getsentry/sentry-go-0.38.0
• dependabot/go_modules/github.com/getsentry/sentry-go-0.39.0
• dependabot/go_modules/github.com/getsentry/sentry-go-0.40.0
• dependabot/go_modules/github.com/getsentry/sentry-go/gin-0.37.0
• dependabot/go_modules/github.com/getsentry/sentry-go/gin-0.38.0
• dependabot/go_modules/github.com/getsentry/sentry-go/gin-0.39.0
• dependabot/go_modules/github.com/getsentry/sentry-go/gin-0.40.0
• dependabot/go_modules/github.com/miekg/dns-1.1.69
• dependabot/go_modules/github.com/rm-hull/godx-0.0.10
• dependabot/go_modules/github.com/rm-hull/godx-0.0.7
• dependabot/go_modules/github.com/rm-hull/godx-0.0.8
• dependabot/go_modules/github.com/spf13/cobra-1.10.2
• dependabot/go_modules/github.com/tavsec/gin-healthcheck-1.7.12
• dependabot/go_modules/golang.org/x/crypto-0.44.0
• dependabot/go_modules/golang.org/x/crypto-0.45.0
• dependabot/go_modules/golang.org/x/crypto-0.46.0
• dependabot/go_modules/golang.org/x/net-0.47.0
• dependabot/go_modules/golang.org/x/net-0.48.0
• feat/cache-reaper
• feat/dns-listener
• feat/reload-blocklist
• feat/sentry-integration
• feat/top-domains
• feat/upstream-latency
• feat/upstream-ttl-metrics
• main
• refactor/error-wrap
• refactor/handle-multiple-questions
• testing/integration

Build # 20411149028

Build Type

push

github

Committed by web-flow

Commit Message

feat: add support for regular DNS on port 53 (#48)

* feat: add support for regular DNS on port 53

*   Implement standard UDP and TCP DNS listeners alongside the existing
    DoT service.
*   Add a new `--dns-port` flag (defaulting to 53) to allow
    configuration of the regular DNS interface.
*   Refactor server lifecycle management using `errgroup` to
    concurrently run HTTP, UDP DNS, TCP DNS, and DoT listeners.
*   Update `GEMINI.md` and `README.md` with instructions for testing
    both regular DNS and DoT in production and development modes.

```mermaid
graph TD
    Main[App RunServer] --> Group[errgroup]
    Group --> HTTP[HTTP Server:80]
    Group --> UDP[UDP DNS:53]
    Group --> TCP[TCP DNS:53]
    Group --> DoT[DoT Server:853]

    HTTP --> ACME[ACME/Metrics/Health]
    UDP --> Dispatcher[DNS Dispatcher]
    TCP --> Dispatcher
    DoT --> Dispatcher
    Dispatcher --> Cache[DNS Cache]
    Dispatcher --> Forwarder[Upstream Forwarder]
```

* refactor: unify DNS and DoT server startup

Refactor the server initialization to reduce duplication and improve
readability.

- Consolidate **DoT** server logic into a single goroutine.
- Use integer variables for port logging to avoid redundant formatting.
- Rename the `errgroup` variable from `g` to `group`.

```mermaid
graph TD
    A[RunServer] --> B{app.DevMode?}
    B -->|Yes| C[Set Dev Ports]
    B -->|No| D[Set Standard Ports]
    C & D --> E[Start HTTP, UDP, TCP Servers]
    C & D --> F[Start DoT Goroutine]
    F --> G{app.DevMode?}
    G -->|Yes| H[Network: tcp]
    G -->|No| I[Network: tcp-tls + Cert Config]
```

* feat: add dot-port flag and simplify port logic

- Add a new `--dot-port` flag to customize the DoT listening port.
- Centralize port initialization in `main.go` to separate CLI concerns
  from internal application logic.
- Improve `DEV_MODE` behavior by allowing explicit port overrides via
  command-line arguments.

Run Details

0 of 57 new or added lines in 2 files covered. (0.0%)

17 existing lines in 1 file now uncovered.

404 of 797 relevant lines covered (50.69%)

0.54 hits per line