mirror of
https://github.com/isc-projects/bind9.git
synced 2026-06-11 12:10:00 -04:00
Drop redundant uses of authoritative=True
The ans3 custom server instance is created with default_aa=True. Do not
pass the authoritative=True keyword argument to the DnsResponseSend
constructor in CookieHandler.get_responses() as it is redundant.
(cherry picked from commit c61539279d)
This commit is contained in:
parent
5f39e1d946
commit
dbcf0e6c4e
1 changed files with 2 additions and 2 deletions
|
|
@ -73,13 +73,13 @@ class CookieHandler(DomainHandler):
|
|||
# If missing cookie entirely, just return SERVFAIL
|
||||
if cookie is None:
|
||||
qctx.response.set_rcode(dns.rcode.SERVFAIL)
|
||||
yield DnsResponseSend(qctx.response, authoritative=True)
|
||||
yield DnsResponseSend(qctx.response)
|
||||
|
||||
# If there is a client cookie, mock BADCOOKIE to trigger
|
||||
# the resend loop logic.
|
||||
qctx.response.use_edns(options=[cookie])
|
||||
qctx.response.set_rcode(dns.rcode.BADCOOKIE)
|
||||
yield DnsResponseSend(qctx.response, authoritative=True)
|
||||
yield DnsResponseSend(qctx.response)
|
||||
|
||||
|
||||
def resend_server() -> AsyncDnsServer:
|
||||
|
|
|
|||
Loading…
Reference in a new issue