minor fix - approved domains should not be case sensitive

Erudika · May 24, 2024 · 3c0a99d · 3c0a99d
1 parent 4f014da
commit 3c0a99d
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/main/java/com/erudika/scoold/utils/ScooldUtils.java b/src/main/java/com/erudika/scoold/utils/ScooldUtils.java
@@ -215,7 +215,7 @@ public static ScooldConfig getConfig() {
 		// multiple domains/admins are allowed only in Scoold PRO
 		String approvedDomain = StringUtils.substringBefore(CONF.approvedDomainsForSignups(), ",");
 		if (!StringUtils.isBlank(approvedDomain)) {
-			APPROVED_DOMAINS.add(approvedDomain);
+			APPROVED_DOMAINS.add(approvedDomain.toLowerCase());
 		}
 		// multiple admins are allowed only in Scoold PRO
 		String admin = StringUtils.substringBefore(CONF.admins(), ",");
@@ -524,7 +524,7 @@ public boolean isEmailDomainApproved(String email) {
 		if (StringUtils.isBlank(email)) {
 			return false;
 		}
-		if (!APPROVED_DOMAINS.isEmpty() && !APPROVED_DOMAINS.contains(StringUtils.substringAfter(email, "@"))) {
+		if (!APPROVED_DOMAINS.isEmpty() && !APPROVED_DOMAINS.contains(StringUtils.substringAfter(email, "@").toLowerCase())) {
 			logger.warn("Attempted signin from an unknown domain - email {} is part of an unapproved domain.", email);
 			return false;
 		}