Merge pull request #1356 from BlackDex/fix-config-bug

Fixed small buggy in validation
4 years ago · 46d31ee5f7
parent d41350050b e794b397d3
commit 46d31ee5f7
1 changed files with 1 additions and 1 deletions
--- a/src/config.rs
+++ b/src/config.rs
@ -557,7 +557,7 @@ fn validate_config(cfg: &ConfigItems) -> Result<(), Error> {
            err!("Both `SMTP_HOST` and `SMTP_FROM` need to be set for email support")
        }

-        if !cfg.smtp_from.contains('@') {
+        if cfg.smtp_host.is_some() && !cfg.smtp_from.contains('@') {
            err!("SMTP_FROM does not contain a mandatory @ sign")
        }