6.0.0-beta1
11/8/25

[#10975] Group ldap backend listGroups($user) doesn't use filter
Summary Group ldap backend listGroups($user) doesn't use filter
Queue Horde Framework Packages
Queue Version Git master
Type Bug
State Resolved
Priority 1. Low
Owners jan (at) horde (dot) org
Requester jesse.sandberg (at) verkkokauppa (dot) com
Created 02/09/2012 (5021 days ago)
Due
Updated 02/22/2012 (5008 days ago)
Assigned
Resolved 02/13/2012 (5017 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch Yes

History
02/22/2012 12:44:55 PM Git Commit Comment #6 Reply to this comment
Changes have been made in Git (master):

commit 1697bd23505df5c68c72398d42e795bf8323bebc
Author: Jan Schneider <jan@horde.org>
Date:   Wed Feb 22 13:06:29 2012 +0100

     [jan] Use configured filter when listing groups (Jesse Sandberg 
<jesse.sandberg@verkkokauppa.com>, Bug #10975).

  framework/Group/package.xml |    4 ++--
  1 files changed, 2 insertions(+), 2 deletions(-)

http://git.horde.org/horde-git/-/commit/1697bd23505df5c68c72398d42e795bf8323bebc
02/14/2012 11:26:04 PM Git Commit Comment #5 Reply to this comment
Changes have been made in Git (develop):

commit 3a81c2b1265c018bdb836930315f014f3edd81f7
Author: Jan Schneider <jan@horde.org>
Date:   Mon Feb 13 19:46:21 2012 +0100

     [jan] Use configured filter when listing groups (Jesse Sandberg 
<jesse.sandberg@verkkokauppa.com>, Bug #10975).

  framework/Ldap/package.xml |    4 ++--
  1 files changed, 2 insertions(+), 2 deletions(-)

http://git.horde.org/horde-git/-/commit/3a81c2b1265c018bdb836930315f014f3edd81f7
02/14/2012 11:25:46 PM Git Commit Comment #4 Reply to this comment
Changes have been made in Git (develop):

commit 42320ddf4d023f147e6fe69e49ceafa3bf5fabfa
Author: Jesse Sandberg <jesse.sandberg@verkkokauppa.com>
Date:   Thu Feb 9 17:01:09 2012 +0200

     listGroups($user) didn't use $this->_filter and returned groups 
outside original filter

     Signed-off-by: Jan Schneider <jan@horde.org>
     Bug: 10975

  framework/Group/lib/Horde/Group/Ldap.php |    1 +
  1 files changed, 1 insertions(+), 0 deletions(-)

http://git.horde.org/horde-git/-/commit/42320ddf4d023f147e6fe69e49ceafa3bf5fabfa
02/13/2012 06:47:28 PM Jan Schneider Assigned to Jan Schneider
State ⇒ Resolved
 
02/13/2012 06:47:15 PM Git Commit Comment #3 Reply to this comment
Changes have been made in Git (master):

commit 3a81c2b1265c018bdb836930315f014f3edd81f7
Author: Jan Schneider <jan@horde.org>
Date:   Mon Feb 13 19:46:21 2012 +0100

     [jan] Use configured filter when listing groups (Jesse Sandberg 
<jesse.sandberg@verkkokauppa.com>, Bug #10975).

  framework/Ldap/package.xml |    4 ++--
  1 files changed, 2 insertions(+), 2 deletions(-)

http://git.horde.org/horde-git/-/commit/3a81c2b1265c018bdb836930315f014f3edd81f7
02/13/2012 06:47:10 PM Git Commit Comment #2 Reply to this comment
Changes have been made in Git (master):

commit 42320ddf4d023f147e6fe69e49ceafa3bf5fabfa
Author: Jesse Sandberg <jesse.sandberg@verkkokauppa.com>
Date:   Thu Feb 9 17:01:09 2012 +0200

     listGroups($user) didn't use $this->_filter and returned groups 
outside original filter

     Signed-off-by: Jan Schneider <jan@horde.org>
     Bug: 10975

  framework/Group/lib/Horde/Group/Ldap.php |    1 +
  1 files changed, 1 insertions(+), 0 deletions(-)

http://git.horde.org/horde-git/-/commit/42320ddf4d023f147e6fe69e49ceafa3bf5fabfa
02/09/2012 03:19:35 PM jesse (dot) sandberg (at) verkkokauppa (dot) com Comment #1
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ Group ldap backend listGroups($user) doesn't use filter
Queue ⇒ Horde Framework Packages
Milestone ⇒
Patch ⇒ Yes
New Attachment: 0001-listGroups-user-didn-t-use-this-_filter-and-returned.patch Download
State ⇒ Unconfirmed
Reply to this comment
function listGroups($user) in Horde/Group/Ldap.php doesn't include 
$this->_filter in it's ldap_search()  so it may return entries outside 
of intended group filtering.

Saved Queries