Skip to content

fix: reject invalid any and local config#8133

Merged
yongtang merged 1 commit into
coredns:masterfrom
immanuwell:fix-any-local-config-validation
May 31, 2026
Merged

fix: reject invalid any and local config#8133
yongtang merged 1 commit into
coredns:masterfrom
immanuwell:fix-any-local-config-validation

Conversation

@immanuwell

Copy link
Copy Markdown
Contributor

1. Why is this pull request needed and what does it do?

any and local were accepting junk config instead of failing fast.

Repro before this patch:

. {
    any example.org
    local { foo }
}

Both cases are reachable in normal startup config parsing, no weird setup needed. They should error, but they slipped through. This patch makes both plugins validate args/block content, and adds tests.

2. Which issues (if any) are related?

No direct issue found. Kinda the same lane as #8120, #8121, #8119 and #8128.

3. Which documentation changes (if any) need to be made?

None.

4. Does this introduce a backward incompatible change or deprecation?

Only for invalid config. Valid configs stay the same.

Signed-off-by: immanuwell <pchpr.00@list.ru>
@yongtang yongtang merged commit 4c07a28 into coredns:master May 31, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants