Author: remm Date: Tue Oct 16 04:22:50 2007 New Revision: 585118 URL: http://svn.apache.org/viewvc?rev=585118&view=rev Log: - Commit patches which have been in the status for some time: * Fix explicit flush before response commit in the org.apache.jk AJP connector. http://svn.apache.org/viewvc?view=rev&revision=580815
* Use newer eclipse jdt - old location is gone (reported by Jason Britian via email) update to jdt.loc=http://sunsite.informatik.rwth-aachen.de/eclipse/downloads/drops/R-3.3.1-200709211145/eclipse-JDT-3.3.1.zip * Call StopAwait at StandardServer.stop as port==-1 * Arrange doc of connectors. http://people.apache.org/~jfclere/patches/tc.docs.patch Modified: tomcat/tc6.0.x/trunk/STATUS tomcat/tc6.0.x/trunk/build.properties.default tomcat/tc6.0.x/trunk/java/org/apache/catalina/core/StandardServer.java tomcat/tc6.0.x/trunk/java/org/apache/jk/core/MsgContext.java tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml tomcat/tc6.0.x/trunk/webapps/docs/config/ajp.xml tomcat/tc6.0.x/trunk/webapps/docs/config/http.xml Modified: tomcat/tc6.0.x/trunk/STATUS URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS?rev=585118&r1=585117&r2=585118&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/STATUS (original) +++ tomcat/tc6.0.x/trunk/STATUS Tue Oct 16 04:22:50 2007 @@ -25,50 +25,6 @@ PATCHES PROPOSED TO BACKPORT: [ New proposals should be added at the end of the list ] -* Fix explicit flush before response commit in the org.apache.jk AJP connector. - http://svn.apache.org/viewvc?view=rev&revision=580815 - +1: remm, pero, funkman - -1: - -* Use newer eclipse jdt - old location is gone (reported by Jason Britian via email) - update to jdt.loc=http://sunsite.informatik.rwth-aachen.de/eclipse/downloads/drops/R-3.3.1-200709211145/eclipse-JDT-3.3.1.zip - +1: pero, funkman, remm - -1: - -* Call StopAwait at StandardServer.stop as port==-1 - -Index: java/org/apache/catalina/core/StandardServer.java -=================================================================== ---- java/org/apache/catalina/core/StandardServer.java (revision 583866) -+++ java/org/apache/catalina/core/StandardServer.java (working copy) -@@ -360,7 +360,7 @@ - if( port==-1 ) { - while( true ) { - try { -- Thread.sleep( 100000 ); -+ Thread.sleep( 10000 ); - } catch( InterruptedException ex ) { - } - if( stopAwait ) return; -@@ -747,6 +747,9 @@ - // Notify our interested LifecycleListeners - lifecycle.fireLifecycleEvent(AFTER_STOP_EVENT, null); - -+ if (port == -1) -+ stopAwait(); -+ - } - - public void init() throws Exception { - - +1: pero, remm, fhanik - -1: - -* Arrange doc of connectors. - http://people.apache.org/~jfclere/patches/tc.docs.patch - +1: jfclere, remm, markt - -1: - * Make server.xml parsing warnings more generic, add support for the NIO connector * Connector is no longer an exception case. Any new component can still use setProperty and return a boolean * to accept or reject the property @@ -88,5 +44,5 @@ * Fix important vulnerability when webdav is enabled for write Patch: http://marc.info/?l=tomcat-dev&m=119245116910632&w=2 - +1: markt,funkman - -1: + +1: markt, funkman, remm + -1: Modified: tomcat/tc6.0.x/trunk/build.properties.default URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/build.properties.default?rev=585118&r1=585117&r2=585118&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/build.properties.default (original) +++ tomcat/tc6.0.x/trunk/build.properties.default Tue Oct 16 04:22:50 2007 @@ -57,7 +57,7 @@ jdt.home=${base.path}/eclipse/plugins jdt.lib=${jdt.home} jdt.jar=${jdt.lib}/org.eclipse.jdt.core_3.2.3.v_686_R32x.jar -jdt.loc=http://sunsite.informatik.rwth-aachen.de/eclipse/downloads/drops/R-3.2.2-200702121330/eclipse-JDT-3.2.2.zip +jdt.loc=http://sunsite.informatik.rwth-aachen.de/eclipse/downloads/drops/R-3.3.1-200709211145/eclipse-JDT-3.3.1.zip # ----- Tomcat native library ----- tomcat-native.home=${base.path}/tomcat-native-1.1.10 Modified: tomcat/tc6.0.x/trunk/java/org/apache/catalina/core/StandardServer.java URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/catalina/core/StandardServer.java?rev=585118&r1=585117&r2=585118&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/java/org/apache/catalina/core/StandardServer.java (original) +++ tomcat/tc6.0.x/trunk/java/org/apache/catalina/core/StandardServer.java Tue Oct 16 04:22:50 2007 @@ -360,7 +360,7 @@ if( port==-1 ) { while( true ) { try { - Thread.sleep( 100000 ); + Thread.sleep( 10000 ); } catch( InterruptedException ex ) { } if( stopAwait ) return; @@ -746,6 +746,9 @@ // Notify our interested LifecycleListeners lifecycle.fireLifecycleEvent(AFTER_STOP_EVENT, null); + + if (port == -1) + stopAwait(); } Modified: tomcat/tc6.0.x/trunk/java/org/apache/jk/core/MsgContext.java URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/jk/core/MsgContext.java?rev=585118&r1=585117&r2=585118&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/java/org/apache/jk/core/MsgContext.java (original) +++ tomcat/tc6.0.x/trunk/java/org/apache/jk/core/MsgContext.java Tue Oct 16 04:22:50 2007 @@ -275,12 +275,15 @@ } else if( actionCode==ActionCode.ACTION_CLIENT_FLUSH ) { if( log.isDebugEnabled() ) log.debug("CLIENT_FLUSH " ); + Response res = (Response)param; + if(!res.isCommitted()) { + action(ActionCode.ACTION_COMMIT, res); + } try { source.flush( null, this ); } catch(IOException iex) { // This is logged elsewhere, so debug only here log.debug("Error during flush",iex); - Response res = (Response)param; res.setErrorException(iex); setStatus(JK_STATUS_ERROR); } Modified: tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml?rev=585118&r1=585117&r2=585118&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml (original) +++ tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Tue Oct 16 04:22:50 2007 @@ -33,6 +33,13 @@ <body> <section name="Tomcat 6.0.15 (remm)"> + <subsection name="General"> + <changelog> + <fix> + Use Eclipse JDT 3.3.1. (pero) + </fix> + </changelog> + </subsection> <subsection name="Catalina"> <changelog> <fix><bug>43487</bug> @@ -110,6 +117,12 @@ org.apache.catalina.core.StandardContext.findStatusPage(int) (funkman) </fix> + <fix> + Fix important vulnerability when webdav is enabled for write. (markt) + </fix> + <fix> + Call stopAwait in StandardServer.stop if port == -1. (pero) + </fix> </changelog> </subsection> <subsection name="Coyote"> @@ -127,6 +140,9 @@ <fix> <bug>42925</bug>: Add maintain for sendfile. (remm) </fix> + <fix> + Fix explicit flush before response commit in the org.apache.jk AJP connector. (pero) + </fix> </changelog> </subsection> <subsection name="Jasper"> @@ -145,6 +161,9 @@ <fix> <bug>42979</bug>: Update sample.war to include recent security fixes in the source code. (markt) + </fix> + <fix> + Minor connector doc fix. (jfclere) </fix> </changelog> </subsection> Modified: tomcat/tc6.0.x/trunk/webapps/docs/config/ajp.xml URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/webapps/docs/config/ajp.xml?rev=585118&r1=585117&r2=585118&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/webapps/docs/config/ajp.xml (original) +++ tomcat/tc6.0.x/trunk/webapps/docs/config/ajp.xml Tue Oct 16 04:22:50 2007 @@ -77,9 +77,7 @@ <attribute name="emptySessionPath" required="false"> <p>If set to <code>true</code>, all paths for session cookies will be set - to <code>/</code>. This can be useful for portlet specification implementations, - but will greatly affect performance if many applications are accessed on a given - server by the client. + to <code>/</code>. This can be useful for portlet specification implementations. If not specified, this attribute is set to <code>false</code>.</p> </attribute> Modified: tomcat/tc6.0.x/trunk/webapps/docs/config/http.xml URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/webapps/docs/config/http.xml?rev=585118&r1=585117&r2=585118&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/webapps/docs/config/http.xml (original) +++ tomcat/tc6.0.x/trunk/webapps/docs/config/http.xml Tue Oct 16 04:22:50 2007 @@ -78,9 +78,7 @@ <attribute name="emptySessionPath" required="false"> <p>If set to <code>true</code>, all paths for session cookies will be set - to <code>/</code>. This can be useful for portlet specification implementations, - but will greatly affect performance if many applications are accessed on a given - server by the client. + to <code>/</code>. This can be useful for portlet specification implementations. If not specified, this attribute is set to <code>false</code>.</p> </attribute> --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]