]> source.dussan.org Git - rspamd.git/commitdiff
[Minor] Slightly improve logging
authorVsevolod Stakhov <vsevolod@highsecure.ru>
Mon, 21 May 2018 14:11:32 +0000 (15:11 +0100)
committerVsevolod Stakhov <vsevolod@highsecure.ru>
Mon, 21 May 2018 14:11:32 +0000 (15:11 +0100)
src/plugins/lua/asn.lua
src/plugins/lua/emails.lua

index 3e84a3824c225daa4eb878b783e181819b63527d..61572a6009b2a260a516fc754e2b9bde03b1072b 100644 (file)
@@ -60,18 +60,19 @@ local function asn_check(task)
 
   local asn_check_func = {}
   function asn_check_func.rspamd(ip)
+    local dnsbl = options['provider_info']['ip' .. ip:get_version()]
+    local req_name = rspamd_logger.slog("%1.%2",
+        table.concat(ip:inversed_str_octets(), '.'), dnsbl)
     local function rspamd_dns_cb(_, _, results, dns_err)
       if dns_err and (dns_err ~= 'requested record is not found' and dns_err ~= 'no records with this name') then
-        rspamd_logger.errx(task, 'error querying dns: %s', dns_err)
+        rspamd_logger.errx(task, 'error querying dns (%s): %s', req_name, dns_err)
       end
       if not (results and results[1]) then return end
       local parts = rspamd_re:split(results[1])
       -- "15169 | 8.8.8.0/24 | US | arin |" for 8.8.8.8
       asn_set(parts[1], parts[2], parts[3])
     end
-    local dnsbl = options['provider_info']['ip' .. ip:get_version()]
-    local req_name = rspamd_logger.slog("%1.%2",
-        table.concat(ip:inversed_str_octets(), '.'), dnsbl)
+
     task:get_resolver():resolve_txt(task:get_session(), task:get_mempool(),
         req_name, rspamd_dns_cb)
   end
index 51df0e9594cc3af199b45844c5f5678c2fe24b83..5f076e1d5e0bd16fa875f6f7fc86060eaab8cfab 100644 (file)
@@ -52,7 +52,7 @@ local function check_email_rule(task, rule, addr)
     local function emails_dns_cb(_, _, results, err)
       if err and (err ~= 'requested record is not found'
           and err ~= 'no records with this name') then
-        logger.errx(task, 'Error querying DNS: %1', err)
+        logger.errx(task, 'Error querying DNS(%s): %s', to_resolve, err)
       elseif results then
         local expected_found = false
         local symbol = rule['symbol']