From 384db5e23efdfa615441f994b97385ace44162b6 Mon Sep 17 00:00:00 2001
From: Robert von Burg <eitch@eitchnet.ch>
Date: Mon, 15 Apr 2019 13:43:20 +0200
Subject: [PATCH] [Minor] Log LDAP groups for user

---
 .../privilege/handler/JsonConfigLdapPrivilegeHandler.java    | 2 ++
 .../privilege/handler/SimpleLdapPrivilegeHandler.java        | 5 ++++-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/li.strolch.privilege/src/main/java/li/strolch/privilege/handler/JsonConfigLdapPrivilegeHandler.java b/li.strolch.privilege/src/main/java/li/strolch/privilege/handler/JsonConfigLdapPrivilegeHandler.java
index ce755c176..2b896a685 100644
--- a/li.strolch.privilege/src/main/java/li/strolch/privilege/handler/JsonConfigLdapPrivilegeHandler.java
+++ b/li.strolch.privilege/src/main/java/li/strolch/privilege/handler/JsonConfigLdapPrivilegeHandler.java
@@ -95,6 +95,8 @@ public class JsonConfigLdapPrivilegeHandler extends BaseLdapPrivilegeHandler {
 	@Override
 	protected Set<String> getLdapGroups(String username, Attributes attrs) throws NamingException {
 		Set<String> ldapGroups = LdapHelper.getLdapGroups(attrs);
+		logger.info("User " + username + " has LDAP Groups: ");
+		ldapGroups.forEach(s -> logger.info("- " + s));
 
 		Set<String> relevantLdapGroups = ldapGroups.stream().filter(s -> this.ldapGroupNames.contains(s))
 				.collect(toSet());
diff --git a/li.strolch.privilege/src/main/java/li/strolch/privilege/handler/SimpleLdapPrivilegeHandler.java b/li.strolch.privilege/src/main/java/li/strolch/privilege/handler/SimpleLdapPrivilegeHandler.java
index d5875c4e4..c65f5b61d 100644
--- a/li.strolch.privilege/src/main/java/li/strolch/privilege/handler/SimpleLdapPrivilegeHandler.java
+++ b/li.strolch.privilege/src/main/java/li/strolch/privilege/handler/SimpleLdapPrivilegeHandler.java
@@ -72,7 +72,10 @@ public class SimpleLdapPrivilegeHandler extends BaseLdapPrivilegeHandler {
 
 	@Override
 	protected Set<String> getLdapGroups(String username, Attributes attrs) throws NamingException {
-		return LdapHelper.getLdapGroups(attrs);
+		Set<String> ldapGroups = LdapHelper.getLdapGroups(attrs);
+		logger.info("User " + username + " has LDAP Groups: ");
+		ldapGroups.forEach(s -> logger.info("- " + s));
+		return ldapGroups;
 	}
 
 	@Override