Files
container/Tests/DNSServerTests/HostTableResolverTest.swift
T
Rado Kubiak | Rado x Tech 745cc1813e Fix container DNS resolution broken by AAAA/IPv6 NXDOMAIN handling (#786)
- In Alpine Linux containers (commonly used as Docker base images),
  standard DNS resolution is provided by **musl**, a lightweight C
  standard library (libc). Musl implements DNS lookups via
  `getaddrinfo()`, which queries AAAA (IPv6) records first.
- Problem: DNS did not work correctly **inside containers**. Any
  system command attempting to resolve hostnames
  (e.g., `ping dynamodb-admin`) **failed** when the DNS server
  responded NXDOMAIN for AAAA records, even if A (IPv4) records
  existed. Explicitly forcing IPv4 (`ping -4dynamodb-admin`) worked
  correctly, showing the issue is specific to musl’s IPv6-first behavior.
- Consequence: In IPv4-only environments, Alpine-based containers
  cannot resolve hostnames using standard tools or libraries.
  Applications relying on `getaddrinfo()` fail with ENOTFOUND,
  breaking networking and inter-container communication.
- Root cause:  Following RFC 8305 / RFC 6724, musl treats NXDOMAIN
  for AAAA as “hostname does not exist” and does not fallback to A
  (IPv4) records.
- Fix: The Apple Container DNS engine now behaves as follows:
  * If an **A record exists**, AAAA queries return **NOERROR with empty
    answer (NODATA)**.
  * If neither **A nor AAAA** exist, NXDOMAIN is returned.
  This ensures that Alpine-based containers in IPv4-only networks can
  correctly resolve hostnames inside containers without modifying
  container images or application code.
2025-10-30 17:03:17 -07:00

132 lines
4.5 KiB
Swift

//===----------------------------------------------------------------------===//
// Copyright © 2025 Apple Inc. and the container project authors.
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
//
// https://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.
//===----------------------------------------------------------------------===//
import DNS
import Testing
@testable import DNSServer
struct HostTableResolverTest {
@Test func testUnsupportedQuestionType() async throws {
guard let ip = IPv4("1.2.3.4") else {
throw DNSResolverError.serverError("cannot create IP address in test")
}
let handler = HostTableResolver(hosts4: ["foo": ip])
let query = Message(
id: UInt16(1),
type: .query,
questions: [
Question(name: "foo", type: .mailExchange)
])
let response = try await handler.answer(query: query)
#expect(.notImplemented == response?.returnCode)
#expect(1 == response?.id)
#expect(.response == response?.type)
#expect(1 == response?.questions.count)
#expect(0 == response?.answers.count)
}
@Test func testAAAAQueryReturnsNoDataWhenARecordExists() async throws {
guard let ip = IPv4("1.2.3.4") else {
throw DNSResolverError.serverError("cannot create IP address in test")
}
let handler = HostTableResolver(hosts4: ["foo": ip])
let query = Message(
id: UInt16(1),
type: .query,
questions: [
Question(name: "foo", type: .host6)
])
let response = try await handler.answer(query: query)
// AAAA queries should return NODATA (noError with empty answers) when A record exists
// to avoid musl libc issues where NXDOMAIN causes complete DNS resolution failure
#expect(.noError == response?.returnCode)
#expect(1 == response?.id)
#expect(.response == response?.type)
#expect(1 == response?.questions.count)
#expect(0 == response?.answers.count)
}
@Test func testAAAAQueryReturnsNilWhenHostDoesNotExist() async throws {
guard let ip = IPv4("1.2.3.4") else {
throw DNSResolverError.serverError("cannot create IP address in test")
}
let handler = HostTableResolver(hosts4: ["foo": ip])
let query = Message(
id: UInt16(1),
type: .query,
questions: [
Question(name: "bar", type: .host6)
])
let response = try await handler.answer(query: query)
// AAAA queries for non-existent hosts should return nil (which becomes NXDOMAIN)
#expect(nil == response)
}
@Test func testHostNotPresent() async throws {
guard let ip = IPv4("1.2.3.4") else {
throw DNSResolverError.serverError("cannot create IP address in test")
}
let handler = HostTableResolver(hosts4: ["foo": ip])
let query = Message(
id: UInt16(1),
type: .query,
questions: [
Question(name: "bar", type: .host)
])
let response = try await handler.answer(query: query)
#expect(nil == response)
}
@Test func testHostPresent() async throws {
guard let ip = IPv4("1.2.3.4") else {
throw DNSResolverError.serverError("cannot create IP address in test")
}
let handler = HostTableResolver(hosts4: ["foo": ip])
let query = Message(
id: UInt16(1),
type: .query,
questions: [
Question(name: "foo", type: .host)
])
let response = try await handler.answer(query: query)
#expect(.noError == response?.returnCode)
#expect(1 == response?.id)
#expect(.response == response?.type)
#expect(1 == response?.questions.count)
#expect("foo" == response?.questions[0].name)
#expect(.host == response?.questions[0].type)
#expect(1 == response?.answers.count)
let answer = response?.answers[0] as? HostRecord<IPv4>
#expect(IPv4("1.2.3.4") == answer?.ip)
}
}