collective / pas.plugins.ldap

Zope (and Plone) PAS Plugin providing users and groups from LDAP directory
http://pypi.python.org/pypi/pas.plugins.ldap
Other
13 stars 20 forks source link

Browsing group users may raise SIZELIMIT_EXCEEDED: {'desc': 'Size limit exceeded'} #29

Closed datakurre closed 7 years ago

datakurre commented 7 years ago

Viewing groups members does not yet use paged searches and may raise exception on large groups (when server requires paging):

Traceback (innermost last):
  Module ZPublisher.Publish, line 138, in publish
  Module ZPublisher.mapply, line 77, in mapply
  Module ZPublisher.Publish, line 48, in call_object
  Module Products.CMFPlone.controlpanel.browser.usergroups_groupmembership, line 76, in __call__
  Module Products.CMFPlone.controlpanel.browser.usergroups_groupmembership, line 73, in update
  Module Products.CMFPlone.controlpanel.browser.usergroups_groupmembership, line 83, in getMembers
  Module Products.PlonePAS.tools.groups, line 315, in getGroupMembers
  Module pas.plugins.ldap.plugin, line 675, in getGroupMembers
  Module node.ext.ldap.ugm._api, line 328, in member_ids
  Module node.utils, line 225, in wrapped
  Module node.ext.ldap._node, line 494, in search
  Module node.ext.ldap.session, line 62, in search
  Module node.ext.ldap.base, line 249, in search
  Module bda.cache.memcached, line 80, in getData
  Module node.ext.ldap.base, line 217, in _search
  Module ldap.ldapobject, line 476, in result3
  Module ldap.ldapobject, line 483, in result4
  Module ldap.ldapobject, line 106, in _ldap_call
SIZELIMIT_EXCEEDED: {'desc': 'Size limit exceeded'}
jensens commented 7 years ago

indeed there is an missing paging.

jensens commented 7 years ago

try to fix this in https://github.com/bluedynamics/node.ext.ldap/pull/15

datakurre commented 7 years ago

@jensens Nice. That pull seems to also fix #27

rnixx commented 7 years ago

Fixed via https://github.com/bluedynamics/node.ext.ldap/pull/15