Log stacktrace

Adds more info on catch to be put into the log instead of just
console.  This will help debug issues when one has access to grid
hub log but no access to console.

Removes now superfluous t.printStackTrace().

Signed-off-by: Andreas Tolfsen <[email protected]>

Author: dima-groupon

Diff for: java/server/src/org/openqa/grid/internal/TestSlot.java

@@ -16,6 +16,8 @@
 */
 package org.openqa.grid.internal;
 
+import com.google.common.base.Throwables;
+
 import org.openqa.grid.common.SeleniumProtocol;
 import org.openqa.grid.common.exception.GridException;
 import org.openqa.grid.internal.listeners.TestSessionListener;
@@ -219,8 +221,9 @@ boolean performAfterSessionEvent() {
         ((TestSessionListener) proxy).afterSession(currentSession);
       }
     } catch (Throwable t) {
-      log.severe("Error running afterSession for " + currentSession + " the test slot is now dead.");
-      t.printStackTrace();
+      log.severe(String.format(
+          "Error running afterSession for %s, the test slot is now dead: %s\n%s", currentSession,
+          t.getMessage(), Throwables.getStackTraceAsString(t)));
       return false;
     }
     return true;