Merge branch 'master' of gitlab.iaik.tugraz.at:egiz/moa-idspssHEADmaster

1 files changed, 21 insertions, 23 deletions

diff --git a/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/validation/oa/OASSOConfigValidation.java b/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/validation/oa/OASSOConfigValidation.java
index 971e11cc4..109257551 100644
--- a/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/validation/oa/OASSOConfigValidation.java
+++ b/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/validation/oa/OASSOConfigValidation.java
@@ -27,33 +27,31 @@ import java.util.List;
 
 import javax.servlet.http.HttpServletRequest;
 
-import org.apache.log4j.Logger;
-
 import at.gv.egovernment.moa.id.commons.validation.ValidationHelper;
 import at.gv.egovernment.moa.id.configuration.data.oa.OASSOConfig;
 import at.gv.egovernment.moa.id.configuration.helper.LanguageHelper;
 import at.gv.egovernment.moa.util.MiscUtil;
+import lombok.extern.slf4j.Slf4j;
 
+@Slf4j
 public class OASSOConfigValidation {
-	
-	private static final Logger log = Logger.getLogger(OASSOConfigValidation.class);
-	
-	public List<String> validate(OASSOConfig form, boolean isAdmin, HttpServletRequest request) {
-		
-		List<String> errors = new ArrayList<String>();
-		
-		String urlString = form.getSingleLogOutURL();
-		if (MiscUtil.isEmpty(urlString)) {
-			log.info("No Single Log-Out URL");
-			//TODO: set error if it is implemented
-			//errors.add(LanguageHelper.getErrorString("validation.sso.logouturl.empty"));
-		} else {
-			if (!ValidationHelper.validateURL(urlString) && form.isUseSSO()) {
-				log.info("Single Log-Out url validation error");
-				errors.add(LanguageHelper.getErrorString("validation.sso.logouturl.valid", request));
-			}
-		}
-		
-		return errors;
-	}
+
+  public List<String> validate(OASSOConfig form, boolean isAdmin, HttpServletRequest request) {
+
+    final List<String> errors = new ArrayList<>();
+
+    final String urlString = form.getSingleLogOutURL();
+    if (MiscUtil.isEmpty(urlString)) {
+      log.info("No Single Log-Out URL");
+      // TODO: set error if it is implemented
+      // errors.add(LanguageHelper.getErrorString("validation.sso.logouturl.empty"));
+    } else {
+      if (!ValidationHelper.validateURL(urlString) && form.isUseSSO()) {
+        log.info("Single Log-Out url validation error");
+        errors.add(LanguageHelper.getErrorString("validation.sso.logouturl.valid", request));
+      }
+    }
+
+    return errors;
+  }
 }