From c4101162d667aa7be34c662198b126079d058833 Mon Sep 17 00:00:00 2001
From: Miro Prasil <miro@circleci.com>
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> {