diff --git a/modoboa_installer/scripts/clamav.py b/modoboa_installer/scripts/clamav.py index a62eda6..181eaf3 100644 --- a/modoboa_installer/scripts/clamav.py +++ b/modoboa_installer/scripts/clamav.py @@ -42,7 +42,7 @@ class Clamav(base.Installer): """Additional tasks.""" if package.backend.FORMAT == "deb": user = self.config.get(self.appname, "user") - if self.config.get("amavis", "enabled").lower() == "true": + if self.config.getboolean("amavis", "enabled"): system.add_user_to_group( user, self.config.get("amavis", "user") ) diff --git a/modoboa_installer/scripts/modoboa.py b/modoboa_installer/scripts/modoboa.py index 4e8d28a..4d9b778 100644 --- a/modoboa_installer/scripts/modoboa.py +++ b/modoboa_installer/scripts/modoboa.py @@ -49,24 +49,11 @@ class Modoboa(base.Installer): self.instance_path = self.config.get("modoboa", "instance_path") self.extensions = self.config.get("modoboa", "extensions").split() self.devmode = self.config.getboolean("modoboa", "devmode") - # Sanity check for amavis and rspamd - self.amavis_enabled = self.sanity_check("modoboa-amavis", "amavis") - self.sanity_check("modoboa-rspamd", "rspamd") - + self.amavis_enabled = self.config.getboolean("amavis", "enabled") self.dovecot_enabled = self.config.getboolean("dovecot", "enabled") self.opendkim_enabled = self.config.getboolean("opendkim", "enabled") self.dkim_cron_enabled = False - def sanity_check(self, extension, plugin): - # Sanity check for plugin requirements - enabled = False - if extension in self.extensions: - if self.config.getboolean(plugin, "enabled"): - enabled = True - else: - self.extensions.remove(extension) - return enabled - def is_extension_ok_for_version(self, extension, version): """Check if extension can be installed with this modo version.""" version = utils.convert_version_to_int(version) diff --git a/run.py b/run.py index 5d0aa3a..1b1c24f 100755 --- a/run.py +++ b/run.py @@ -196,9 +196,10 @@ def main(input_args): config.set("modoboa", "version", args.version) config.set("modoboa", "install_beta", str(args.beta)) - PRIMARY_APPS.append( - "amavis" if config.get("antispam", "type") == "amavis" else "rspamd" - ) + if config.get("antispam", "type") == "amavis": + PRIMARY_APPS += ["amavis", "opendkim"] + else: + PRIMARY_APPS += ["rspamd"] if args.backup or args.silent_backup: backup_system(config, args)