diff options
| author |  Miek Gieben <miek@miek.nl>
| 2018-06-12 14:54:37 +0100 |
|---|---|---|
| committer |  GitHub <noreply@github.com>
| 2018-06-12 14:54:37 +0100 |
| commit | 26c41a0c177fbe89f1f81214634aed49790581f4 (patch) | |
| tree | 7e957105a4256d875c4985a75a3c9fb92b49234d /request | |
| parent | 6e466d509281953fdf0209a5b50611e89b4689ae (diff) | |
| download | coredns-26c41a0c177fbe89f1f81214634aed49790581f4.tar.gz coredns-26c41a0c177fbe89f1f81214634aed49790581f4.tar.zst coredns-26c41a0c177fbe89f1f81214634aed49790581f4.zip | |
plugin/file: fix local CNAME lookup (#1866)
* plugin/file: fix local CNAME lookup
Issue #1864 explains it will, when we serve the child zone as well we
should just recursive into ourself (upstream self). Thus relax the
IsSubDomain check in file/lookup.go and just query (even if the query
will hit a remote server).
I've looped over all other plugins that do something similar (CNAME
resolving) and they didn't do the IsSubDomain check; therefor I've
removed it from *file* as well.
Added test in file_upstream_test that shows this failed before but now
results in a reply.
Fixes #1864
* self does not need to be exported
* Fix test
We don't know if we had a valid reply. Check this.
Diffstat (limited to 'request')
0 files changed, 0 insertions, 0 deletions