search

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

Connection Test may give ERROR: LDAP users; {'desc': 'Size limit exceeded'} #27

Closed datakurre closed 7 years ago

datakurre commented 7 years ago

When the LDAP directory is big enough to require paging.

That seems to be, because node.ext.ldap has API not always passing page_size for LDAP searches.

I don't know yet, if that really affects the use of pas.plugins.ldap. Should know early next week.

datakurre commented 7 years ago

It seems that calls through PAS have page_size passed, so this errors only occurs on connection test (when configuring PAS plugin).

jensens commented 7 years ago

Good catch, connection test should pass page_size as well.

jensens commented 7 years ago

Also ldap explorer should pass it (need to check).

rnixx commented 7 years ago

page_size setting should be properly set with a default of 1000. @datakurre Has your LDAP server a lower limit?

rnixx commented 7 years ago

well, it looks like this search also needs page size passed https://github.com/bluedynamics/node.ext.ldap/blob/master/src/node/ext/ldap/ugm/_api.py#L499

datakurre commented 7 years ago

@rnixx page_size 1000 is good default, but is not passed for search in connection test.

rnixx commented 7 years ago

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