| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 1.6.4 |
| Bug #10125 | LDAP fetchData only work for 1 entry | ||
|---|---|---|---|
| Submitted: | 2007-02-19 15:28 UTC | ||
| From: | brunovern dot a at gmail dot com | Assigned: | aashley |
| Status: | Closed | Package: | Auth (version 1.5.0) |
| PHP Version: | 5.2.1 | OS: | w2k3 |
| Roadmaps: | 1.5.1 | ||
[2007-02-19 15:28 UTC] brunovern dot a at gmail dot com
(Bruno Vernay)
Description:
------------
The fetchData function in Auth/container/LDAP.php line 547 contains an obvious error.
547 function fetchData($username, $password)
580 $attributes = $this->options['attributes'];
600 do {
614 if (is_array($attributes) ...
620 if ($attributes = @ldap_get_attributes(...
683 } while ...
There is a confusion between the attributes to fetch as defined in the options and the attributes fetched in the LDAP.
The test line 614 could be precalculated, since it doesn't change in the "do while".
Comments
[2007-02-23 00:43 UTC] aashley at php dot net
(Adam Ashley)
This bug has been fixed in CVS.
If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET).
If this was a problem with the pear.php.net website, the change should be live shortly.
Otherwise, the fix will appear in the package's next release.
Thank you for the report and for helping us make PEAR better.
[2024-12-15 15:22 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!