From 25669bf85e5f2bb3f9ba5a6c9c43e149b07095a9 Mon Sep 17 00:00:00 2001
From: ian.packer <ian.packer@forgerock.com>
Date: Mon, 30 Nov 2015 15:50:55 +0000
Subject: [PATCH] OPENDJ-2327: Improve log messages on certain 'server error' disconnects
---
opendj-server-legacy/src/main/java/org/opends/server/protocols/ldap/LDAPRequestHandler.java | 28 +++++++---------------------
1 files changed, 7 insertions(+), 21 deletions(-)
diff --git a/opendj-server-legacy/src/main/java/org/opends/server/protocols/ldap/LDAPRequestHandler.java b/opendj-server-legacy/src/main/java/org/opends/server/protocols/ldap/LDAPRequestHandler.java
index fce174f..3a23f1a 100644
--- a/opendj-server-legacy/src/main/java/org/opends/server/protocols/ldap/LDAPRequestHandler.java
+++ b/opendj-server-legacy/src/main/java/org/opends/server/protocols/ldap/LDAPRequestHandler.java
@@ -289,27 +289,13 @@
{
clientConnection = (LDAPClientConnection) key.attachment();
- try
+ int readResult = clientConnection.processDataRead();
+ if (readResult < 0)
{
- int readResult = clientConnection.processDataRead();
- if (readResult < 0)
- {
- key.cancel();
- }
- if (readResult > 0) {
- readyConnections.add(clientConnection);
- }
- }
- catch (Exception e)
- {
- logger.traceException(e);
-
- // Some other error occurred while we were trying to read data
- // from the client.
- // FIXME -- Should we log this?
key.cancel();
- clientConnection.disconnect(DisconnectReason.SERVER_ERROR,
- false, null);
+ }
+ if (readResult > 0) {
+ readyConnections.add(clientConnection);
}
}
catch (Exception e)
@@ -323,8 +309,8 @@
if (clientConnection != null)
{
- clientConnection.disconnect(DisconnectReason.SERVER_ERROR,
- false, null);
+ clientConnection.disconnect(DisconnectReason.SERVER_ERROR, false,
+ ERR_UNEXPECTED_EXCEPTION_ON_CLIENT_CONNECTION.get(getExceptionMessage(e)));
}
}
}
--
Gitblit v1.10.0