diff options
| author | Vsevolod Stakhov <vsevolod@highsecure.ru> | 2019-04-21 16:58:20 +0100 |
|---|---|---|
| committer | Vsevolod Stakhov <vsevolod@highsecure.ru> | 2019-04-21 16:58:20 +0100 |
| commit | ba4c9baa69a7ae1285aeadb40d48a1a9832b1ee8 (patch) | |
| tree | 4d3a24c57f13f6078e3d5a6903346fbeefdf111b /src/plugins/lua | |
| parent | 2972451188c08b3a741d8381a63087421d6e9a1b (diff) | |
| download | rspamd-ba4c9baa69a7ae1285aeadb40d48a1a9832b1ee8.tar.gz rspamd-ba4c9baa69a7ae1285aeadb40d48a1a9832b1ee8.zip | |
[Minor] Unify settings checks
Diffstat (limited to 'src/plugins/lua')
| -rw-r--r-- | src/plugins/lua/arc.lua | 8 | ||||
| -rw-r--r-- | src/plugins/lua/dkim_signing.lua | 9 |
2 files changed, 3 insertions, 14 deletions
diff --git a/src/plugins/lua/arc.lua b/src/plugins/lua/arc.lua index d3fe818da..5dfba662c 100644 --- a/src/plugins/lua/arc.lua +++ b/src/plugins/lua/arc.lua @@ -637,13 +637,7 @@ for k,v in pairs(opts) do end end -if not (settings.use_redis or - settings.path or - settings.domain or - settings.path_map or - settings.selector_map or - settings.use_http_headers or - (settings.signing_table and settings.key_table)) then +if not dkim_sign_tools.validate_signing_settings(settings) then rspamd_logger.infox(rspamd_config, 'mandatory parameters missing, disable arc signing') return end diff --git a/src/plugins/lua/dkim_signing.lua b/src/plugins/lua/dkim_signing.lua index d8e1f7519..05a7739e7 100644 --- a/src/plugins/lua/dkim_signing.lua +++ b/src/plugins/lua/dkim_signing.lua @@ -138,17 +138,12 @@ for k,v in pairs(opts) do end end -if not (settings.use_redis or - settings.path or - settings.domain or - settings.path_map or - settings.selector_map or - settings.use_http_headers or - (settings.signing_table and settings.key_table)) then +if not dkim_sign_tools.validate_signing_settings(settings) then rspamd_logger.infox(rspamd_config, 'mandatory parameters missing, disable dkim signing') lua_util.disable_module(N, "config") return end + if settings.use_redis then redis_params = lua_redis.parse_redis_server('dkim_signing') |