svn commit: r1361615 - /thrift/trunk/lib/java/src/org/apache/thrift/transport/TSaslTransport.java

[jfarrell]

Author: jfarrell
Date: Sat Jul 14 23:56:20 2012
New Revision: 1361615

URL: http://svn.apache.org/viewvc?rev=1361615&view=rev
Log:
Thrift-1652: TSaslTransport does not log the error when kerberos auth fails
Client: java
Patch: Rohini Palaniswamy

The actual kerberos authentication failure is not logged and it is very 
difficult to find the actual reason for the failure.


Modified:
    thrift/trunk/lib/java/src/org/apache/thrift/transport/TSaslTransport.java

Modified: 
thrift/trunk/lib/java/src/org/apache/thrift/transport/TSaslTransport.java
URL: 
http://svn.apache.org/viewvc/thrift/trunk/lib/java/src/org/apache/thrift/transport/TSaslTransport.java?rev=1361615&r1=1361614&r2=1361615&view=diff
==============================================================================
--- thrift/trunk/lib/java/src/org/apache/thrift/transport/TSaslTransport.java 
(original)
+++ thrift/trunk/lib/java/src/org/apache/thrift/transport/TSaslTransport.java 
Sat Jul 14 23:56:20 2012
@@ -293,6 +293,7 @@ abstract class TSaslTransport extends TT
       }
     } catch (SaslException e) {
       try {
+        LOGGER.error("SASL negotiation failure", e);
         sendAndThrowMessage(NegotiationStatus.BAD, e.getMessage());
       } finally {
         underlyingTransport.close();