From: Andrew Lewis <nerf@judo.za.org>
Date: Thu, 7 Sep 2017 17:43:11 +0000 (+0200)
Subject: [Minor] Fix previous commit: avoid doing useless things
X-Git-Tag: 1.6.4~3
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=f5ada793b2df638f5f65003e08b36ecb20ab70ff;p=rspamd.git

[Minor] Fix previous commit: avoid doing useless things
---

diff --git a/lualib/auth_results.lua b/lualib/auth_results.lua
index 8f7e67553..5f4b2f896 100644
--- a/lualib/auth_results.lua
+++ b/lualib/auth_results.lua
@@ -133,14 +133,14 @@ local function gen_auth_results(task, settings)
         end
         table.insert(hdr_parts, hdr)
       elseif auth_type == 'dkim' and key ~= 'none' then
-        local dkim_parts = {}
         if common.symbols[auth_types['dkim'][key]][1] then
+          local dkim_parts = {}
           local opts = common.symbols[auth_types['dkim'][key]][1]['options']
           for _, v in ipairs(opts) do
             table.insert(dkim_parts, auth_type .. '=' .. key .. ' header.d=' .. v)
           end
+          table.insert(hdr_parts, table.concat(dkim_parts, '; '))
         end
-        table.insert(hdr_parts, table.concat(dkim_parts, '; '))
       elseif auth_type == 'arc' and key ~= 'none' then
         if common.symbols[auth_types['arc'][key]][1] then
           local opts = common.symbols[auth_types['arc'][key]][1]['options'] or {}