From 6ff0b969d5bf397636f195dd8762530fefcaf9df Mon Sep 17 00:00:00 2001
From: Luke Taylor <luke.taylor@springsource.com>
Date: Mon, 23 Jun 2008 23:43:48 +0000
Subject: [PATCH] Corrected ldap sample config (traditional bean version was
 wrong)

---
 .../src/main/webapp/WEB-INF/applicationContext-security.xml  | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/samples/ldap/src/main/webapp/WEB-INF/applicationContext-security.xml b/samples/ldap/src/main/webapp/WEB-INF/applicationContext-security.xml
index 29ba9d56a3..4bceec8759 100644
--- a/samples/ldap/src/main/webapp/WEB-INF/applicationContext-security.xml
+++ b/samples/ldap/src/main/webapp/WEB-INF/applicationContext-security.xml
@@ -18,7 +18,7 @@
     <!-- Simple namespace-based configuration -->
 
     <s:ldap-server ldif="classpath:users.ldif" port="33389"/>
- 
+
     <s:ldap-authentication-provider 
         group-search-filter="member={0}" 
         group-search-base="ou=groups"
@@ -51,8 +51,7 @@
 			<bean class="org.springframework.security.ldap.populator.DefaultLdapAuthoritiesPopulator">
 				<constructor-arg ref="contextSource" />
 				<constructor-arg value="ou=groups" />
-				<property name="groupSearchFilter" value="member={0}"/>
-				<property name="groupRoleAttribute" value="ou" />
+				<property name="groupSearchFilter" value="(member={0})"/>
 				<property name="rolePrefix" value="ROLE_"/>
 				<property name="searchSubtree" value="true"/>
 				<property name="convertToUpperCase" value="true"/>