rm-hull / dot-block / 27231114288

61%

push

github

feat: implement DNS-over-HTTPS (DoH) support (#150)

* feat: implement DNS-over-HTTPS (DoH) support

Added DoH endpoint (`/dns-query`) to the HTTP server to support both GET
 and POST requests. This allows clients to perform DNS lookups over
HTTPS using standard base64-encoded DNS wire format.

- Added `SourceDoH` to `DNSSource`.
- Created `DoHHandler` to parse, resolve, and serialize DNS messages.
- Implemented `doHResponseWriter` to interface with the existing
`DNSDispatcher`.

* refactor: improve DoH request handling and IP parsing

- Update `NewDoHResponseWriter` to accept IP string instead of
`*http.Request` for better testability.
- Simplify DoH handler logic for decoding DNS messages from POST body
and GET parameters.
- Add support for binary DNS message files in `test.http`.
- Apply minor style fixes to `internal/app.go`.

* chore: iterative updates from Gemini based on PR comment

I have added integration tests for both the `GET` and `POST` `/dns-query` endpoints to `internal/integration_test.go`.

### Changes Made:
- **Imports:** Added `bytes`, `encoding/base64`, `io`, and `net/http` to support HTTP requests and DNS message encoding.
- **Configuration:** Updated the test `App` configuration to use a fixed `HttpPort` (8080) to ensure the DoH tests have a reliable endpoint.
- **Test Cases:** Added four new test cases to the `TestIntegration_DNSFunctionality` suite:
    - `DoH GET - Good Domain`
    - `DoH GET - Blocked Domain`
    - `DoH POST - Good Domain`
    - `DoH POST - Blocked Domain`
- **Test Logic:** Implemented the logic to:
    - Pack DNS messages.
    - Perform `GET` requests with base64-encoded DNS queries.
    - Perform `POST` requests with `application/dns-message` bodies.
    - Unpack the response and validate that domains are correctly allowed or blocked (verified by the presence of the `ns.blocked.local.` SOA record).

### Verification:
I ran the integration tests using `go test -v ./internal`, and all tests passed, including ... (continued)

43 of 93 new or added lines in 2 files covered. (46.24%)

1250 of 2071 relevant lines covered (60.36%)

852.06 hits per line