mirror of
https://github.com/apple/container.git
synced 2026-07-15 05:57:01 +00:00
745cc1813e
- 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.
132 lines
4.5 KiB
Swift
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)
|
|
}
|
|
}
|