diff options
| author |  Miek Gieben <miek@miek.nl>
| 2018-07-28 10:27:38 +0100 |
|---|---|---|
| committer |  GitHub <noreply@github.com>
| 2018-07-28 10:27:38 +0100 |
| commit | 62df253371f354200a3efde9e86e80eb50a2441b (patch) | |
| tree | d2231d3beff3cd2c5f4dc4e8ad23588ba0bf99c5 /test | |
| parent | 395be9adfc07017d8dc9a8c260aed2541b4adda3 (diff) | |
| download | coredns-62df253371f354200a3efde9e86e80eb50a2441b.tar.gz coredns-62df253371f354200a3efde9e86e80eb50a2441b.tar.zst coredns-62df253371f354200a3efde9e86e80eb50a2441b.zip | |
request: re-adjust size for edns0 RR (#2007)
* Add regression test for broken DNS truncation
With the latest changes to the implementation, request.Scrub() returns
invalid response messages which are larger than the advertised buffer
size. This commit adds a failing test for that.
* Add benchmark for response truncation
* request: re-adjust size for edns0 RR
Cherry-picked the test and benchmark commits from #1962 and make the
failing test not fail, by adding minimal fix in request.go.
This makes size 12 bytes smaller if we need to add back the OPT RR in
the message.
Closes: #1962
Signed-off-by: Miek Gieben <miek@miek.nl>
* Scrub: correctly check for EDNS0, not only Do
Make Scrub check for EDNS0, not only if the Do bit is set.
Signed-off-by: Miek Gieben <miek@miek.nl>
Diffstat (limited to 'test')
0 files changed, 0 insertions, 0 deletions