diff options
author | 2017-08-18 14:45:20 +0100 | |
---|---|---|
committer | 2017-08-18 14:45:20 +0100 | |
commit | f96cf27193032120ba727316c4057dffed0cbe48 (patch) | |
tree | c76caa111aba281560a3b7db91953e307c06f1c9 /middleware/autopath/autopath.go | |
parent | cc4e4a0626bf8881e2ae8b2c1e52e09c6308e74e (diff) | |
download | coredns-f96cf27193032120ba727316c4057dffed0cbe48.tar.gz coredns-f96cf27193032120ba727316c4057dffed0cbe48.tar.zst coredns-f96cf27193032120ba727316c4057dffed0cbe48.zip |
mw/federation: add federation back as separate mw for k8s (#929)
* mw/federaration
This PR add the federation back as a middleware to keep it more
contained from the main kubernetes code.
It also makes parseRequest less import and pushes this functionlity down
in the k.Entries. This minimizes (or tries to) the importance for the
qtype in the query. In the end the qtype checking should only happen
in ServeDNS - but for k8s this might proof difficult.
Numerous other cleanup in code and kubernetes tests.
* up test coverage
Diffstat (limited to 'middleware/autopath/autopath.go')
-rw-r--r-- | middleware/autopath/autopath.go | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/middleware/autopath/autopath.go b/middleware/autopath/autopath.go index d3a8c84d2..ec6bb674b 100644 --- a/middleware/autopath/autopath.go +++ b/middleware/autopath/autopath.go @@ -1,8 +1,6 @@ -package autopath - /* -Autopath is a hack; it shortcuts the client's search path resolution by performing -these lookups on the server... +Autopath package implement autopathing. This is a hack; it shortcuts the +client's search path resolution by performing these lookups on the server... The server has a copy (via AutoPathFunc) of the client's search path and on receiving a query it first establish if the suffix matches the FIRST configured @@ -31,6 +29,7 @@ func (m Middleware ) AutoPath(state request.Request) []string { return []string{"first", "second", "last", ""} } */ +package autopath import ( "log" @@ -108,7 +107,6 @@ func (a *AutoPath) ServeDNS(ctx context.Context, w dns.ResponseWriter, r *dns.Ms if err != nil { // Return now - not sure if this is the best. We should also check if the write has happened. return rcode, err - } if i == 0 { firstReply = nw.Msg |