On Jun 24, 2010, at 5:47 PM, tmortagne (SVN) wrote:
Author: tmortagne
Date: 2010-06-24 17:47:15 +0200 (Thu, 24 Jun 2010)
New Revision: 29712
Modified:
platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/user/impl/LDAP/XWikiLDAPAuthServiceImpl.java
Log:
[misc] More useful debug log
Modified:
platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/user/impl/LDAP/XWikiLDAPAuthServiceImpl.java
===================================================================
---
platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/user/impl/LDAP/XWikiLDAPAuthServiceImpl.java 2010-06-24
15:43:19 UTC (rev 29711)
+++
platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/user/impl/LDAP/XWikiLDAPAuthServiceImpl.java 2010-06-24
15:47:15 UTC (rev 29712)
@@ -416,6 +416,10 @@
if ("1".equals(config.getLDAPParam("ldap_validate_password",
"0", context))) {
String passwordField = config.getLDAPParam("ldap_password_field",
"userPassword", context);
if (!connector.checkPassword(ldapDn, password, passwordField)) {
+ LOG.debug("Password comparision failed, are you really sure you
need validate_password ?" +
typo: comparison.
-Vincent
+ " If you don't enable it
it does not mean use credentiel are not validated." +
+ " The goal of this property is to bypass standard LDAP
bind.");
+
throw new XWikiException(XWikiException.MODULE_XWIKI_USER,
XWikiException.ERROR_XWIKI_USER_INIT,
"LDAP authentication failed:" + " could not validate
the password: wrong password for " + ldapDn);
}