Browse Source

[Rework] Allow multiple conditions for dkim signing when using vault/redis

pull/4866/head
Vsevolod Stakhov 2 months ago
parent
commit
3de247b318
No account linked to committer's email address
2 changed files with 6 additions and 7 deletions
  1. 5
    6
      lualib/lua_dkim_tools.lua
  2. 1
    1
      src/plugins/lua/dkim_signing.lua

+ 5
- 6
lualib/lua_dkim_tools.lua View File

@@ -461,21 +461,20 @@ local function prepare_dkim_signing(N, task, settings)
if settings.use_vault then
if settings.vault_domains then
if settings.vault_domains:get_key(dkim_domain) then
return true, {
table.insert(p, {
domain = dkim_domain,
vault = true,
}
})
else
lua_util.debugm(N, task, 'domain %s is not designated for vault',
dkim_domain)
return false, {}
end
else
-- TODO: try every domain in the vault
return true, {
table.insert(p, {
domain = dkim_domain,
vault = true,
}
})
end
end

@@ -546,7 +545,7 @@ local function prepare_dkim_signing(N, task, settings)
insert_or_update_prop(N, task, p, 'domain', 'dkim_domain',
dkim_domain)

return true, p
return #p > 0 and true or false, p
end

exports.prepare_dkim_signing = prepare_dkim_signing

+ 1
- 1
src/plugins/lua/dkim_signing.lua View File

@@ -108,7 +108,7 @@ end
local function dkim_signing_cb(task)
local ret, selectors = dkim_sign_tools.prepare_dkim_signing(N, task, settings)

if not ret then
if not ret or #selectors == 0 then
return
end


Loading…
Cancel
Save