Return-Path: Delivered-To: apmail-httpd-bugs-archive@www.apache.org Received: (qmail 64178 invoked from network); 8 Sep 2005 20:57:22 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 8 Sep 2005 20:57:22 -0000 Received: (qmail 37257 invoked by uid 500); 8 Sep 2005 20:57:21 -0000 Delivered-To: apmail-httpd-bugs-archive@httpd.apache.org Received: (qmail 37226 invoked by uid 500); 8 Sep 2005 20:57:21 -0000 Mailing-List: contact bugs-help@httpd.apache.org; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: Reply-To: "Apache HTTPD Bugs Notification List" List-Id: Delivered-To: mailing list bugs@httpd.apache.org Received: (qmail 37211 invoked by uid 99); 8 Sep 2005 20:57:21 -0000 X-ASF-Spam-Status: No, hits=0.2 required=10.0 tests=NO_REAL_NAME X-Spam-Check-By: apache.org Received: from [192.87.106.226] (HELO ajax.apache.org) (192.87.106.226) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 08 Sep 2005 13:57:21 -0700 Received: by ajax.apache.org (Postfix, from userid 99) id 6986F187; Thu, 8 Sep 2005 22:57:20 +0200 (CEST) From: bugzilla@apache.org To: bugs@httpd.apache.org Subject: DO NOT REPLY [Bug 36564] - make mod_ldap really universal X-Bugzilla-Reason: AssignedTo Message-Id: <20050908205720.6986F187@ajax.apache.org> Date: Thu, 8 Sep 2005 22:57:20 +0200 (CEST) X-Virus-Checked: Checked by ClamAV on apache.org X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG� RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT . ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND� INSERTED IN THE BUG DATABASE. http://issues.apache.org/bugzilla/show_bug.cgi?id=36564 ------- Additional Comments From ondrej@sury.org 2005-09-08 22:57 ------- You're right that it's pure cosmetic issue, I am merely talking about these lines: util_ldap.c:1057 ldc->reason = "ldap_search_ext_s() for user failed with server down"; util_ldap.c:1064 ldc->reason = "ldap_search_ext_s() for user failed"; util_ldap.c:1076 ldc->reason = "User not found"; util_ldap.c:1078 ldc->reason = "User is not unique (search found two or more matches)"; Maybe simple replace user to entry will do the trick? (I think snprintf warnings are overkill :-) I thought about: renaming util_ldap_cache_getuserdn to util_ldap_cache_find, replacing user to entry and creating: LDAP_DECLARE(int) util_ldap_cache_userdn(request_rec *r, util_ldap_connection_t *ldc, const char *url, const char *basedn, int scope, char **attrs, const char *filter, const char **binddn, const char ***retvals) { return util_ldap_cache_find(r, ldc, url, basedn, scope, attrs, filter, binddn, retvals); } *** In fact it's just my feeling doing something wrong when using getuserdn, so I would welcome to clean it a bit, but I would not protest if you just close it as WONTFIX bug. Ondrej -- Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee. --------------------------------------------------------------------- To unsubscribe, e-mail: bugs-unsubscribe@httpd.apache.org For additional commands, e-mail: bugs-help@httpd.apache.org