Re: UPDATED: Patch to fix BUG: ldap_search: sizelimit exceeded, no result..
| From: | Stig Venaas | Date: | Thu, 20 Jul 2000 11:12:42 +0000 |
| Subject: | Re: UPDATED: Patch to fix BUG: ldap_search: sizelimit exceeded, no result.. | ||
| References: | 1 2 | Groups: | php.dev |
| Request: | Send a blank email to php-dev+get-25202@lists.php.net to get a copy of this message | ||
OK, I see, sorry if I was a bit harsh.
Just one small point, the C API also allows for getting just the attributes,
this is something that might be useful, at leat I have an application where
it could be used. The C API is like
int ldap_search(
LDAP *ld,
char *base,
int scope,
char *filter,
char *attrs[],
int attrsonly
where ld can also specify sizelimit, timelimit and deref.
PHP's ldap_search is now
int ldap_search (int link_identifier, string base_dn, string filter [, array attributes])
Wouldn't it be better if we changed ldap_search into
int ldap_search (int link_identifier, string base_dn, string filter [, array attributes, int
attrsonly, int sizelimit, int timelimit, int deref])
What do you and others think of that?
In short, I think your patch is useful and should be added (haven't studied
the code though), but I want attrsonly to be added at the same time. That's
done in a minute. I'll do it myself if you don't.
Unless anyone else does, I'll apply your patch and possible additions in
a day or two, I'm a bit busy right now.
Stig