From c4101162d667aa7be34c662198b126079d058833 Mon Sep 17 00:00:00 2001 From: Miro Prasil Date: Wed, 29 Jan 2020 11:32:42 +0000 Subject: [PATCH] SIGNUPS_ALLOWED with no whitelist [fixes #830] This reverts back to `SIGNUPS_ALLOWED` when there is no domain whitelist set. The functionality was broken in 64d6f72. --- src/config.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/config.rs b/src/config.rs index 854cbf00..93161e51 100644 --- a/src/config.rs +++ b/src/config.rs @@ -515,7 +515,7 @@ impl Config { // (it doesn't contain any domains), or if it matches at least // one domain. let whitelist_str = self.signups_domains_whitelist(); - whitelist_str.is_empty() || whitelist_str.split(',').filter(|s| !s.is_empty()).any(|d| d == e[0]) + ( whitelist_str.is_empty() && CONFIG.signups_allowed() )|| whitelist_str.split(',').filter(|s| !s.is_empty()).any(|d| d == e[0]) } pub fn delete_user_config(&self) -> Result<(), Error> {