From e0701823d6892138a77812b76d87e55bb114b8ae Mon Sep 17 00:00:00 2001
From: Maxime Dor <max@kamax.io>
Date: Fri, 4 Aug 2017 15:00:14 +0200
Subject: [PATCH] Enforce mandatory LDAP config

---
 .../io/kamax/mxisd/config/LdapConfig.groovy     | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/src/main/groovy/io/kamax/mxisd/config/LdapConfig.groovy b/src/main/groovy/io/kamax/mxisd/config/LdapConfig.groovy
index 58451f5..e90046e 100644
--- a/src/main/groovy/io/kamax/mxisd/config/LdapConfig.groovy
+++ b/src/main/groovy/io/kamax/mxisd/config/LdapConfig.groovy
@@ -20,6 +20,7 @@
 
 package io.kamax.mxisd.config
 
+import org.apache.commons.lang.StringUtils
 import org.slf4j.Logger
 import org.slf4j.LoggerFactory
 import org.springframework.beans.factory.InitializingBean
@@ -133,7 +134,23 @@ class LdapConfig implements InitializingBean {
 
     @Override
     void afterPropertiesSet() throws Exception {
+        log.info("LDAP enabled: {}", getEnabled())
+
+        if (!getEnabled()) {
+            return
+        }
+
         log.info("Matrix ID type: {}", getType())
+        log.info("LDAP Host: {}", getHost())
+        log.info("LDAP Bind DN: {}", getBindDn())
+        log.info("LDAP Attribute: {}", getAttribute())
+
+        if (StringUtils.isBlank(getHost())) {
+            throw new IllegalStateException("LDAP Host must be configured!")
+        }
+        if (StringUtils.isBlank(getAttribute())) {
+            throw new IllegalStateException("LDAP attribute must be configured!")
+        }
     }
 
 }