See https://github.com/coredns/coredns/pull/3209#issuecomment-525016357
for the previous issues, also see https://github.com/Azure/go-autorest/issues/414
for some further background.

Basically go mod has some issue when multiple tags exists (as main module and submodule)
within the same repo. That was fixed in go-autorest very recently, though
azure-sdk-for-go has not been fully updated yet.

This fix is a temporarily one with `replace` fix. Once azure-sdk-for-go
is updated then we could drop the `replace fix`
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>

Minor textual cleanup.

* use all local IPs

* mult/bind ips

* gofmt + boundIPs fix

* fix no matching endpoint case

* don't duplicate NS records in answer

* fix answer dedup

* fix comment

* add multi local ip test case

OWNERS file was padded, remove empty lines. Fix code block in README,
and set codeblock type to 'corefile' without the upper case C.
Signed-off-by: Miek Gieben <miek@miek.nl>

* Add plugin ACL for source ip filtering
Signed-off-by: An Xiao <hac@zju.edu.cn>

* Allow all arguments to be optional and support multiple qtypes in a single policy
Signed-off-by: An Xiao <hac@zju.edu.cn>

* Add newline before third party imports
Signed-off-by: An Xiao <hac@zju.edu.cn>

* Use camel instead of underscore in method name
Signed-off-by: An Xiao <hac@zju.edu.cn>

* Start with an upper case letter in t.Errorf()
Signed-off-by: An Xiao <hac@zju.edu.cn>

* Use the qtype parse logic in miekg/dns
Signed-off-by: An Xiao <hac@zju.edu.cn>

* Use third party trie implementation as the ip filter
Signed-off-by: An Xiao <hac@zju.edu.cn>

* Update based on rdrozhdzh's comment
Signed-off-by: An Xiao <hac@zju.edu.cn>

* Change the type of action to int
Signed-off-by: An Xiao <hac@zju.edu.cn>

* Add IPv6 support
Signed-off-by: An Xiao <hac@zju.edu.cn>

* Update plugin.cfg
Signed-off-by: An Xiao <hac@zju.edu.cn>

* Remove file functionality
Signed-off-by: An Xiao <hac@zju.edu.cn>

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

* Update README
Signed-off-by: Xiao An <hac@zju.edu.cn>

* remove comments
Signed-off-by: Xiao An <hac@zju.edu.cn>

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

* Update dependency
Signed-off-by: Xiao An <hac@zju.edu.cn>

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

* Update test
Signed-off-by: Xiao An <hac@zju.edu.cn>

* Add OWNERS
Signed-off-by: Xiao An <hac@zju.edu.cn>

* Refactor shouldBlock and skip useless check
Signed-off-by: Xiao An <hac@zju.edu.cn>

* Introduce ActionNone
Signed-off-by: Xiao An <hac@zju.edu.cn>

* Update label name
Signed-off-by: Xiao An <hac@zju.edu.cn>

* Avoid capitalizing private types
Signed-off-by: Xiao An <hac@zju.edu.cn>

Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.23.8 to 1.23.13.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Changelog](https://github.com/aws/aws-sdk-go/blob/master/CHANGELOG.md)
- [Commits](https://github.com/aws/aws-sdk-go/compare/v1.23.8...v1.23.13)
Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Fix for 3235. Adds go.sum and go.mod to .dockerignore to avoid permission issues during docker build process. (#3236)

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

* Notes update for the release.

acl might not make tomorrow (plan to release then), so remove it for
now. Add other PRs that got merged and flesh it out some more.
Signed-off-by: Miek Gieben <miek@miek.nl>

* Remove xfr highlight for k8s
Signed-off-by: Miek Gieben <miek@miek.nl>

* Small tweaks while preparing the release
Signed-off-by: Miek Gieben <miek@miek.nl>

Add the new plugins ones: clouddns and sign. Remove federation from it.
Signed-off-by: Miek Gieben <miek@miek.nl>

* startup: add logo

As discussed in #3225, lets add a little logo. This PR incorperates this
in the startup text. It also removes the logging output as this outputs
identical lines of text, but for no clear reason. This means --quiet now
means "no output on startup at all".

Currently it looks like this:

~~~
.:1053
   ______                ____  _   _______
  / ____/___  ________  / __ \/ | / / ___/	CoreDNS-1.6.2
 / /   / __ \/ ___/ _ \/ / / /  |/ /\__ \ 	linux/amd64, go1.12.9,
/ /___/ /_/ / /  /  __/ /_/ / /|  /___/ /
\____/\____/_/   \___/_____/_/ |_//____/
~~~

We have 2 lines extra on the right if we need to print more (no ideas
come to mind currently).
Signed-off-by: Miek Gieben <miek@miek.nl>

* Add distinct marker for grep and cut purposes
Signed-off-by: Miek Gieben <miek@miek.nl>

* plugin/file: rework outgoing axfr
Signed-off-by: Miek Gieben <miek@miek.nl>

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

* Actually properly test xfr
Signed-off-by: Miek Gieben <miek@miek.nl>

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

* plugin/sign: a plugin that signs zones

Sign is a plugin that signs zone data (on disk). The README.md details
what exactly happens to should be accurate related to the code.

Signs are signed with a CSK, resigning and first time signing is all
handled by *sign* plugin.

Logging with a test zone looks something like this:

~~~ txt
[INFO] plugin/sign: Signing "miek.nl." because open plugin/sign/testdata/db.miek.nl.signed: no such file or directory
[INFO] plugin/sign: Signed "miek.nl." with key tags "59725" in 11.670985ms, saved in "plugin/sign/testdata/db.miek.nl.signed". Next: 2019-07-20T15:49:06.560Z
[INFO] plugin/file: Successfully reloaded zone "miek.nl." in "plugin/sign/testdata/db.miek.nl.signed" with serial 1563636548
[INFO] plugin/sign: Signing "miek.nl." because resign was: 10m0s ago
[INFO] plugin/sign: Signed "miek.nl." with key tags "59725" in 2.055895ms, saved in "plugin/sign/testdata/db.miek.nl.signed". Next: 2019-07-20T16:09:06.560Z
[INFO] plugin/file: Successfully reloaded zone "miek.nl." in "plugin/sign/testdata/db.miek.nl.signed" with serial 1563637748
~~~
Signed-off-by: Miek Gieben <miek@miek.nl>

* Adjust readme and remove timestamps
Signed-off-by: Miek Gieben <miek@miek.nl>

* Comment on the newline
Signed-off-by: Miek Gieben <miek@miek.nl>

* Update plugin/sign/README.md
Co-Authored-By: Michael Grosser <development@stp-ip.net>

We're not making any backward incompat changes so this can just be 1.6.3
Signed-off-by: Miek Gieben <miek@miek.nl>

* plugin/etcd: clarify it's not a generic backend

Clarify the docs that this is not a generic backend.

Wrap it on 80 columns as well.
Signed-off-by: Miek Gieben <miek@miek.nl>

* Update plugin/etcd/README.md
Co-Authored-By: Chris O'Haver <cohaver@infoblox.com>

* Update plugin/etcd/README.md
Co-Authored-By: Chris O'Haver <cohaver@infoblox.com>

journald timestamps, kubernetes timestamps, syslog timestamps. It seems
silly to add our own timestamps to the logging output as these external
ones *also* do it. Only when just running coredns this might be weird.

Remove the timestamping from pkg/log.

Remove test that tested for this.

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

This is needed as well to make it compile cleanly. Follow up to #3214.

Manually removed github.com/coreos/etcd/ from go.mod
Signed-off-by: Miek Gieben <miek@miek.nl>

This moves the entire etcd to go.etcd.io, so that it compiles cleanly
when using bazel.

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

Say once that we've sent notifies, instead of for every upstream
primary.
Signed-off-by: Miek Gieben <miek@miek.nl>

* release: initial release notes

Core of the release notes.
Signed-off-by: Miek Gieben <miek@miek.nl>

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

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

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

* bump k8s.io/client-go and azure/go-autorest

* bump azure

* bump azure 2

https://help.github.com/en/articles/adding-a-security-policy-to-your-repository
https://help.github.com/en/articles/about-maintainer-security-advisoriesSigned-off-by: Chris Aniszczyk <caniszczyk@gmail.com>

Signed-off-by: hwdef <hwdef97@gmail.com>

* Fuzz: fix forward build
Signed-off-by: Miek Gieben <miek@miek.nl>

* Flag not released yet, wget quiet
Signed-off-by: Miek Gieben <miek@miek.nl>

Bumps [github.com/coreos/etcd](https://github.com/coreos/etcd) from 3.3.13+incompatible to 3.3.15+incompatible.
- [Release notes](https://github.com/coreos/etcd/releases)
- [Changelog](https://github.com/etcd-io/etcd/blob/master/CHANGELOG-3.4.md)
- [Commits](https://github.com/coreos/etcd/compare/v3.3.13...v3.3.15)
Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.23.3 to 1.23.8.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Changelog](https://github.com/aws/aws-sdk-go/blob/master/CHANGELOG.md)
- [Commits](https://github.com/aws/aws-sdk-go/compare/v1.23.3...v1.23.8)
Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

This reloader didn't close the openened file handle. Add a close. Can't
use `defer` because this is in a endless loop.
Signed-off-by: Miek Gieben <miek@miek.nl>

Bumps [google.golang.org/api](https://github.com/google/google-api-go-client) from 0.8.0 to 0.9.0.
- [Release notes](https://github.com/google/google-api-go-client/releases)
- [Changelog](https://github.com/googleapis/google-api-go-client/blob/master/CHANGES.md)
- [Commits](https://github.com/google/google-api-go-client/compare/v0.8.0...v0.9.0)
Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Upgrade to new version and add if-not-exists to the cmdline
Signed-off-by: Miek Gieben <miek@miek.nl>

* plugin/file: respond correctly to IXFR message

Respond with a sing SOA record to an IXFR request if the SOA serials
match.

The added test fails on the current code with:

~~~
=== RUN   TestIxfrResponse
--- FAIL: TestIxfrResponse (0.00s)
    secondary_test.go:122: Expected answer section with single RR
FAIL
exit status 1
~~~

And obviously passes with the new code. This should cut down on the
weird number of zone transfers that I was seeing. At some point IXFR
support might be cool.

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

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

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

This can't be used in its current form; revert the entire PR.

Revert "fuzzing: allow setup function to be called (#3175)"

This reverts commit 62451fd3.

* plugin/forward: add fuzzing

Add fuzz.go for forward
Signed-off-by: Miek Gieben <miek@miek.nl>

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

* fuzz: use gofuzz build tag instead of fuzz

Since go-fuzz does not support Go modules yet, vendor dependencies.
Signed-off-by: Muhammad Falak R Wani <falakreyaz@gmail.com>

* fuzz: avoid vendoring code for go-fuzz
Signed-off-by: Muhammad Falak R Wani <falakreyaz@gmail.com>

TCPServer was only used in secondary_test.go and even there it wasn't
needed. Remove the file.
Signed-off-by: Miek Gieben <miek@miek.nl>

Went over the list at https://goreportcard.com/report/github.com/coredns/coredns
and removed/fix some code to make it slightly happier.
Signed-off-by: Miek Gieben <miek@miek.nl>