From: Otto Moerbeek <otto.moerbeek@open-xchange.com>
Date: Fri, 4 Feb 2022 10:00:48 +0000 (+0100)
Subject: The on the same line with if, whitespace issues
X-Git-Tag: auth-4.7.0-alpha1~20^2
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=b59e87957b25a6bbfe736bcea83feafb99f0ed7c;p=thirdparty%2Fpdns.git

The on the same line with if, whitespace issues
---

diff --git a/pdns/recursordist/contrib/powerdns-example-script.lua b/pdns/recursordist/contrib/powerdns-example-script.lua
index d98e7200f7..eb9dad9efe 100644
--- a/pdns/recursordist/contrib/powerdns-example-script.lua
+++ b/pdns/recursordist/contrib/powerdns-example-script.lua
@@ -18,8 +18,8 @@ badips:addMask("127.1.0.0/16")
 
 -- this check is applied before any packet parsing is done
 function ipfilter(rem, loc, dh)
-  pdnslog("ipfilter called, rem: "..rem:toStringWithPort().."loc: "..loc:toStringWithPort().." match:"..tostring(badips:match(rem)))
-  pdnslog("id: "..dh:getID().."aa: "..tostring(dh:getAA()).."ad: "..tostring(dh:getAD()).." arcount: "..dh:getARCOUNT())
+  pdnslog("ipfilter called, rem: "..rem:toStringWithPort().." loc: "..loc:toStringWithPort().." match:"..tostring(badips:match(rem)))
+  pdnslog("id: "..dh:getID().." aa: "..tostring(dh:getAA()).." ad: "..tostring(dh:getAD()).." arcount: "..dh:getARCOUNT())
   pdnslog("ports: "..rem:getPort().." "..loc:getPort())
   return badips:match(rem)
 end
@@ -31,14 +31,12 @@ function preresolve(dq)
   pdnslog("Got question for "..dq.qname:toString().." from "..dq.remoteaddr:toString().." to "..dq.localaddr:toString())
 
   local ednssubnet = dq:getEDNSSubnet()
-  if ednssubnet
-  then
+  if ednssubnet then
     pdnslog("Packet EDNS subnet source: "..ednssubnet:toString()..", "..ednssubnet:getNetwork():toString())
   end
 
   local a = dq:getEDNSOption(3)
-  if a
-  then
+  if a then
     pdnslog("There is an EDNS option 3 present: "..a)
   end
 
@@ -48,21 +46,18 @@ function preresolve(dq)
   end
 
   -- note that the comparisons below are CaSe InSensiTivE and you don't have to worry about trailing dots
-  if dq.qname:equal("magic.com")
-  then
+  if dq.qname:equal("magic.com") then
     magicMetric:inc()
     pdnslog("Magic!")
   else
     pdnslog("not magic..")
   end
 
-  if dq.qname == magic2
-  then
+  if dq.qname == magic2 then
     pdnslog("Faster magic") -- compares against existing DNSName
   end
 
-  if blockset:check(dq.qname)
-  then
+  if blockset:check(dq.qname) then
     dq.variable = true      -- disable packet cache in any case
     if dq.qtype == pdns.A then
       dq:addAnswer(pdns.A, "1.2.3.4")
@@ -71,15 +66,13 @@ function preresolve(dq)
     end
   end
 
-  if dropset:check(dq.qname)
-  then
+  if dropset:check(dq.qname) then
    pdnslog("dopping query")
    dq.appliedPolicy.policyKind = pdns.policykinds.Drop
    return false -- recursor still needs to handle the policy
   end
 
-  if malwareset:check(dq.qname)
-  then
+  if malwareset:check(dq.qname) then
     dq:addAnswer(pdns.CNAME, "blog.powerdns.com.")
     dq.rcode = 0
     dq.followupFunction = "followCNAMERecords"    -- this makes PowerDNS lookup your CNAME
@@ -92,16 +85,14 @@ end
 -- this implements DNS64
 
 function nodata(dq)
-  if dq.qtype == pdns.AAAA
-  then
+  if dq.qtype == pdns.AAAA then
     dq.followupFunction = "getFakeAAAARecords"
     dq.followupName = dq.qname
     dq.followupPrefix="fe80::"
     return true
   end
 
-  if dq.qtype == pdns.PTR
-  then
+  if dq.qtype == pdns.PTR then
     dq.followupFunction = "getFakePTRRecords"
     dq.followupName = dq.qname
     dq.followupPrefix = "fe80::"
@@ -117,8 +108,7 @@ function postresolve(dq)
   local records = dq:getRecords()
   for k,v in pairs(records) do
     pdnslog(k.." "..v.name:toString().." "..v:getContent())
-    if v.type == pdns.A and v:getContent() == "185.31.17.73"
-    then
+    if v.type == pdns.A and v:getContent() == "185.31.17.73" then
       pdnslog("Changing content!")
       v:changeContent("130.161.252.29")
       v.ttl = 1
@@ -132,8 +122,7 @@ nxdomainsuffix = newDN("com")
 
 function nxdomain(dq)
   pdnslog("nxdomain called for: "..dq.qname:toString())
-  if dq.qname:isPartOf(nxdomainsuffix)
-  then
+  if dq.qname:isPartOf(nxdomainsuffix) then
     dq.rcode = 0 -- make it a normal answer
     dq:addAnswer(pdns.CNAME, "ourhelpfulservice.com")
     dq:addAnswer(pdns.A, "1.2.3.4", 60, "ourhelpfulservice.com")