From ae4d7f775ecfad1c464a8a7459f0dcefc65d756b Mon Sep 17 00:00:00 2001
From: Mat Jaggard <github@jaggard.org.uk>
Date: Fri, 23 Oct 2015 13:35:31 +0100
Subject: [PATCH] Fixed issue with syntax for filter

---
 .../archiva/redback/common/ldap/role/DefaultLdapRoleMapper.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/redback-common/redback-common-ldap/src/main/java/org/apache/archiva/redback/common/ldap/role/DefaultLdapRoleMapper.java b/redback-common/redback-common-ldap/src/main/java/org/apache/archiva/redback/common/ldap/role/DefaultLdapRoleMapper.java
index d7f2ead9..c4193a5d 100644
--- a/redback-common/redback-common-ldap/src/main/java/org/apache/archiva/redback/common/ldap/role/DefaultLdapRoleMapper.java
+++ b/redback-common/redback-common-ldap/src/main/java/org/apache/archiva/redback/common/ldap/role/DefaultLdapRoleMapper.java
@@ -151,7 +151,7 @@ public class DefaultLdapRoleMapper
 
             if ( !StringUtils.isEmpty( this.groupFilter ) )
             {
-                filter = "&(" + filter + ")(" + this.groupFilter + ")";
+                filter = "(&(" + filter + ")(" + this.groupFilter + "))";
             }
 
             namingEnumeration = context.search( getGroupsDn(), filter, searchControls );