Mercurial > dovecot > original-hg > dovecot-1.2

changeset 4750:c02b31f07299 HEAD

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Do better logging.
author Timo Sirainen <tss@iki.fi>
date Sat, 04 Nov 2006 22:22:31 +0200
parents 2b899c97de29
children aadf124d273f
files src/auth/passdb-ldap.c src/auth/userdb-ldap.c
diffstat 2 files changed, 12 insertions(+), 6 deletions(-) [+]
line wrap: on
line diff
--- a/src/auth/passdb-ldap.c	Sat Nov 04 22:09:57 2006 +0200
+++ b/src/auth/passdb-ldap.c	Sat Nov 04 22:22:31 2006 +0200
@@ -135,7 +135,7 @@
 
 	if (ctx.debug != NULL) {
 		auth_request_log_debug(auth_request, "ldap",
-				       "%s", str_c(ctx.debug));
+				       "result: %s", str_c(ctx.debug));
 	}
 }
 
@@ -155,7 +155,8 @@
 		ret = ldap_result2error(conn->ld, res, 0);
 		if (ret != LDAP_SUCCESS) {
 			auth_request_log_error(auth_request, "ldap",
-					       "ldap_search() failed: %s",
+					       "ldap_search(%s) failed: %s",
+					       request->request.filter,
 					       ldap_err2string(ret));
 		} else {
 			/* get the reply */
@@ -265,6 +266,9 @@
 		return;
 	}
 
+	auth_request_log_debug(auth_request, "ldap", "bind: dn=%s",
+			       ldap_request->base);
+
 	/* Bind started */
 	auth_request_ref(auth_request);
 	hash_insert(conn->requests, POINTER_CAST(msgid), ldap_request);
@@ -286,9 +290,11 @@
 		ret = ldap_result2error(conn->ld, res, 0);
 		if (ret == LDAP_SUCCESS)
 			passdb_result = PASSDB_RESULT_OK;
-		else if (ret == LDAP_INVALID_CREDENTIALS)
+		else if (ret == LDAP_INVALID_CREDENTIALS) {
+			auth_request_log_info(auth_request, "ldap",
+					      "invalid credentials");
 			passdb_result = PASSDB_RESULT_PASSWORD_MISMATCH;
-		else {
+		} else {
 			auth_request_log_error(auth_request, "ldap",
 					       "ldap_bind() failed: %s",
 					       ldap_err2string(ret));
@@ -358,7 +364,7 @@
 	ldap_request->context = auth_request;
 	ldap_request->attributes = conn->pass_attr_names;
 
-	auth_request_log_debug(auth_request, "ldap",
+	auth_request_log_debug(auth_request, "ldap", "pass search: "
 			       "base=%s scope=%s filter=%s fields=%s",
 			       ldap_request->base, conn->set.scope,
 			       ldap_request->filter,
--- a/src/auth/userdb-ldap.c	Sat Nov 04 22:09:57 2006 +0200
+++ b/src/auth/userdb-ldap.c	Sat Nov 04 22:22:31 2006 +0200
@@ -197,7 +197,7 @@
 
 	request->request.attributes = conn->user_attr_names;
 
-	auth_request_log_debug(auth_request, "ldap",
+	auth_request_log_debug(auth_request, "ldap", "user search: "
 			       "base=%s scope=%s filter=%s fields=%s",
 			       request->request.base, conn->set.scope,
 			       request->request.filter,