]> git.ipfire.org Git - thirdparty/pdns.git/commitdiff
Remove "place" from Lua axfrfilter
authorChristian Hofstaedtler <christian.hofstaedtler@deduktiva.com>
Fri, 7 Oct 2016 12:23:53 +0000 (14:23 +0200)
committerChristian Hofstaedtler <christian.hofstaedtler@deduktiva.com>
Thu, 8 Dec 2016 16:53:45 +0000 (17:53 +0100)
The slave code did not pay attention to d_place in the received
DNSResourceRecords anyway, so throwing away that info is alright.

pdns/lua-auth.cc
pdns/lua-auth4.cc

index 38d8913978f9f4ec70e275ed285db7b1fce74690..63447e235b844bab2f04d9fc0c3608279ced4097 100644 (file)
@@ -110,7 +110,6 @@ bool AuthLua::axfrfilter(const ComboAddress& remote, const DNSName& zone, const
 
   /* get the result */
   DNSResourceRecord rr;
-  rr.d_place = DNSResourceRecord::ANSWER;
   rr.ttl = 3600;
   rr.domain_id = in.domain_id;
 
@@ -140,15 +139,10 @@ bool AuthLua::axfrfilter(const ComboAddress& remote, const DNSName& zone, const
     else
       rr.qname=DNSName(qname);
 
-    if(!getFromTable("place", tmpnum))
-      rr.d_place = DNSResourceRecord::ANSWER;
-    else
-      rr.d_place = static_cast<DNSResourceRecord::Place>(tmpnum);
-
     /* removes 'value'; keeps 'key' for next iteration */
     lua_pop(d_lua, 1); // table
 
-    //    cerr<<"Adding content '"<<rr.content<<"' with place "<<(int)rr.d_place<<" \n";
+    //    cerr<<"Adding content '"<<rr.content<<"'\n";
     out.push_back(rr);
   }
   lua_pop(d_lua, 2); // c
index 4263b730ade6e2c255094eabff328cf8293fa31d..6503e9cbd15d4be7c0afc5bfe77f9e06009a436e 100644 (file)
@@ -55,7 +55,6 @@ AuthLua4::AuthLua4(const std::string& fname) {
   d_lw->registerFunction<int(DNSResourceRecord::*)()>("domain_id", [](DNSResourceRecord& rec) { return rec.domain_id; });
   d_lw->registerFunction<uint16_t(DNSResourceRecord::*)()>("qtype", [](DNSResourceRecord& rec) { return rec.qtype.getCode(); });
   d_lw->registerFunction<uint16_t(DNSResourceRecord::*)()>("qclass", [](DNSResourceRecord& rec) { return rec.qclass; });
-  d_lw->registerFunction<int(DNSResourceRecord::*)()>("d_place", [](DNSResourceRecord& rec) { return rec.d_place; });
   d_lw->registerFunction<uint8_t(DNSResourceRecord::*)()>("scopeMask", [](DNSResourceRecord& rec) { return rec.scopeMask; });
   d_lw->registerFunction<bool(DNSResourceRecord::*)()>("auth", [](DNSResourceRecord& rec) { return rec.auth; });
   d_lw->registerFunction<bool(DNSResourceRecord::*)()>("disabled", [](DNSResourceRecord& rec) { return rec.disabled; });
@@ -198,7 +197,7 @@ AuthLua4::AuthLua4(const std::string& fname) {
       stubDoResolve(DNSName(qname), qtype, ret);
       int i = 0;
       for(const auto &row: ret) {
-        luaResult[++i] = DNSResourceRecord(row.dr);
+        luaResult[++i] = DNSResourceRecord::fromWire(row.dr);
         luaResult[i].auth = row.auth;
       }
       return luaResult;