Browse Source

[trac505] No authority section on long DNAME

It seems the RFC 2672 isn't too clear about it, but all checked
authoritative servers don't provide the authority section in case of
YXDOMAIN caused by too long synthesized name.
Michal 'vorner' Vaner 14 years ago
parent
commit
32ca390dad
2 changed files with 2 additions and 3 deletions
  1. 0 1
      src/bin/auth/query.cc
  2. 2 2
      src/bin/auth/tests/query_unittest.cc

+ 0 - 1
src/bin/auth/query.cc

@@ -196,7 +196,6 @@ Query::process() const {
                  */
                 catch (const isc::dns::TooLongName&) {
                     response_.setRcode(Rcode::YXDOMAIN());
-                    getAuthAdditional(*result.zone);
                     return;
                 }
                 // No break; here, fall trough.

+ 2 - 2
src/bin/auth/tests/query_unittest.cc

@@ -641,8 +641,8 @@ TEST_F(QueryTest, LongDNAME) {
         "somethingveryveryverylong.longdname.example.com"), RRType::A(),
         response).process());
 
-    responseCheck(response, Rcode::YXDOMAIN(), AA_FLAG, 1, 3, 3,
-        dname_long_txt, zone_ns_txt, ns_addrs_txt);
+    responseCheck(response, Rcode::YXDOMAIN(), AA_FLAG, 1, 0, 0,
+        dname_long_txt, NULL, NULL);
 }
 
 }