Return-Path: Delivered-To: apmail-tomcat-dev-archive@www.apache.org Received: (qmail 22205 invoked from network); 22 Sep 2010 14:37:27 -0000 Received: from unknown (HELO mail.apache.org) (140.211.11.3) by 140.211.11.9 with SMTP; 22 Sep 2010 14:37:27 -0000 Received: (qmail 42167 invoked by uid 500); 22 Sep 2010 14:37:27 -0000 Delivered-To: apmail-tomcat-dev-archive@tomcat.apache.org Received: (qmail 41809 invoked by uid 500); 22 Sep 2010 14:37:25 -0000 Mailing-List: contact dev-help@tomcat.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "Tomcat Developers List" Delivered-To: mailing list dev@tomcat.apache.org Received: (qmail 41795 invoked by uid 99); 22 Sep 2010 14:37:24 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 22 Sep 2010 14:37:24 +0000 X-ASF-Spam-Status: No, hits=-2000.0 required=10.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.4] (HELO eris.apache.org) (140.211.11.4) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 22 Sep 2010 14:37:22 +0000 Received: by eris.apache.org (Postfix, from userid 65534) id 14A0923888E8; Wed, 22 Sep 2010 14:37:00 +0000 (UTC) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: svn commit: r999983 - in /tomcat/tc6.0.x/trunk: STATUS.txt java/org/apache/tomcat/util/net/AprEndpoint.java webapps/docs/changelog.xml Date: Wed, 22 Sep 2010 14:36:59 -0000 To: dev@tomcat.apache.org From: mturk@apache.org X-Mailer: svnmailer-1.0.8 Message-Id: <20100922143700.14A0923888E8@eris.apache.org> Author: mturk Date: Wed Sep 22 14:36:59 2010 New Revision: 999983 URL: http://svn.apache.org/viewvc?rev=999983&view=rev Log: Backport the unlockAccept Modified: tomcat/tc6.0.x/trunk/STATUS.txt tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Modified: tomcat/tc6.0.x/trunk/STATUS.txt URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=999983&r1=999982&r2=999983&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/STATUS.txt (original) +++ tomcat/tc6.0.x/trunk/STATUS.txt Wed Sep 22 14:36:59 2010 @@ -28,13 +28,6 @@ RELEASE SHOWSTOPPERS: PATCHES ACCEPTED TO BACKPORT: [ start all new proposals below, under PATCHES PROPOSED. ] -* Fix unlockAccept when the soTimeout is -1 (default). - Default conf has soTimeout -1 and that will throw exception if - used in setSoTimeout method. - http://people.apache.org/~mturk/tomcat/patches/tomcat-6.0.x-aprunlockaccept.patch - +1: mturk, kfujino, markt - -1: - PATCHES PROPOSED TO BACKPORT: [ New proposals should be added at the end of the list ] Modified: tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java?rev=999983&r1=999982&r2=999983&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java (original) +++ tomcat/tc6.0.x/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java Wed Sep 22 14:36:59 2010 @@ -901,7 +901,7 @@ public class AprEndpoint { saddr = new InetSocketAddress(address,port); } s = new java.net.Socket(); - s.setSoTimeout(soTimeout); + s.setSoTimeout(soTimeout > 0 ? soTimeout : 60000); s.setSoLinger(true ,0); if (log.isDebugEnabled()) { log.debug("About to unlock socket for: " + saddr); 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=999983&r1=999982&r2=999983&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml (original) +++ tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Wed Sep 22 14:36:59 2010 @@ -78,6 +78,10 @@ + + Avoid a NPE for APR connector unlockAccept with default soTimeout + (mturk). + 48738: Add support for flushing gzipped output. Based on a patch by Jiong Wang. (markt) @@ -92,7 +96,7 @@ - 49343: When ChannelException is thrown, remove listener from + 49343: When ChannelException is thrown, remove listener from channel. (kfujino) @@ -112,7 +116,7 @@ Use native line endings for example Eclipse configuration files in - source distribution. (markt) + source distribution. (markt) @@ -132,7 +136,7 @@ 49598: When session is changed and the session cookie is replaced, ensure that the new Set-Cookie header overwrites the old - Set-Cookie header. (markt) + Set-Cookie header. (markt) Fix order when listing Webapp loader search URLs. (rjung) @@ -287,11 +291,11 @@ Clerc. (markt) - 49018: Fix processing of time argument in the Expire sessions + 49018: Fix processing of time argument in the Expire sessions action in the Manager web application. (kkolinko) - 49116: If session is already invalid, expire session to + 49116: If session is already invalid, expire session to prevent memory leak. (kfujino) @@ -404,11 +408,11 @@ permanently disabled session replication. (fhanik) - 49051: memberAlive is not called if member has not already + 49051: memberAlive is not called if member has not already existed in membership. (kfujino) - 49151: Avoid ClassCastException in BackupManager#stop. + 49151: Avoid ClassCastException in BackupManager#stop. (kfujino) @@ -492,7 +496,7 @@ 48584: Prevent the APR connector logging an error if the - acceptor fails during shutdown since this is expected. (mturk) + acceptor fails during shutdown since this is expected. (mturk) 48660: Using compression should not overwrite any Vary header @@ -507,7 +511,7 @@ correct location when using JspC and allow the option that controls this to be configured on the command line. Also allow the encoding of web.xml to be configured when using JspC and deprecate some unused JspC methods. - (markt/kkolinko) + (markt/kkolinko) 48498: Avoid ArrayIndexOutOfBoundsException triggered by a @@ -520,7 +524,7 @@ 48827: Correct a regression in the fix for 47977 that caused an incorrect non-empty body error to be reported for valid - JSP documents. (markt) + JSP documents. (markt) @@ -655,7 +659,7 @@ 48170: Remove unnecessary synchronization that is causing - issues under load. (markt) + issues under load. (markt) 48580: Prevent AccessControlException if first access is to a @@ -668,7 +672,7 @@ 48616: Don't declare or synchronize scripting variables for JSP fragments since they are scriptless. This is an alternative fix for 42390 that avoids both the original problem and the - regression in the first fix. (kkolinko) + regression in the first fix. (kkolinko) 48627: Fix regression in re-factored EL parsing. Keep @@ -769,14 +773,14 @@ operation. Patch provided by sebb. (markt) - 48417: Update French translations. Patch provided by André + 48417: Update French translations. Patch provided by Andr� Warnier. (markt/kkolinko) 48421: Fix file descriptor and potential memory leak when a web application uses a local logging.properties file. Allow a web applciation's log files to be deleted once the web application has been - stopped. (markt) + stopped. (markt) 48454: Ensure stderr is completely read before terminating @@ -913,7 +917,7 @@ Remove duplicate mime-mapping entries in web.xml. Re-order entries - alphabetically to make it easier to identify duplicates. (markt) + alphabetically to make it easier to identify duplicates. (markt) Use a more sensible default (webapps) for a Host's appBase. @@ -938,7 +942,7 @@ 40380: Fix potential synchronization issue in - StandardSession.expire(). (markt) + StandardSession.expire(). (markt) 41059: Reduce chances of errors when ENABLE_CLEAR_REFERENCES @@ -962,7 +966,7 @@ 45403: Add additional checks on web application deployment - and do not swallow IO errors. (kkolinko) + and do not swallow IO errors. (kkolinko) 45785: Additional fix required for the extension validator. @@ -993,7 +997,7 @@ 47158: Fix some thread safety issues in the AccessLogValve. - (markt) + (markt) 47228: Correct French translations. Patch provided by sebb. @@ -1070,7 +1074,7 @@ 47918: Correct mbean descriptors for the host deployer. Patch - provided by Uwe Günther. (markt) + provided by Uwe G�nther. (markt) 47930: Fix thread safety issues on session swap-in in the @@ -1101,7 +1105,7 @@ 48257: Correct error in Spanish translations. Patch provided - by Guillermo Gutiérrez. (markt) + by Guillermo Guti�rrez. (markt) 48306, 48307: Correct French translations. Patches @@ -1113,7 +1117,7 @@ 48413: Correct some French translations. Patch provided by - André Warnier. (markt) + Andr� Warnier. (markt) Deprecate the caseSensitive option on the @@ -1126,7 +1130,7 @@ Better logging for parameter decoding issues to help identify broken - requests. (markt) + requests. (markt) Update Apache Commons Pool from 1.4 to 1.5.4. This update includes @@ -1156,7 +1160,7 @@ Various JNDI realm improvements for Active Directory. These include the ability to specify a default role, optional handling for nested roles - and an option to ignore PartialResultExceptions (markt). + and an option to ignore PartialResultExceptions (markt). Expose Servlet Filters via JMX. Based on a patch by Xie Xiaodong as part @@ -1215,7 +1219,7 @@ Prevent NPE in JULI during shutdown when resources try to log messages - after JULI has been shutdown. (fhanik/kkolinko) + after JULI has been shutdown. (fhanik/kkolinko) Make the JULI FileHandler easier to extend. (fhanik) @@ -1265,7 +1269,7 @@ Align tcnative native and Java method names. (rjung) - Dont report thread count from connector if an external executor is used. + Dont report thread count from connector if an external executor is used. 39637: Enable the AJP connectors to correctly handle client certificate chains. Patch by Patrik Schnellmann. (markt) @@ -1500,7 +1504,7 @@ 47769: Clarify the JNDI docs with repect to use of <resource-ref> and related elements, specifically when they are - required and when they may be omitted. (markt) + required and when they may be omitted. (markt) 48381: Add information on how Tomcat treats host names to the @@ -1537,7 +1541,7 @@ Don't add blank lines to end of files when fixing line-endings for - tar.gz distribution. (markt) + tar.gz distribution. (markt) Use explicit encoding during filtering operations when building Tomcat @@ -1765,7 +1769,7 @@ 38553: Return 401 rather than 400 if client does not present - a certificate CLIENT-CERT authentication. (markt) + a certificate CLIENT-CERT authentication. (markt) 38570: When checking docBase against appBase, make sure we @@ -1790,7 +1794,7 @@ Fix read/write timeout of async comet operations (r719264) - + Implement async close behaviour for Comet/NIO. No-op for APR (same behavior as before) (r719262) @@ -1865,7 +1869,7 @@ Provide full stacktrace and message when the ErrorReportValveClass can't - be instantiated. (funkman) + be instantiated. (funkman) 45608: Make allocated servlet count synchronized to ensure @@ -1874,7 +1878,7 @@ 45628: When checking MANIFEST dependancies, JARs without - dependencies should allows be considered to be full-filled. (markt) + dependencies should allows be considered to be full-filled. (markt) 45735: Improve ETag handling. (remm) @@ -1897,7 +1901,7 @@ Add the CombinedRealm that enables authentication to be attempted - against multiple realms. (markt) + against multiple realms. (markt) Add the LockOutRealm that enables a standard Realm to be wrapped with @@ -2013,7 +2017,7 @@ Fix possible synchronisation bottleneck in cookie creation. (markt) - Fix various spelling errors reported on the mailing lists. (markt) + Fix various spelling errors reported on the mailing lists. (markt) Make the logging manager and properties file configurable via @@ -2162,7 +2166,7 @@ 46047: Include the path to the JAR when recording dependencies that are located inside a JAR file. Patch provided by - Cédric Mailleux. (markt) + C�dric Mailleux. (markt) 46381: Composite expressions used for attribute values must @@ -2232,7 +2236,7 @@ 46035: Fix multiple typos in monitoring how to. (markt) - 46067: Fix typos in Advanced IO how to. (markt) + 46067: Fix typos in Advanced IO how to. (markt) 46115: Correct Manager UI to show that path is required when @@ -2299,7 +2303,7 @@ 46366: Correct information in RUNNING.txt regarding use of - CATALINA_HOME and CATALINA_BASE. (markt) + CATALINA_HOME and CATALINA_BASE. (markt) Use more useful JPDA defaults in catalina.bat. (markt) @@ -2386,7 +2390,7 @@ Expose alwaysSend flag for message dispatch interceptor. (fhanik) - + 29936: Create digesters and parsers earlier so we aren't using the webapp class loader when we create them. (markt) @@ -2593,7 +2597,7 @@ 44391: Correct handling of escaped values in SSI processing. - (markt) + (markt) 44392: HTML entities now handled correctly in SSI processing. @@ -2614,7 +2618,7 @@ connector. (markt) - Log errors for AJP signoffs at DEBUG level, + Log errors for AJP signoffs at DEBUG level, since it is harmless if mod_jk has hung up the phone. (billbarker) @@ -2668,8 +2672,8 @@ 45015: You can't use an unescaped quote if you quote the - value with that character. (markt/fhanik) - + value with that character. (markt/fhanik) + Add HTML filtering of error messages for included resources in case the app has tried to include an unsafe URL that does not exist. This is @@ -2699,7 +2703,7 @@ 43366: Provide backwards compatibility for manager sessions command. (markt) - + 44541: Document packetSize attribute for AJP connector. (markt) @@ -3446,7 +3450,7 @@ Fix various paths in the manager webapps (remm) - Session viewer and editor for the HTML manager. Submitted by Cédrik Lime. (remm) + Session viewer and editor for the HTML manager. Submitted by C�drik Lime. (remm) Session handling tools for the manager. Submitted by Rainer Jung. (remm) @@ -3578,7 +3582,7 @@ Cleanup hello webapp from the docs and fix a XSS issue in the JSP. (remm) - Examples webapp cleanup. Submitted by Takayuki Kaneko and Markus Schönhaber. (remm) + Examples webapp cleanup. Submitted by Takayuki Kaneko and Markus Sch�nhaber. (remm) 41289: Create configBase, since it is no longer created elsewhere. @@ -3723,7 +3727,7 @@ Use 2.5 xsd in Tomcat webapps. (markt) - Compression filter improvements, submitted by Eric Hedström. (markt) + Compression filter improvements, submitted by Eric Hedstr�m. (markt) @@ -3873,7 +3877,7 @@ 39572: Improvements to CompressionFilter example provided by - Eric Hedström. (markt) + Eric Hedstr�m. (markt) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org For additional commands, e-mail: dev-help@tomcat.apache.org