aboutsummaryrefslogtreecommitdiff
path: root/plugin/secondary (follow)
AgeCommit message (Collapse)AuthorFilesLines
2018-07-28Fix ineffassign (#1959)Gravatar Yong Tang 1-2/+1
* Fix ineffassign This fix tries to fix ineffassign, as was reported in: https://goreportcard.com/report/github.com/coredns/coredns#ineffassign Signed-off-by: Yong Tang <yong.tang.github@outlook.com> * Update setup.go Revert this one change, so this can be merged.
2018-07-19Clean up tests logging (#1979)Gravatar Miek Gieben 1-0/+5
* Clean up tests logging This cleans up the travis logs so you can see the failures better. Older tests in tests/ would call log.SetOutput(ioutil.Discard) in a haphazard way. This add log.Discard and put an `init` function in each package's dir (no way to do this globally). The cleanup in tests/ is clear. All plugins also got this init function to have some uniformity and kill any (future) logging there in the tests as well. There is a one-off in pkg/healthcheck because that does log. Signed-off-by: Miek Gieben <miek@miek.nl> * bring back original log_test.go Signed-off-by: Miek Gieben <miek@miek.nl> * suppress logging here as well Signed-off-by: Miek Gieben <miek@miek.nl>
2018-06-15plugin/forward: add REFUSED test (#1878)Gravatar Miek Gieben 1-1/+1
add a test to see if we copy the rcode correctly. Some minor cleanup in import ordering and renaming NewUpstream to New as we already are in the upstream package.
2018-06-12plugin/file: fix local CNAME lookup (#1866)Gravatar Miek Gieben 1-1/+1
* 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.
2018-02-16plugin/auto/file/secondary: Use new upstream resolver (#1534)Gravatar Chris O'Haver 2-10/+7
* move file, auto, secondary to new upstream * include context in request
2018-02-08Add OWNERS file (#1486)Gravatar Miek Gieben 1-0/+6
This should have everyone, but the process was quite manual. The rename from middleware -> plugin also meant I had to do some extra digging on who actually submitted the PR. I also double checked the current list of people with commit access. Every plugin now has an OWNERS, except *reverse*. I'll file a bug for that.
2018-01-23plugin/secondary: fix a bunch of things and tests (#1406)Gravatar Miek Gieben 1-0/+4
Fix the error handling. Log when we have an error during any of the transfer state. And if there isn't an error transfer the zones. Also fix the tests in test/ so we, at least, check the initial transfer. Update the docs to show more about how errors are handled. Ref #1400
2018-01-10Doc (#1369)Gravatar Miek Gieben 1-1/+1
* Constent atx headers * Regen manual pages
2018-01-04Manual pages (#1346)Gravatar Miek Gieben 1-2/+12
* Add manual pages Generate manual pages from the README and extend README with Name and Description sections. The generation requires 'ronn' which may not be available. Just check in all generated manual pages.
2017-12-13Moving TransferParse from file to its own package (#1286)Gravatar Brad Beam 1-1/+2
* Moving TransferParse from file to its own package * Adding tests for parse
2017-09-14Remove the word middleware (#1067)Gravatar Miek Gieben 4-0/+237
* Rename middleware to plugin first pass; mostly used 'sed', few spots where I manually changed text. This still builds a coredns binary. * fmt error * Rename AddMiddleware to AddPlugin * Readd AddMiddleware to remain backwards compat