You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by yo...@apache.org on 2007/03/26 01:05:23 UTC
svn commit: r522378 - in /tomcat:
connectors/trunk/http11/src/java/org/apache/coyote/http11/Http11Processor.java
container/tc5.5.x/webapps/docs/changelog.xml
Author: yoavs
Date: Sun Mar 25 16:05:22 2007
New Revision: 522378
URL: http://svn.apache.org/viewvc?view=rev&rev=522378
Log:
Bugzilla 41675: add a couple of DEBUG-level logging statements to Http11Processor.java when sending error responses.
Modified:
tomcat/connectors/trunk/http11/src/java/org/apache/coyote/http11/Http11Processor.java
tomcat/container/tc5.5.x/webapps/docs/changelog.xml
Modified: tomcat/connectors/trunk/http11/src/java/org/apache/coyote/http11/Http11Processor.java
URL: http://svn.apache.org/viewvc/tomcat/connectors/trunk/http11/src/java/org/apache/coyote/http11/Http11Processor.java?view=diff&rev=522378&r1=522377&r2=522378
==============================================================================
--- tomcat/connectors/trunk/http11/src/java/org/apache/coyote/http11/Http11Processor.java (original)
+++ tomcat/connectors/trunk/http11/src/java/org/apache/coyote/http11/Http11Processor.java Sun Mar 25 16:05:22 2007
@@ -841,6 +841,10 @@
log.debug(sm.getString("http11processor.header.parse"), t);
}
// 400 - Bad Request
+ if (log.isDebugEnabled()) {
+ log.debug(sm.getString("http11processor.request.prepare")+
+ " host header missing");
+ }
response.setStatus(400);
error = true;
}
@@ -1203,6 +1207,10 @@
http11 = false;
error = true;
// Send 505; Unsupported HTTP version
+ if (log.isDebugEnabled()) {
+ log.debug(sm.getString("http11processor.request.prepare")+
+ " Unsupported HTTP version \""+protocolMB+"\"");
+ }
response.setStatus(505);
}
@@ -1311,6 +1319,10 @@
// Unsupported transfer encoding
error = true;
// 501 - Unimplemented
+ if (log.isDebugEnabled()) {
+ log.debug(sm.getString("http11processor.request.prepare")+
+ " Unsupported transfer encoding \""+encodingName+"\"");
+ }
response.setStatus(501);
}
}
Modified: tomcat/container/tc5.5.x/webapps/docs/changelog.xml
URL: http://svn.apache.org/viewvc/tomcat/container/tc5.5.x/webapps/docs/changelog.xml?view=diff&rev=522378&r1=522377&r2=522378
==============================================================================
--- tomcat/container/tc5.5.x/webapps/docs/changelog.xml (original)
+++ tomcat/container/tc5.5.x/webapps/docs/changelog.xml Sun Mar 25 16:05:22 2007
@@ -58,6 +58,10 @@
<bug>40960</bug> Inconsistent exception type thrown on socket timeout in
InternalAprInputBuffer. Patch by Christophe Pierret. (yoavs)
</fix>
+ <add>
+ <bug>41675</bug> Add a couple of DEBUG-level logging statements to Http11Processors
+ when sending error responses. Patch by Ralf Hauser. (yoavs)
+ </add>
</changelog>
</subsection>
<subsection name="Cluster">
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org