diff options
| author |  Miek Gieben <miek@miek.nl>
| 2020-11-03 15:31:34 +0100 |
|---|---|---|
| committer |  GitHub <noreply@github.com>
| 2020-11-03 06:31:34 -0800 |
| commit | a136b7128b7113c79a971dc8ed728e73cd37420b (patch) | |
| tree | d5e9a77f6c253d4a7bf55c239807cb4ae793c984 /test | |
| parent | f286a24b4e2c8027b9045b4fb36f92ecf5ed33f1 (diff) | |
| download | coredns-a136b7128b7113c79a971dc8ed728e73cd37420b.tar.gz coredns-a136b7128b7113c79a971dc8ed728e73cd37420b.tar.zst coredns-a136b7128b7113c79a971dc8ed728e73cd37420b.zip | |
plugin/dnstap: remove custom encoder (#4242)
* plugin/dnstap: remove encoder*.go
Those files reimplemented parts of the dnstap spec, we can just use the
dnstap functions for that. This leaves all the queuing that is enabled
and drops messages if the dnstap reader can't keep up. In the new code
flush() would never return an error (at least I couldn't make it do so),
so the reconnect functionally is moved to kick off when we get write
errors.
Some smaller cosmetic changes as well, `d.socket` is now `proto`, which
makes the dial() function smaller.
Total testing time is now <1s (which was the impetus to look into this
plugin *again*).
See #4238
The buffered channel needs to be sized correctly, as we may need to do
some queing if the dnstap reader can't keep up.
Signed-off-by: Miek Gieben <miek@miek.nl>
* add missing file
Signed-off-by: Miek Gieben <miek@miek.nl>
* update doc on queing
Signed-off-by: Miek Gieben <miek@miek.nl>
Diffstat (limited to 'test')
0 files changed, 0 insertions, 0 deletions