Author: markt
Date: Thu Sep 19 14:53:59 2013
New Revision: 1524763

URL: http://svn.apache.org/r1524763
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=55570
Correct log exceptions so they appear in the logs

Added:
    tomcat/tc7.0.x/trunk/conf/Tomcat/
    tomcat/tc7.0.x/trunk/conf/Tomcat/localhost/
    tomcat/tc7.0.x/trunk/embed-webapps/
    tomcat/tc7.0.x/trunk/embed-webapps/examples.war   (with props)
Modified:
    tomcat/tc7.0.x/trunk/   (props changed)
    
tomcat/tc7.0.x/trunk/java/org/apache/catalina/authenticator/SpnegoAuthenticator.java
    tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc7.0.x/trunk/
------------------------------------------------------------------------------
  Merged /tomcat/trunk:r1524761

Added: tomcat/tc7.0.x/trunk/embed-webapps/examples.war
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/embed-webapps/examples.war?rev=1524763&view=auto
==============================================================================
Binary file - no diff available.

Propchange: tomcat/tc7.0.x/trunk/embed-webapps/examples.war
------------------------------------------------------------------------------
    svn:mime-type = application/octet-stream

Modified: 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/authenticator/SpnegoAuthenticator.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/catalina/authenticator/SpnegoAuthenticator.java?rev=1524763&r1=1524762&r2=1524763&view=diff
==============================================================================
--- 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/authenticator/SpnegoAuthenticator.java
 (original)
+++ 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/authenticator/SpnegoAuthenticator.java
 Thu Sep 19 14:53:59 2013
@@ -251,14 +251,13 @@ public class SpnegoAuthenticator extends
                     isStoreDelegatedCredential());
         } catch (GSSException e) {
             if (log.isDebugEnabled()) {
-                
log.debug(sm.getString("spnegoAuthenticator.ticketValidateFail",
-                        e));
+                
log.debug(sm.getString("spnegoAuthenticator.ticketValidateFail"), e);
             }
             response.setHeader("WWW-Authenticate", "Negotiate");
             response.sendError(HttpServletResponse.SC_UNAUTHORIZED);
             return false;
         } catch (PrivilegedActionException e) {
-            log.error(sm.getString("spnegoAuthenticator.serviceLoginFail", e));
+            log.error(sm.getString("spnegoAuthenticator.serviceLoginFail"), e);
             response.setHeader("WWW-Authenticate", "Negotiate");
             response.sendError(HttpServletResponse.SC_UNAUTHORIZED);
             return false;

Modified: tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml?rev=1524763&r1=1524762&r2=1524763&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Thu Sep 19 14:53:59 2013
@@ -55,7 +55,7 @@
   They eventually become mixed with the numbered issues. (I.e., numbered
   issues to not "pop up" wrt. others).
 -->
-<section name="Tomcat 7.0.43 (markt)">
+<section name="Tomcat 7.0.43 (voiletagg)">
   <subsection name="Catalina">
     <changelog>
       <add>
@@ -175,6 +175,10 @@
           <code>org.apache.juli.ClassLoaderLogManager.debug=true</code>
           is set.
       </add>
+      <fix>
+        <bug>55570</bug>: Correctly log exceptions for all error conditions in
+        the SPNEGO authenticator. (markt)
+      </fix>
     </changelog>
   </subsection>
   <subsection name="Coyote">



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to