Upgrade miekg/dns to 1.1.8
Signed-off-by: Miek Gieben <miek@miek.nl>

Various plugins still had this documented, the setup functions still
allow it (which is fine), but update the docs that this arguments isn't
there any more.
Signed-off-by: Miek Gieben <miek@miek.nl>

* cache failures

* use ServerError

This formats and simplifies all code by running gofmt -w -s on all Go
files.
Signed-off-by: Miek Gieben <miek@miek.nl>

* WIP: travis changes

Simplify Travis so it fails less often.

We don't need docker any more, let alone trusty and sudo, so simplifies
this, to just os: linux.
Signed-off-by: Miek Gieben <miek@miek.nl>

* This?
Signed-off-by: Miek Gieben <miek@miek.nl>

Signed-off-by: Miek Gieben <miek@miek.nl>

each readme starts with the plugin's name as a header, this needs to be
chopped off to provide a good manual page.
Signed-off-by: Miek Gieben <miek@miek.nl>

* fix date on 150 release notes
* fix syntax in route53 plugin
Signed-off-by: Miek Gieben <miek@miek.nl>

Mmark recently became able to create manual pages. This removed the
dependency on 'ronn' and just uses mmark (Go program).

Re-hookup Makefile.doc to generate the correct header mmark needs to
see and regenate them all.

Spot checking a few pages suggest they look good and actually better
than rendered with ronn, esp. lists in lists.

Fixes #2757
Signed-off-by: Miek Gieben <miek@miek.nl>

*proxy* is removed, so this is moot
Signed-off-by: Miek Gieben <miek@miek.nl>

Signed-off-by: Miek Gieben <miek@miek.nl>

Signed-off-by: Miek Gieben <miek@miek.nl>

* remove mips from manifest

* derp

* add comment

* Disable resync by default

* Document default resync

Signed-off-by: Miek Gieben <miek@miek.nl>

* release: add 1.5.0 release notes

Add the release notes to the notes/ directory to prepare for a release.
Signed-off-by: Miek Gieben <miek@miek.nl>

* Update notes/coredns-1.5.0.md
Co-Authored-By: miekg <miek@miek.nl>

* More
Signed-off-by: Miek Gieben <miek@miek.nl>

* Update notes/coredns-1.5.0.md
Co-Authored-By: miekg <miek@miek.nl>

fully remove proxy and make it completely external.
Signed-off-by: Miek Gieben <miek@miek.nl>

Small tweaks, tell we're a DNS server.
Signed-off-by: Miek Gieben <miek@miek.nl>

* Fully deprecate NO_RELOAD
Signed-off-by: Xiao An <hac@zju.edu.cn>

* Fully deprecate TIMEOUT
Signed-off-by: Xiao An <hac@zju.edu.cn>

* Update CI tests to adapt to the deprecation of TIMEOUT
Signed-off-by: Xiao An <hac@zju.edu.cn>

* Add documents for directive transfer in plugin auto
Signed-off-by: Xiao An <hac@zju.edu.cn>

* plugin/chaos: add default list of authors

Add a owners_generate.go that generates a Owners variables for use in
the chaos plugin.

Add a default list of authors in the authors.bind CH zone. When doing a
query this now returns:

~~~ sh
% dig authors.bind TXT CH

;; ->>HEADER<<- opcode: QUERY, status: NOERROR, id: 5456
;; flags: qr rd; QUERY: 1, ANSWER: 22, AUTHORITY: 0, ADDITIONAL: 1
;; WARNING: recursion requested but not available

;; OPT PSEUDOSECTION:
; EDNS: version: 0, flags:; udp: 4096
;; QUESTION SECTION:
;authors.bind.			CH	TXT

;; ANSWER SECTION:
authors.bind.		0	CH	TXT	"bradbeam"
authors.bind.		0	CH	TXT	"chrisohaver"
authors.bind.		0	CH	TXT	"dilyevsky"
authors.bind.		0	CH	TXT	"ekleiner"
authors.bind.		0	CH	TXT	"fastest963"
authors.bind.		0	CH	TXT	"fturib"
authors.bind.		0	CH	TXT	"greenpau"
authors.bind.		0	CH	TXT	"grobie"
authors.bind.		0	CH	TXT	"inigohu"
authors.bind.		0	CH	TXT	"isolus"
authors.bind.		0	CH	TXT	"johnbelamaric"
authors.bind.		0	CH	TXT	"miekg"
authors.bind.		0	CH	TXT	"nchrisdk"
authors.bind.		0	CH	TXT	"nitisht"
authors.bind.		0	CH	TXT	"pmoroney"
authors.bind.		0	CH	TXT	"rajansandeep"
authors.bind.		0	CH	TXT	"rdrozhdzh"
authors.bind.		0	CH	TXT	"rtreffer"
authors.bind.		0	CH	TXT	"stp-ip"
authors.bind.		0	CH	TXT	"superq"
authors.bind.		0	CH	TXT	"varyoo"
authors.bind.		0	CH	TXT	"yongtang"
~~~

This was hard to do previously as we didn't hardcode this in the source,
but now with OWNERS files we can just generate this list.

Privacy wise this isn't worse than being listed in OWNERS file in the
first place. And it's a nice hat tip to the people making CoreDNS
better.
Signed-off-by: Miek Gieben <miek@miek.nl>

* Sticklet bot comments
Signed-off-by: Miek Gieben <miek@miek.nl>

* [plugin/route53]: Do not return NXDOMAIN where it should be NODATA.
Signed-off-by: Dmitry Ilyevskiy <dmitry.ilyevskiy@getcruise.com>

* Fix bad merge.
Signed-off-by: Dmitry Ilyevskiy <dmitry.ilyevskiy@getcruise.com>

Rename connTimeout to graceTimeout to be more instructive on it's use,
some whitespace for imports lines.
Signed-off-by: Miek Gieben <miek@miek.nl>

A 'make -f Makefile.doc clean all'.
Signed-off-by: Miek Gieben <miek@miek.nl>

I can't find what's this code should be doing and it looks like it's
not doing anything.
Signed-off-by: Miek Gieben <miek@miek.nl>

* plugin/cancel: add context cancelation plugin

Per review comments on #2704, move this into a plugin that gets called.
Add the most minimal plugin, tests and documenation.
Signed-off-by: Miek Gieben <miek@miek.nl>

* plugin/cache: add timeout option

review feedback: add option to set custom timeout.
Signed-off-by: Miek Gieben <miek@miek.nl>

* spelling
Signed-off-by: Miek Gieben <miek@miek.nl>

Rework this a little to be more inline with the other READMEs
Signed-off-by: Miek Gieben <miek@miek.nl>

* - add an option for block profiling to plugin pprof

* - move option block into nested block

* Remove context.Context from request.Request

This removes the context from request.Request and makes all the changes
in the code to make it compile again. It's all mechanical. It did
unearth some weirdness in that the context was kept in handler structs
which may cause havoc with concurrently handling of requests.

Fixes #2721
Signed-off-by: Miek Gieben <miek@miek.nl>

* Make test compile
Signed-off-by: Miek Gieben <miek@miek.nl>

Add another test case for impossible DNS messages which should not be
cached. This adds a check for a message that denies its own existence.

Fixes #2724.
Signed-off-by: Miek Gieben <miek@miek.nl>

* Don't double report metrics on error

When there is an error use a different function to report the metrics,
in case the plugin chain handled the request the metrics are already
reported.

Fixes: #2717
Signed-off-by: Miek Gieben <miek@miek.nl>

* Compile again
Signed-off-by: Miek Gieben <miek@miek.nl>

* more
Signed-off-by: Miek Gieben <miek@miek.nl>

* Remove server addr from the context

This was added twice, just leave the server which also holds the
address.

Conflicts with #2719 but should be easy to fix.
Signed-off-by: Miek Gieben <miek@miek.nl>

* doesn't need server context
Signed-off-by: Miek Gieben <miek@miek.nl>

The server handles this case no need to also do it in the log plugin.

Means DefaultErrorFunc can be private to the dnsserver and is now
renamed to just errorFunc

Fixes: #2715
Signed-off-by: Miek Gieben <miek@miek.nl>

ErrorMessage was a very shallow function that didn't add anything.
Remove it and remove its usage.
Signed-off-by: Miek Gieben <miek@miek.nl>