You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hc.apache.org by ol...@apache.org on 2007/10/18 18:22:28 UTC

svn commit: r586026 - in /jakarta/httpcomponents/httpcore/trunk: RELEASE_NOTES.txt module-main/src/main/java/org/apache/http/protocol/ResponseConnControl.java module-main/src/test/java/org/apache/http/protocol/TestStandardInterceptors.java

Author: olegk
Date: Thu Oct 18 09:22:27 2007
New Revision: 586026

URL: http://svn.apache.org/viewvc?rev=586026&view=rev
Log:
Do not include 'Connection: close' to 500 responses per default

Modified:
    jakarta/httpcomponents/httpcore/trunk/RELEASE_NOTES.txt
    jakarta/httpcomponents/httpcore/trunk/module-main/src/main/java/org/apache/http/protocol/ResponseConnControl.java
    jakarta/httpcomponents/httpcore/trunk/module-main/src/test/java/org/apache/http/protocol/TestStandardInterceptors.java

Modified: jakarta/httpcomponents/httpcore/trunk/RELEASE_NOTES.txt
URL: http://svn.apache.org/viewvc/jakarta/httpcomponents/httpcore/trunk/RELEASE_NOTES.txt?rev=586026&r1=586025&r2=586026&view=diff
==============================================================================
--- jakarta/httpcomponents/httpcore/trunk/RELEASE_NOTES.txt (original)
+++ jakarta/httpcomponents/httpcore/trunk/RELEASE_NOTES.txt Thu Oct 18 09:22:27 2007
@@ -1,5 +1,8 @@
 Changes since 4.0 Alpha 6
 
+* Do not include "Connection: close" to 500 responses per default. 
+  Contributed by Oleg Kalnichevski <olegk at apache.org>
+
 * [HTTPCORE-121] new interface HeaderElementIterator
   Contributed by Andrea Selva <selva.andre at gmail.com>
 

Modified: jakarta/httpcomponents/httpcore/trunk/module-main/src/main/java/org/apache/http/protocol/ResponseConnControl.java
URL: http://svn.apache.org/viewvc/jakarta/httpcomponents/httpcore/trunk/module-main/src/main/java/org/apache/http/protocol/ResponseConnControl.java?rev=586026&r1=586025&r2=586026&view=diff
==============================================================================
--- jakarta/httpcomponents/httpcore/trunk/module-main/src/main/java/org/apache/http/protocol/ResponseConnControl.java (original)
+++ jakarta/httpcomponents/httpcore/trunk/module-main/src/main/java/org/apache/http/protocol/ResponseConnControl.java Thu Oct 18 09:22:27 2007
@@ -74,7 +74,6 @@
         		status == HttpStatus.SC_LENGTH_REQUIRED ||
         		status == HttpStatus.SC_REQUEST_TOO_LONG ||
         		status == HttpStatus.SC_REQUEST_URI_TOO_LONG ||
-        		status == HttpStatus.SC_INTERNAL_SERVER_ERROR ||
         		status == HttpStatus.SC_SERVICE_UNAVAILABLE ||
         		status == HttpStatus.SC_NOT_IMPLEMENTED) {
             response.setHeader(HTTP.CONN_DIRECTIVE, HTTP.CONN_CLOSE);

Modified: jakarta/httpcomponents/httpcore/trunk/module-main/src/test/java/org/apache/http/protocol/TestStandardInterceptors.java
URL: http://svn.apache.org/viewvc/jakarta/httpcomponents/httpcore/trunk/module-main/src/test/java/org/apache/http/protocol/TestStandardInterceptors.java?rev=586026&r1=586025&r2=586026&view=diff
==============================================================================
--- jakarta/httpcomponents/httpcore/trunk/module-main/src/test/java/org/apache/http/protocol/TestStandardInterceptors.java (original)
+++ jakarta/httpcomponents/httpcore/trunk/module-main/src/test/java/org/apache/http/protocol/TestStandardInterceptors.java Thu Oct 18 09:22:27 2007
@@ -550,7 +550,6 @@
                 HttpStatus.SC_LENGTH_REQUIRED,
                 HttpStatus.SC_REQUEST_TOO_LONG,
                 HttpStatus.SC_REQUEST_URI_TOO_LONG,
-                HttpStatus.SC_INTERNAL_SERVER_ERROR,
                 HttpStatus.SC_SERVICE_UNAVAILABLE,
                 HttpStatus.SC_NOT_IMPLEMENTED };