Bug report for Taglibs [2010/06/06]

2010-06-06 Thread bugzilla
+---+
| Bugzilla Bug ID   |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned|
| | OPN=ReopenedVER=Verified(Skipped Closed/Resolved)   |
| |   +-+
| |   | Severity: BLK=Blocker CRI=Critical  REG=Regression  MAJ=Major   |
| |   |   MIN=Minor   NOR=NormalENH=Enhancement TRV=Trivial |
| |   |   +-+
| |   |   | Date Posted |
| |   |   |  +--+
| |   |   |  | Description  |
| |   |   |  |  |
|27717|New|Maj|2004-03-16| very slow in JSTL 1.1 |
|33934|New|Cri|2005-03-09|[standard] memory leak in jstl c:set tag  |
|38193|Ass|Enh|2006-01-09|[RDC] BuiltIn Grammar support for Field   |
|38600|Ass|Enh|2006-02-10|[RDC] Enable RDCs to be used in X+V markup (X+RDC)|
|42413|New|Nor|2007-05-14|[PATCH] Log Taglib enhancements   |
|43640|New|Nor|2007-10-16|Move the tests package to JUnit   |
|45197|Ass|Nor|2008-06-12|Need to support the JSTL 1.2 specification|
|46052|New|Nor|2008-10-21|SetLocaleSupport is slow to initialize when many l|
|48333|New|Nor|2009-12-02|TLD generator |
|48773|New|Nor|2010-02-19|DataSourceWrapper and DriverManager problems  |
|49292|New|Nor|2010-05-14|Memory leak in org.apache.taglibs.standard.lang.js|
+-+---+---+--+--+
| Total   11 bugs   |
+---+

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



Bug report for Tomcat 5 [2010/06/06]

2010-06-06 Thread bugzilla
+---+
| Bugzilla Bug ID   |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned|
| | OPN=ReopenedVER=Verified(Skipped Closed/Resolved)   |
| |   +-+
| |   | Severity: BLK=Blocker CRI=Critical  REG=Regression  MAJ=Major   |
| |   |   MIN=Minor   NOR=NormalENH=Enhancement TRV=Trivial |
| |   |   +-+
| |   |   | Date Posted |
| |   |   |  +--+
| |   |   |  | Description  |
| |   |   |  |  |
|27122|Opn|Enh|2004-02-20|IE plugins cannot access components through Tomcat|
|28039|Opn|Enh|2004-03-30|Cluster Support for SingleSignOn  |
|29494|Inf|Enh|2004-06-10|No way to set PATH when running as a service on Wi|
|33262|Inf|Enh|2005-01-27|Service Manager autostart should check for adminis|
|33453|Opn|Enh|2005-02-08|Jasper should recompile JSP files whose datestamps|
|33671|Opn|Enh|2005-02-21|Manual Windows service installation with custom na|
|34801|New|Enh|2005-05-08|PATCH: CGIServlet does not terminate child after a|
|34805|Ass|Enh|2005-05-08|warn about invalid security constraint url pattern|
|34868|Ass|Enh|2005-05-11|allow to register a trust store for a session that|
|35054|Inf|Enh|2005-05-25|warn if appBase is not existing as a File or direc|
|36133|Inf|Enh|2005-08-10|Support JSS SSL implementation|
|36362|New|Enh|2005-08-25|missing check for Java reserved keywords in tag fi|
|36569|Inf|Enh|2005-09-09|Redirects produce illegal URL's   |
|36837|Inf|Enh|2005-09-28|Looking for ProxyHandler implementation of Http re|
|36922|Inf|Enh|2005-10-04|setup.sh file mis-advertised and missing  |
|37018|Ass|Enh|2005-10-11|Document how to use tomcat-SSL with a pkcs11 token|
|37334|Inf|Enh|2005-11-02|Realm digest property not aligned with the adminis|
|37449|Opn|Enh|2005-11-10|Two UserDatabaseRealm break manager user  |
|37485|Inf|Enh|2005-11-14|I'd like to run init SQL after JDBC Connection cre|
|38216|Inf|Enh|2006-01-10|Extend Jmxproxy to allow call of MBean Operations |
|38268|Inf|Enh|2006-01-13|User friendly: Need submit button on adding/deleti|
|38360|Inf|Enh|2006-01-24|Domain for session cookies|
|38546|Inf|Enh|2006-02-07|Google bot sends invalid If-Modifed-Since Header, |
|38577|Inf|Enh|2006-02-08|Enhance logging of security failures  |
|38916|Inf|Enh|2006-03-10|HttpServletRequest cannot handle multipart request|
|39053|Inf|Enh|2006-03-21|include Tomcat embedded sample|
|39740|New|Enh|2006-06-07|semi-colon ; isn't allowed as a query argument sep|
|39862|Inf|Enh|2006-06-22|provide support for protocol-independent GenericSe|
|40211|Inf|Enh|2006-08-08|Compiled JSP don't indent HTML code   |
|40222|Inf|Enh|2006-08-09|Default Tomcat configuration alows easy session hi|
|40402|New|Enh|2006-09-03|Manager should display Exceptions |
|40510|New|Enh|2006-09-14|installer does not create shortcuts for all users |
|40712|New|Enh|2006-10-10|Realm admin error.|
|40728|Inf|Enh|2006-10-11|Catalina MBeans use non-serializable classes  |
|40766|New|Enh|2006-10-16|Using an unsecure jsessionid with mod_proxy_ajp ov|
|40881|Opn|Enh|2006-11-02|Unable to receive message through  TCP channel -> |
|41007|Opn|Enh|2006-11-20|Can't define customized 503 error page|
|41179|New|Enh|2006-12-15|400 Bad Request response during auto re-deployment|
|41227|Opn|Enh|2006-12-21|When the jasper compiler fails to compile a JSP, i|
|41337|Opn|Enh|2007-01-10|Display an error page if no cert is available on C|
|41496|New|Enh|2007-01-30|set a security provider for jsse in a connector co|
|41498|New|Enh|2007-01-30|allRolesMode Realm configuration option not docume|
|41539|Inf|Enh|2007-02-05|NullPointerException during Embedded tomcat restar|
|41673|New|Enh|2007-02-21|Jasper output the message of compiling error using|
|41697|Ver|Enh|2007-02-25|make visible in debug output if charset from brows|
|41709|Inf|Enh|2007-02-26|When calling the API that relates to the buffer af|
|41718|New|Enh|2007-02-27|Status 302 response to GET request has no body whe|
|42416|New|Enh|2007-05-14|Tomcat startup hangs and AJP13 connector port 8009|
|43423|New|Enh|2007-09-18|catalina.sh -force too fast   |
|43538|New|Enh|2007-10-02|[patch] Show the hostname and IP address in the ma|
|43796|Inf|Enh|2007-11-05|Add MIME type mapping for the "log" extension |
|43866|

Bug report for Tomcat 6 [2010/06/06]

2010-06-06 Thread bugzilla
+---+
| Bugzilla Bug ID   |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned|
| | OPN=ReopenedVER=Verified(Skipped Closed/Resolved)   |
| |   +-+
| |   | Severity: BLK=Blocker CRI=Critical  REG=Regression  MAJ=Major   |
| |   |   MIN=Minor   NOR=NormalENH=Enhancement TRV=Trivial |
| |   |   +-+
| |   |   | Date Posted |
| |   |   |  +--+
| |   |   |  | Description  |
| |   |   |  |  |
|39661|Opn|Enh|2006-05-25|Please document JULI FileHandler configuration pro|
|41128|Inf|Enh|2006-12-07|Reference to java Thread name from RequestProcesso|
|41679|New|Enh|2007-02-22|SemaphoreValve should be able to filter on url pat|
|41791|New|Enh|2007-03-07|Tomcat behaves inconsistently concerning flush-pac|
|41883|Ass|Enh|2007-03-18|use abstract wrapper instead of plain X509Certific|
|41944|New|Enh|2007-03-25|Start running the RAT tool to see where we're miss|
|41992|New|Enh|2007-03-30|Need ability to set OS process title  |
|42463|New|Enh|2007-05-20|"crossContext" and classloader issues - pls amend |
|43001|New|Enh|2007-07-30|JspC lacks setMappedFile and setDie for use in Ant|
|43003|New|Enh|2007-07-30|Separate dependent component download and build ta|
|43400|New|Enh|2007-09-14|enum support for tag libs |
|43497|New|Enh|2007-09-26|Add ability to escape rendered output of JSP expre|
|43548|Opn|Enh|2007-10-04|xml schema for tomcat-users.xml   |
|43642|New|Enh|2007-10-17|Add prestartminSpareThreads attribute for Executor|
|43682|New|Enh|2007-10-23|JULI: web-inf/classes/logging.properties to suppor|
|43742|New|Enh|2007-10-30|.tag compiles  performed one at a time -- extremel|
|43790|Ass|Enh|2007-11-03|concurrent access issue on TagHandlerPool |
|43979|New|Enh|2007-11-27|Add abstraction for Java and Classfile output |
|44047|New|Enh|2007-12-10|Provide a way for Tomcat to serve up error pages w|
|44106|New|Enh|2007-12-19|autodeploy configures directories which do not con|
|44199|New|Enh|2008-01-10|expose current backlog queue size |
|44225|New|Enh|2008-01-14|SSL connector tries to load the private keystore f|
|44264|New|Enh|2008-01-18|Clustering - Support for disabling multicasting an|
|44284|New|Enh|2008-01-23|Support java.lang.Iterable in c:forEach tag   |
|44294|New|Enh|2008-01-25|Support for EL functions with varargs |
|44299|New|Enh|2008-01-26|Provider manager app with a log out button|
|44312|New|Enh|2008-01-28|Warn when overwritting docBase of the default Host|
|44598|New|Enh|2008-03-13|JAASRealm is suppressing Exceptions   |
|44645|New|Enh|2008-03-20|[Patch] JNDIRealm - Doesn't support JNDI "java.nam|
|44787|New|Enh|2008-04-09|provide more error context on "java.lang.IllegalSt|
|44818|New|Enh|2008-04-13|tomcat hangs with GET when content-length is defin|
|45014|New|Enh|2008-05-15|Request and Response classes should have wrappers |
|45282|New|Enh|2008-06-25|NioReceiver doesn't close cleanly, leaving sockets|
|45283|Opn|Enh|2008-06-25|Allow multiple authenticators to be added to pipel|
|45428|New|Enh|2008-07-18|warn if the tomcat stop doesn't complete  |
|45654|New|Enh|2008-08-19|use static methods and attributes in a direct way!|
|45731|New|Enh|2008-09-02|Enhancement request : pluggable httpsession cache |
|45832|New|Enh|2008-09-18|add DIGEST authentication support to Ant tasks|
|45871|New|Enh|2008-09-23|Support for salted and digested patches in DataSou|
|45878|New|Enh|2008-09-24|Generated jars do not contain proper manifests or |
|45879|Opn|Enh|2008-09-24|Windows installer fails to install NOTICE and RELE|
|45931|Opn|Enh|2008-10-01|trimSpaces incorrectly modifies output|
|45995|New|Enh|2008-10-13|RFE - MIME type extension not case sensitive  |
|46173|New|Enh|2008-11-09|Small patch for manager app: Setting an optional c|
|46263|New|Enh|2008-11-21|Tomcat reloading of context does not update contex|
|46264|New|Enh|2008-11-21|Shutting down tomcat with large number of contexts|
|46284|New|Enh|2008-11-24|Add flag to DeltaManager that blocks processing cl|
|46350|New|Enh|2008-12-05|Maven repository should contain source bundles|
|46451|New|Enh|2008-12-30|Configure svn:bugtraq properties  |
|46461|New|Enh|2009-01-01|fail graceful on dns changes for connectors/hosts |
|46497|New|Enh|2009-01-08|Install Tomcat Deployer/ANT on Windows Platform   |
|46558|

Bug report for Tomcat 7 [2010/06/06]

2010-06-06 Thread bugzilla
+---+
| Bugzilla Bug ID   |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned|
| | OPN=ReopenedVER=Verified(Skipped Closed/Resolved)   |
| |   +-+
| |   | Severity: BLK=Blocker CRI=Critical  REG=Regression  MAJ=Major   |
| |   |   MIN=Minor   NOR=NormalENH=Enhancement TRV=Trivial |
| |   |   +-+
| |   |   | Date Posted |
| |   |   |  +--+
| |   |   |  | Description  |
| |   |   |  |  |
|48240|New|Nor|2009-11-19|Tomcat-Lite missing @Override markers |
|48268|New|Nor|2009-11-23|Patch to fix generics in tomcat-lite  |
|48297|New|Nor|2009-11-28|webservices.ServiceRefFactory.initHandlerChain add|
|48550|New|Enh|2010-01-14|Update examples and default server.xml to use UTF-|
|48648|New|Nor|2010-01-31|Blank page (dropped connection) when running TC7 w|
|48692|New|Enh|2010-02-07|Provide option to parse application/x-www-form-url|
|48817|New|Nor|2010-02-25|Skip validation query and use JDBC API for validat|
|48837|New|Enh|2010-03-01|Memory leaks protection does not cure leaks trigge|
|48861|New|Nor|2010-03-04|Files without AL headers  |
|48870|New|Enh|2010-03-08|avoid parallel arrays of base types   |
|48891|New|Enh|2010-03-11|Missing EOL-style settings in tomcat/jk/trunk |
|48892|New|Enh|2010-03-11|Use URIEncoding from server.xml for decoding post |
|48998|New|Enh|2010-03-26|Proposal : port mod_expires in java as ExpiresFilt|
|49000|New|Enh|2010-03-26|Cookie parsing bug when an empty value has an equa|
|49100|New|Enh|2010-04-12|Mutable public constants: MemberImpl.TRIBES_MBR_BE|
|49101|New|Enh|2010-04-12|All constants in tribes.transport.Constants are mu|
|49102|New|Enh|2010-04-12|coyote.ajp.Constants - 3 "constant" arrays are mut|
|49122|New|Enh|2010-04-14|Update of ROOT application index.html |
|49125|New|Enh|2010-04-14|toString on byte[] array probably is not correct  |
|49126|New|Enh|2010-04-14|jasper.compiler.Generator.createJspId() can genera|
|49127|New|Enh|2010-04-14|SimpleTcpReplicationManager.startInternal() ignore|
|49128|New|Enh|2010-04-14|loader.WebappClassLoader.start() ignores Exception|
|49130|New|Enh|2010-04-14|NSIS - clarify that service is always installed   |
|49142|New|Enh|2010-04-16|Missing serialVersionUIDs |
|49159|New|Enh|2010-04-20|Improve ThreadLocal memory leak clean-up  |
|49165|New|Enh|2010-04-21|Enhancement - Allow %{TIME_FORMAT}t As Configurati|
|49180|New|Enh|2010-04-24|Add option to disable log rotation in FileHandler.|
|49182|New|Enh|2010-04-24|Documentation patch for setclasspath.sh   |
|49234|New|Nor|2010-04-30|JMX Descriptor Modifications  |
|49268|New|Enh|2010-05-10|Use checkstyle to enforce common code style   |
|49284|New|Enh|2010-05-12|Implement SSL renegotiation for the NIO connector |
|49290|New|Enh|2010-05-14|Using a JarScanner with scanAllDirectories=true ca|
|49295|New|Enh|2010-05-14|JMXAccessorTask.getProperties() - return Propertie|
|49297|New|Min|2010-05-15|Whitespace absence is allowed before attribute nam|
|49317|New|Enh|2010-05-20|org.apache.catalina.util.Base64 interface needs re|
|49318|New|Enh|2010-05-20|add a Negotiate (Kerberos/NTLM) authenticator / in|
|49372|New|Min|2010-06-02|7.0.0RC: Starting second instance of Tomcat on the|
|49376|New|Trv|2010-06-03|Minor corrections to info properties. |
+-+---+---+--+--+
| Total   38 bugs   |
+---+

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



Bug report for Tomcat Connectors [2010/06/06]

2010-06-06 Thread bugzilla
+---+
| Bugzilla Bug ID   |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned|
| | OPN=ReopenedVER=Verified(Skipped Closed/Resolved)   |
| |   +-+
| |   | Severity: BLK=Blocker CRI=Critical  REG=Regression  MAJ=Major   |
| |   |   MIN=Minor   NOR=NormalENH=Enhancement TRV=Trivial |
| |   |   +-+
| |   |   | Date Posted |
| |   |   |  +--+
| |   |   |  | Description  |
| |   |   |  |  |
|34526|Opn|Nor|2005-04-19|Truncated content in decompressed requests from mo|
|35959|Opn|Enh|2005-08-01|mod_jk not independant of UseCanonicalName|
|36155|Opn|Nor|2005-08-12|tomcat chooses wrong host if using mod_jk |
|39967|Inf|Nor|2006-07-05|mod_jk gives segmentation fault when apache is sta|
|40208|Inf|Nor|2006-08-08|Request-Dump when ErrorDocument in httpd.conf is a|
|41170|Inf|Nor|2006-12-13|single crlf in header termination crashes app.|
|41923|Opn|Nor|2007-03-21|Tomcat doesnt recognized client abort |
|42366|Inf|Nor|2007-05-09|Memory leak in newer mod_jk version when connectio|
|42554|Opn|Nor|2007-05-31|mod_ssl + mod_jk with status_worker does not work |
|43303|New|Enh|2007-09-04|Versioning under Windows not reported by many conn|
|43968|New|Enh|2007-11-26|[patch] support ipv6 with mod_jk  |
|44290|New|Nor|2008-01-24|mod_jk/1.2.26: retry is not useful for an importan|
|44349|New|Maj|2008-02-04|mod_jk/1.2.26 module does not read worker.status.s|
|44379|New|Enh|2008-02-07|convert the output of strftime into UTF-8 |
|44454|New|Nor|2008-02-19|busy count reported in mod_jk inflated, causes inc|
|44571|New|Enh|2008-03-10|Limits busy per worker to a threshold |
|45063|New|Nor|2008-05-22|JK-1.2.26 IIS ISAPI filter issue when running diff|
|45313|New|Nor|2008-06-30|mod_jk 1.2.26 & apache 2.2.9 static compiled on so|
|45395|New|Min|2008-07-14|MsgAjp dump method does not dump packet when being|
|46337|New|Nor|2008-12-04|real worker name is wrong |
|46406|New|Enh|2008-12-16|Supporting relative paths in isapi_redirect.proper|
|46676|New|Enh|2009-02-09|Configurable test request for Watchdog thread |
|46767|New|Enh|2009-02-25|mod_jk to send DECLINED in case no fail-over tomca|
|47038|New|Enh|2009-04-15|USE_FLOCK_LK redefined compiler warning when using|
|47327|New|Enh|2009-06-07|remote_user not logged in apache logfile  |
|47617|New|Enh|2009-07-31|include time spent doing ajp_get_endpoint() in err|
|47678|New|Nor|2009-08-11|Unable to allocate shared memory when using isapi_|
|47679|New|Nor|2009-08-11|Not all headers get passed to Tomcat server from i|
|47692|New|Reg|2009-08-12|Can not compile mod_jk with apache2.0.63 and tomca|
|47714|New|Cri|2009-08-20|Reponse mixed between users   |
|47750|New|Maj|2009-08-27|Loss of worker settings when changing via jkstatus|
|47795|New|Maj|2009-09-07|service sticky_session not being set correctly wit|
|47840|Inf|Min|2009-09-14|A broken worker name is written in the log file.  |
|48191|New|Maj|2009-11-13|Problem with mod_jk 1.2.28 - Can not render up the|
|48490|New|Nor|2010-01-05|Changing a node to stopped in uriworkermap.propert|
|48501|New|Enh|2010-01-07|Log rotation for ISAPI Redirector |
|48513|New|Enh|2010-01-09|IIS Quick setup instructions  |
|48564|New|Nor|2010-01-18|Unable to turn off retries for LB worker  |
|48830|New|Nor|2010-03-01|IIS shutdown blocked in endpoint service when serv|
|48925|New|Maj|2010-03-16|((ServletRequest) request).getLocalAddr() returns |
|48940|New|Maj|2010-03-18|IIS to Tomcat occasionally fails on POST with T-E |
|49035|New|Maj|2010-04-01|data lost when post a multipart/form-data form|
|49048|New|Nor|2010-04-05|ACL not applied to redirect URLs  |
|49063|New|Enh|2010-04-07|Please add JkStripSession status in jk-status work|
|49135|New|Enh|2010-04-16|SPDY Connector for The Tomcat |
+-+---+---+--+--+
| Total   45 bugs   |
+---+

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



Bug report for Tomcat Native [2010/06/06]

2010-06-06 Thread bugzilla
+---+
| Bugzilla Bug ID   |
| +-+
| | Status: UNC=Unconfirmed NEW=New ASS=Assigned|
| | OPN=ReopenedVER=Verified(Skipped Closed/Resolved)   |
| |   +-+
| |   | Severity: BLK=Blocker CRI=Critical  REG=Regression  MAJ=Major   |
| |   |   MIN=Minor   NOR=NormalENH=Enhancement TRV=Trivial |
| |   |   +-+
| |   |   | Date Posted |
| |   |   |  +--+
| |   |   |  | Description  |
| |   |   |  |  |
|38372|Inf|Cri|2006-01-25|tcnative-1.dll response overflow corruption, parti|
|41361|New|Nor|2007-01-14|Content lost when read by a slow client.  |
|42090|New|Cri|2007-04-11|tcnative badly handles some OpenSSL disconnections|
|45392|New|Nor|2008-07-14|No OCSP support for client SSL verification   |
|46041|New|Cri|2008-10-20|Tomcat service is terminated unexpectedly (tcnativ|
|46179|New|Maj|2008-11-10|apr ssl client authentication |
|46571|New|Nor|2009-01-21|tcnative blocks in APR poll on Solaris|
|47319|New|Nor|2009-06-05|With APR, getRemoteHost() returns NULL for unknown|
|47851|New|Nor|2009-09-16|thread-safety issues in the TC native Java code   |
|48253|New|Min|2009-11-20|Tomcat Native patch - adding dynamic locking callb|
|48655|New|Nor|2010-02-02|Active multipart downloads prevent tomcat shutdown|
|49038|Inf|Nor|2010-04-02|Crash in tcnative |
+-+---+---+--+--+
| Total   12 bugs   |
+---+

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



svn commit: r951874 - /tomcat/tc6.0.x/trunk/conf/catalina.policy

2010-06-06 Thread kkolinko
Author: kkolinko
Date: Sun Jun  6 15:25:51 2010
New Revision: 951874

URL: http://svn.apache.org/viewvc?rev=951874&view=rev
Log:
Update a comment

Modified:
tomcat/tc6.0.x/trunk/conf/catalina.policy

Modified: tomcat/tc6.0.x/trunk/conf/catalina.policy
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/conf/catalina.policy?rev=951874&r1=951873&r2=951874&view=diff
==
--- tomcat/tc6.0.x/trunk/conf/catalina.policy (original)
+++ tomcat/tc6.0.x/trunk/conf/catalina.policy Sun Jun  6 15:25:51 2010
@@ -14,14 +14,15 @@
 // limitations under the License.
 
 // 
-// catalina.corepolicy - Security Policy Permissions for Tomcat @VERSION_MAJOR@
+// catalina.policy - Security Policy Permissions for Tomcat @VERSION_MAJOR@
 //
 // This file contains a default set of security policies to be enforced (by the
 // JVM) when Catalina is executed with the "-security" option.  In addition
 // to the permissions granted here, the following additional permissions are
 // granted to the codebase specific to each web application:
 //
-// * Read access to the document root directory
+// * Read access to its document root directory
+// * Read, write and delete access to its working directory
 //
 // $Id$
 // 



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



Re: Release plans for 6.0.27 - ?

2010-06-06 Thread jean-frederic clere
On 06/04/2010 09:57 PM, Konstantin Kolinko wrote:
> Hi!
> 
> When can we make 6.0.27?
> 
> There were a couple of important fixes since 6.0.26, like BZ 48903
> (classloader deadlock).
> 
> I think it is about time to make 6.0.27.

+1 I will start to prepare the stuff. Tag on Monday, packaging , votes
and release before the next week-end, is that ok?

Cheers

Jean-Frederic

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



svn commit: r951880 - in /tomcat/trunk: conf/catalina.policy webapps/docs/security-manager-howto.xml

2010-06-06 Thread kkolinko
Author: kkolinko
Date: Sun Jun  6 16:02:30 2010
New Revision: 951880

URL: http://svn.apache.org/viewvc?rev=951880&view=rev
Log:
Rearrange tomcat-juli permissions, for better readability.

Modified:
tomcat/trunk/conf/catalina.policy
tomcat/trunk/webapps/docs/security-manager-howto.xml

Modified: tomcat/trunk/conf/catalina.policy
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/conf/catalina.policy?rev=951880&r1=951879&r2=951880&view=diff
==
--- tomcat/trunk/conf/catalina.policy (original)
+++ tomcat/trunk/conf/catalina.policy Sun Jun  6 16:02:30 2010
@@ -64,30 +64,35 @@ grant codeBase "file:${catalina.home}/bi
 // These permissions apply to the logging API
 // Note: If tomcat-juli.jar is in ${catalina.base} and not in ${catalina.home},
 // update this section accordingly.
+//  grant codeBase "file:${catalina.base}/bin/tomcat-juli.jar" {..}
 grant codeBase "file:${catalina.home}/bin/tomcat-juli.jar" {
-permission java.util.PropertyPermission 
"java.util.logging.config.class", "read";
-permission java.util.PropertyPermission 
"java.util.logging.config.file", "read";
-permission java.util.PropertyPermission "catalina.base", "read";
 permission java.io.FilePermission
  
"${java.home}${file.separator}lib${file.separator}logging.properties", "read"; 
+
 permission java.io.FilePermission
  
"${catalina.base}${file.separator}conf${file.separator}logging.properties", 
"read";
 permission java.io.FilePermission
  "${catalina.base}${file.separator}logs", "read, write";
 permission java.io.FilePermission
  "${catalina.base}${file.separator}logs${file.separator}*", "read, 
write";
+
 permission java.lang.RuntimePermission "shutdownHooks";
 permission java.lang.RuntimePermission "getClassLoader";
 permission java.lang.RuntimePermission "setContextClassLoader";
+
 permission java.util.logging.LoggingPermission "control";
 
-// To enable per context logging configuration, permit read access to
+permission java.util.PropertyPermission 
"java.util.logging.config.class", "read";
+permission java.util.PropertyPermission 
"java.util.logging.config.file", "read";
+permission java.util.PropertyPermission "catalina.base", "read";
+
+// Note: To enable per context logging configuration, permit read 
access to
 // the appropriate file. Be sure that the logging configuration is
-// secure before enabling such access. E.g. for the examples web
-// application:
+// secure before enabling such access.
+// E.g. for the examples web application, all in one line:
 // permission java.io.FilePermission "${catalina.base}${file.separator}
-//  webapps${file.separator}examples${file.separator}
-//  
WEB-INF${file.separator}classes${file.separator}logging.properties", "read";
+//  webapps${file.separator}examples${file.separator}WEB-INF
+//  ${file.separator}classes${file.separator}logging.properties", 
"read";
 };
 
 // These permissions apply to the server startup code

Modified: tomcat/trunk/webapps/docs/security-manager-howto.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/security-manager-howto.xml?rev=951880&r1=951879&r2=951880&view=diff
==
--- tomcat/trunk/webapps/docs/security-manager-howto.xml (original)
+++ tomcat/trunk/webapps/docs/security-manager-howto.xml Sun Jun  6 16:02:30 
2010
@@ -230,30 +230,35 @@ grant codeBase "file:${catalina.home}/bi
 // These permissions apply to the logging API
 // Note: If tomcat-juli.jar is in ${catalina.base} and not in ${catalina.home},
 // update this section accordingly.
+//  grant codeBase "file:${catalina.base}/bin/tomcat-juli.jar" {..}
 grant codeBase "file:${catalina.home}/bin/tomcat-juli.jar" {
-permission java.util.PropertyPermission 
"java.util.logging.config.class", "read";
-permission java.util.PropertyPermission 
"java.util.logging.config.file", "read";
-permission java.util.PropertyPermission "catalina.base", "read";
 permission java.io.FilePermission
  
"${java.home}${file.separator}lib${file.separator}logging.properties", "read"; 
+
 permission java.io.FilePermission
  
"${catalina.base}${file.separator}conf${file.separator}logging.properties", 
"read";
 permission java.io.FilePermission
  "${catalina.base}${file.separator}logs", "read, write";
 permission java.io.FilePermission
  "${catalina.base}${file.separator}logs${file.separator}*", "read, 
write";
+
 permission java.lang.RuntimePermission "shutdownHooks";
 permission java.lang.RuntimePermission "getClassLoader";
 permission java.lang.RuntimePermission "setContextClassLoader";
+
 permission java.util.lo

svn commit: r951883 - /tomcat/tc6.0.x/trunk/conf/catalina.policy

2010-06-06 Thread kkolinko
Author: kkolinko
Date: Sun Jun  6 16:10:55 2010
New Revision: 951883

URL: http://svn.apache.org/viewvc?rev=951883&view=rev
Log:
Add a comment regarding placing tomcat-juli.jar in ${catalina.base}/bin

Modified:
tomcat/tc6.0.x/trunk/conf/catalina.policy

Modified: tomcat/tc6.0.x/trunk/conf/catalina.policy
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/conf/catalina.policy?rev=951883&r1=951882&r2=951883&view=diff
==
--- tomcat/tc6.0.x/trunk/conf/catalina.policy (original)
+++ tomcat/tc6.0.x/trunk/conf/catalina.policy Sun Jun  6 16:10:55 2010
@@ -62,6 +62,9 @@ grant codeBase "file:${catalina.home}/bi
 };
 
 // These permissions apply to the logging API
+// Note: If tomcat-juli.jar is in ${catalina.base} and not in ${catalina.home},
+// update this section accordingly.
+//  grant codeBase "file:${catalina.base}/bin/tomcat-juli.jar" {..}
 grant codeBase "file:${catalina.home}/bin/tomcat-juli.jar" {
 permission java.util.PropertyPermission 
"java.util.logging.config.class", "read";
 permission java.util.PropertyPermission 
"java.util.logging.config.file", "read";
@@ -74,9 +77,11 @@ grant codeBase "file:${catalina.home}/bi
 permission java.io.FilePermission 
"${catalina.base}${file.separator}logs${file.separator}*", "read, write";
 permission java.lang.RuntimePermission "getClassLoader";
 permission java.lang.RuntimePermission "setContextClassLoader";
-// To enable per context logging configuration, permit read access to 
the appropriate file.
-// Be sure that the logging configuration is secure before enabling 
such access
-// eg for the examples web application:
+
+// Note: To enable per context logging configuration, permit read 
access to
+// the appropriate file. Be sure that the logging configuration is
+// secure before enabling such access.
+// E.g. for the examples web application, all in one line:
 // permission java.io.FilePermission 
"${catalina.base}${file.separator}webapps${file.separator}examples${file.separator}WEB-INF${file.separator}classes${file.separator}logging.properties",
 "read";
 };
 



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



svn commit: r951887 - /tomcat/tc6.0.x/trunk/webapps/docs/security-manager-howto.xml

2010-06-06 Thread kkolinko
Author: kkolinko
Date: Sun Jun  6 16:20:50 2010
New Revision: 951887

URL: http://svn.apache.org/viewvc?rev=951887&view=rev
Log:
Document default working directory permissions
Update in-document copy of the default catalina.policy file

Modified:
tomcat/tc6.0.x/trunk/webapps/docs/security-manager-howto.xml

Modified: tomcat/tc6.0.x/trunk/webapps/docs/security-manager-howto.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/webapps/docs/security-manager-howto.xml?rev=951887&r1=951886&r2=951887&view=diff
==
--- tomcat/tc6.0.x/trunk/webapps/docs/security-manager-howto.xml (original)
+++ tomcat/tc6.0.x/trunk/webapps/docs/security-manager-howto.xml Sun Jun  6 
16:20:50 2010
@@ -124,12 +124,19 @@ permission  org.apache.naming.JndiPermis
 but disallow it from using file access to read any other files (unless
 permissions for those files are explicitly granted).
 
-Also, Tomcat always dynamically creates the following file 
permission:
-  
+Also, Tomcat always dynamically creates the following file 
permissions:
+
 permission java.io.FilePermission "** your application context**", "read";
-  
-Where **your application context** equals the folder(or WAR file) under 
which 
-your application has been deployed.   
+
+permission java.io.FilePermission
+  "** application working directory**", "read,write";
+permission java.io.FilePermission
+  "** application working directory**/-", "read,write,delete";
+
+Where **your application context** equals the folder (or WAR file) 
under which 
+your application has been deployed and **application working directory** 
is the
+temporary directory provided to your application as required by the
+Servlet Specification.
 
   
 
@@ -174,14 +181,15 @@ grant [signedBy ,] [codeBa
   looks like this:
 
 // 
-// catalina.corepolicy - Security Policy Permissions for Tomcat 6
+// catalina.policy - Security Policy Permissions for Tomcat 6
 //
 // This file contains a default set of security policies to be enforced (by the
 // JVM) when Catalina is executed with the "-security" option.  In addition
 // to the permissions granted here, the following additional permissions are
 // granted to the codebase specific to each web application:
 //
-// * Read access to the document root directory
+// * Read access to its document root directory
+// * Read, write and delete access to its working directory
 //
 // 
 
@@ -220,8 +228,37 @@ grant codeBase "file:${catalina.home}/bi
 };
 
 // These permissions apply to the logging API
+// Note: If tomcat-juli.jar is in ${catalina.base} and not in ${catalina.home},
+// update this section accordingly.
+//  grant codeBase "file:${catalina.base}/bin/tomcat-juli.jar" {..}
 grant codeBase "file:${catalina.home}/bin/tomcat-juli.jar" {
-permission java.security.AllPermission;
+permission java.io.FilePermission
+ 
"${java.home}${file.separator}lib${file.separator}logging.properties", "read"; 
+
+permission java.io.FilePermission
+ 
"${catalina.base}${file.separator}conf${file.separator}logging.properties", 
"read";
+permission java.io.FilePermission
+ "${catalina.base}${file.separator}logs", "read, write";
+permission java.io.FilePermission
+ "${catalina.base}${file.separator}logs${file.separator}*", "read, 
write";
+
+permission java.lang.RuntimePermission "shutdownHooks";
+permission java.lang.RuntimePermission "getClassLoader";
+permission java.lang.RuntimePermission "setContextClassLoader";
+
+permission java.util.logging.LoggingPermission "control";
+
+permission java.util.PropertyPermission 
"java.util.logging.config.class", "read";
+permission java.util.PropertyPermission 
"java.util.logging.config.file", "read";
+permission java.util.PropertyPermission "catalina.base", "read";
+
+// Note: To enable per context logging configuration, permit read 
access to
+// the appropriate file. Be sure that the logging configuration is
+// secure before enabling such access.
+// E.g. for the examples web application, all in one line:
+// permission java.io.FilePermission "${catalina.base}${file.separator}
+//  webapps${file.separator}examples${file.separator}WEB-INF
+//  ${file.separator}classes${file.separator}logging.properties", 
"read";
 };
 
 // These permissions apply to the server startup code
@@ -237,6 +274,13 @@ grant codeBase "file:${catalina.home}/li
 };
 
 
+// If using a per instance lib directory, i.e. ${catalina.base}/lib,
+// then the following permission will need to be uncommented
+// grant codeBase "file:${catalina.base}/lib/-" {
+// permission java.security.AllPermission;
+// };

svn commit: r951891 - /tomcat/tc6.0.x/trunk/STATUS.txt

2010-06-06 Thread kkolinko
Author: kkolinko
Date: Sun Jun  6 16:39:59 2010
New Revision: 951891

URL: http://svn.apache.org/viewvc?rev=951891&view=rev
Log:
proposal

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=951891&r1=951890&r2=951891&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Sun Jun  6 16:39:59 2010
@@ -223,3 +223,12 @@ PATCHES PROPOSED TO BACKPORT:
   (Easier to review if you ignore whitespaces during the diff)
   +1: kkolinko, kfujino
   -1:
+
+* Rearrange tomcat-juli permissions in catalina.policy for better readability,
+  and wrap long lines, as already done in trunk and in security-manager-howto.
+  The reason is that we have a copy of this file in 
security-manager-howto.html,
+  and long lines break layout of the document.
+  I am not very confident that this patch needs applying, but here it is.
+  http://people.apache.org/~kkolinko/patches/2010-06-06_tc6_policy-juli.patch
+  +1: kkolinko
+  -1:



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



svn commit: r951892 - /tomcat/trunk/java/org/apache/catalina/valves/JDBCAccessLogValve.java

2010-06-06 Thread markt
Author: markt
Date: Sun Jun  6 16:42:56 2010
New Revision: 951892

URL: http://svn.apache.org/viewvc?rev=951892&view=rev
Log:
Add AccessLog interface to JDBCAccessLog

Modified:
tomcat/trunk/java/org/apache/catalina/valves/JDBCAccessLogValve.java

Modified: tomcat/trunk/java/org/apache/catalina/valves/JDBCAccessLogValve.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/valves/JDBCAccessLogValve.java?rev=951892&r1=951891&r2=951892&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/valves/JDBCAccessLogValve.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/valves/JDBCAccessLogValve.java Sun 
Jun  6 16:42:56 2010
@@ -29,6 +29,7 @@ import java.util.Properties;
 
 import javax.servlet.ServletException;
 
+import org.apache.catalina.AccessLog;
 import org.apache.catalina.LifecycleException;
 import org.apache.catalina.LifecycleState;
 import org.apache.catalina.connector.Request;
@@ -114,7 +115,7 @@ import org.apache.tomcat.util.res.String
  * @author Peter Rossbach
  */
 
-public final class JDBCAccessLogValve extends ValveBase {
+public final class JDBCAccessLogValve extends ValveBase implements AccessLog {
 
 // --- Constructors
 
@@ -445,10 +446,17 @@ public final class JDBCAccessLogValve ex
 @Override
 public void invoke(Request request, Response response) 
 throws IOException, ServletException {
-final String EMPTY = "" ;
 
 getNext().invoke(request, response);
 
+log (request, response, 0);
+}
+
+
+@Override
+public void log(Request request, Response response, long time) {
+final String EMPTY = "" ;
+
 String remoteHost;
 if(resolveHosts)
 remoteHost = request.getRemoteHost();



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



svn commit: r951898 - /tomcat/trunk/webapps/docs/config/manager.xml

2010-06-06 Thread markt
Author: markt
Date: Sun Jun  6 17:04:42 2010
New Revision: 951898

URL: http://svn.apache.org/viewvc?rev=951898&view=rev
Log:
Remove the warning. The bug reports received indicate that folks are using 
this, generally successfully.

Modified:
tomcat/trunk/webapps/docs/config/manager.xml

Modified: tomcat/trunk/webapps/docs/config/manager.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/config/manager.xml?rev=951898&r1=951897&r2=951898&view=diff
==
--- tomcat/trunk/webapps/docs/config/manager.xml (original)
+++ tomcat/trunk/webapps/docs/config/manager.xml Sun Jun  6 17:04:42 2010
@@ -164,10 +164,6 @@
 
 Persistent Manager Implementation
 
-WARNING - Use of this Manager implementation
-has not been thoroughly tested, and should be considered experimental!
-
-
 NOTE: You must set either the
 org.apache.catalina.session.StandardSession.ACTIVITY_CHECK or
 org.apache.catalina.STRICT_SERVLET_COMPLIANCE



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



svn commit: r951908 - /tomcat/tc6.0.x/trunk/STATUS.txt

2010-06-06 Thread rjung
Author: rjung
Date: Sun Jun  6 17:30:27 2010
New Revision: 951908

URL: http://svn.apache.org/viewvc?rev=951908&view=rev
Log:
Vote, propose, comment

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=951908&r1=951907&r2=951908&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Sun Jun  6 17:30:27 2010
@@ -83,6 +83,8 @@ PATCHES PROPOSED TO BACKPORT:
 
   Expose properties of VirtualWebappLoader and WebappClassLoader via JMX.
   http://svn.apache.org/viewvc?view=revision&revision=936819
+  Added per Konstantin's review comment:
+  http://svn.apache.org/viewvc?view=revision&revision=947617
   +1: rjung, markt, kkolinko
   -1: 
kkolinko:
@@ -92,6 +94,7 @@ PATCHES PROPOSED TO BACKPORT:
 I tested and either one of these two capitalization variants works 
correctly.
 Though you need to configure a VirtualWebappLoader to test that:
 without it the property is always a zero-length array.
+   Mark: Does your +1 stand including the second part of the path?
 
   Add property "searchExternalFirst" to WebappLoader:
   If set, the external repositories will be searched before
@@ -139,24 +142,24 @@ PATCHES PROPOSED TO BACKPORT:
 
 * Fix ByteChunk.indexOf(String, ...) when the string is of length 1
   http://svn.apache.org/viewvc?rev=945231&view=rev
-  +1: kkolinko, markt
+  +1: kkolinko, markt, rjung
   -1:
-  
+
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48990
   Fix the skip.installer build property so once set, the Windows installer is
   skipped but all the other artifacts are created
   https://issues.apache.org/bugzilla/attachment.cgi?id=25477&action=edit
-  +1: markt, kkolinko
+  +1: markt, kkolinko, rjung
   -1: 
 
 * Fix issues running AJP BIO coyote connector under a security manager
   http://svn.apache.org/viewvc?rev=947717&view=rev (less changes to 
loadUtilPackage())
-  +1: markt, kkolinko
+  +1: markt, kkolinko, rjung
   -1: 
 
 * Additional clean-up possible as a result of above fix
   http://svn.apache.org/viewvc?rev=950905&view=rev (associated clean-up)
-  +1: markt, kkolinko
+  +1: markt, kkolinko, rjung
   -1: 
   
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49095
@@ -168,7 +171,7 @@ PATCHES PROPOSED TO BACKPORT:
   +0: kkolinko: I updated the above patch to use explicit encoding when
   creating the writer:
   https://issues.apache.org/bugzilla/attachment.cgi?id=25510
-  +1: kkolinko
+  +1: kkolinko, rjung
   -1:
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49196
@@ -230,5 +233,7 @@ PATCHES PROPOSED TO BACKPORT:
   and long lines break layout of the document.
   I am not very confident that this patch needs applying, but here it is.
   http://people.apache.org/~kkolinko/patches/2010-06-06_tc6_policy-juli.patch
-  +1: kkolinko
+  +1: kkolinko, rjung
   -1:
+  rjung: Please remove the "all in one line" comment directly before the 
commented
+ example line that you were breaking up into multiple lines.



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



Re: Release plans for 6.0.27 - ?

2010-06-06 Thread Rainer Jung

On 06.06.2010 17:41, jean-frederic clere wrote:

On 06/04/2010 09:57 PM, Konstantin Kolinko wrote:

Hi!

When can we make 6.0.27?

There were a couple of important fixes since 6.0.26, like BZ 48903
(classloader deadlock).

I think it is about time to make 6.0.27.


+1 I will start to prepare the stuff. Tag on Monday, packaging , votes
and release before the next week-end, is that ok?


There are about 10 proposals in STATUS that have enough votes. Applying 
them would cut STATUS down to half (roughly).


Regards,

Rainer

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



Re: Release plans for 6.0.27 - ?

2010-06-06 Thread Mark Thomas
On 06/06/2010 18:32, Rainer Jung wrote:
> On 06.06.2010 17:41, jean-frederic clere wrote:
>> On 06/04/2010 09:57 PM, Konstantin Kolinko wrote:
>>> Hi!
>>>
>>> When can we make 6.0.27?
>>>
>>> There were a couple of important fixes since 6.0.26, like BZ 48903
>>> (classloader deadlock).
>>>
>>> I think it is about time to make 6.0.27.
>>
>> +1 I will start to prepare the stuff. Tag on Monday, packaging , votes
>> and release before the next week-end, is that ok?
> 
> There are about 10 proposals in STATUS that have enough votes. Applying
> them would cut STATUS down to half (roughly).

I've got some 7.0.x stuff I want to finish before a try another 7.0.0
RC. Reviewing the 6.0.x status file is next on my todo list after that.

Mark



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



svn commit: r951909 - in /tomcat/trunk: java/org/apache/catalina/session/StandardSession.java webapps/docs/config/systemprops.xml

2010-06-06 Thread markt
Author: markt
Date: Sun Jun  6 17:36:34 2010
New Revision: 951909

URL: http://svn.apache.org/viewvc?rev=951909&view=rev
Log:
Use STRICT_SERVLET_COMPLIANCE to override the default for ACTIVITY_CHECK to 
align behaviour with other system properties

Modified:
tomcat/trunk/java/org/apache/catalina/session/StandardSession.java
tomcat/trunk/webapps/docs/config/systemprops.xml

Modified: tomcat/trunk/java/org/apache/catalina/session/StandardSession.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/session/StandardSession.java?rev=951909&r1=951908&r2=951909&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/session/StandardSession.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/session/StandardSession.java Sun Jun  
6 17:36:34 2010
@@ -84,14 +84,28 @@ import org.apache.catalina.security.Secu
 public class StandardSession
 implements HttpSession, Session, Serializable {
 
+protected static final boolean STRICT_SERVLET_COMPLIANCE;
 
-protected static final boolean ACTIVITY_CHECK = 
-Globals.STRICT_SERVLET_COMPLIANCE
-|| 
Boolean.valueOf(System.getProperty("org.apache.catalina.session.StandardSession.ACTIVITY_CHECK",
 "false")).booleanValue();
+protected static final boolean ACTIVITY_CHECK;
 
 
 // --- Constructors
 
+static {
+STRICT_SERVLET_COMPLIANCE = Boolean.valueOf(System.getProperty(
+"org.apache.catalina.STRICT_SERVLET_COMPLIANCE",
+"false")).booleanValue();
+
+String activityCheck = System.getProperty(
+"org.apache.catalina.session.StandardSession.ACTIVITY_CHECK");
+if (activityCheck == null) {
+ACTIVITY_CHECK = STRICT_SERVLET_COMPLIANCE;
+} else {
+ACTIVITY_CHECK =
+Boolean.valueOf(activityCheck).booleanValue();
+}
+}
+
 
 /**
  * Construct a new Session associated with the specified Manager.

Modified: tomcat/trunk/webapps/docs/config/systemprops.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/config/systemprops.xml?rev=951909&r1=951908&r2=951909&view=diff
==
--- tomcat/trunk/webapps/docs/config/systemprops.xml (original)
+++ tomcat/trunk/webapps/docs/config/systemprops.xml Sun Jun  6 17:36:34 2010
@@ -239,6 +239,7 @@
   
 The default value will be changed for:
 
+
org.apache.catalina.session.StandardSession.ACTIVITY_CHECK
 
org.apache.tomcat.util.http.ServerCookie.ALWAYS_ADD_EXPIRES.
 
org.apache.tomcat.util.http.ServerCookie.FWD_SLASH_IS_SEPARATOR.
 
org.apache.tomcat.util.http.ServerCookie.STRICT_NAMING.
@@ -277,16 +278,6 @@
 
 
 
-  If this is true or if
-  org.apache.catalina.STRICT_SERVLET_COMPLIANCE is
-  true Tomcat will track the number of active requests for 
each
-  session. When determining if a session is valid, any session with at 
least
-  one active request will always be considered valid. If not specified, the
-  default value of false will be used.
-
-
-
   If this is true Tomcat will allow =
   characters when parsing unquoted cookie values. If false,
@@ -352,6 +343,16 @@
   JSESSIONIDSSO.
 
 
+
+  If this is true Tomcat will track the number of active
+  requests for each session. When determining if a session is valid, any
+  session with at least one active request will always be considered valid.
+  If org.apache.catalina.STRICT_SERVLET_COMPLIANCE is set to
+  true, the default of this setting will be true,
+  else the default value will be false.
+
+
   
 
 



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



svn commit: r951921 - /tomcat/trunk/java/org/apache/catalina/session/StandardSession.java

2010-06-06 Thread markt
Author: markt
Date: Sun Jun  6 18:23:33 2010
New Revision: 951921

URL: http://svn.apache.org/viewvc?rev=951921&view=rev
Log:
Put the comment in the right place

Modified:
tomcat/trunk/java/org/apache/catalina/session/StandardSession.java

Modified: tomcat/trunk/java/org/apache/catalina/session/StandardSession.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/session/StandardSession.java?rev=951921&r1=951920&r2=951921&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/session/StandardSession.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/session/StandardSession.java Sun Jun  
6 18:23:33 2010
@@ -89,8 +89,6 @@ public class StandardSession
 protected static final boolean ACTIVITY_CHECK;
 
 
-// --- Constructors
-
 static {
 STRICT_SERVLET_COMPLIANCE = Boolean.valueOf(System.getProperty(
 "org.apache.catalina.STRICT_SERVLET_COMPLIANCE",
@@ -107,6 +105,9 @@ public class StandardSession
 }
 
 
+// --- Constructors
+
+
 /**
  * Construct a new Session associated with the specified Manager.
  *



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



svn commit: r951922 - in /tomcat/trunk: java/org/apache/catalina/core/StandardHostValve.java webapps/docs/config/systemprops.xml

2010-06-06 Thread markt
Author: markt
Date: Sun Jun  6 18:29:26 2010
New Revision: 951922

URL: http://svn.apache.org/viewvc?rev=951922&view=rev
Log:
Part of refactoring STRICT_SERVLET_COMPLIANCE so it just changes defaults
New ACCESS_SESSION property

Modified:
tomcat/trunk/java/org/apache/catalina/core/StandardHostValve.java
tomcat/trunk/webapps/docs/config/systemprops.xml

Modified: tomcat/trunk/java/org/apache/catalina/core/StandardHostValve.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/StandardHostValve.java?rev=951922&r1=951921&r2=951922&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/core/StandardHostValve.java (original)
+++ tomcat/trunk/java/org/apache/catalina/core/StandardHostValve.java Sun Jun  
6 18:29:26 2010
@@ -61,6 +61,25 @@ final class StandardHostValve
 
 private static final Log log = LogFactory.getLog(StandardHostValve.class);
 
+protected static final boolean STRICT_SERVLET_COMPLIANCE;
+
+protected static final boolean ACCESS_SESSION;
+
+static {
+STRICT_SERVLET_COMPLIANCE = Boolean.valueOf(System.getProperty(
+"org.apache.catalina.STRICT_SERVLET_COMPLIANCE",
+"false")).booleanValue();
+
+String accessSession = System.getProperty(
+"org.apache.catalina.core.StandardHostValve.ACCESS_SESSION");
+if (accessSession == null) {
+ACCESS_SESSION = STRICT_SERVLET_COMPLIANCE;
+} else {
+ACCESS_SESSION =
+Boolean.valueOf(accessSession).booleanValue();
+}
+}
+
 //-- Constructor
 public StandardHostValve() {
 super(true);
@@ -147,7 +166,7 @@ final class StandardHostValve
 
 // Access a session (if present) to update last accessed time, based 
on a
 // strict interpretation of the specification
-if (Globals.STRICT_SERVLET_COMPLIANCE) {
+if (ACCESS_SESSION) {
 request.getSession(false);
 }
 
@@ -205,7 +224,7 @@ final class StandardHostValve
 
 // Access a session (if present) to update last accessed time, based 
on a
 // strict interpretation of the specification
-if (Globals.STRICT_SERVLET_COMPLIANCE) {
+if (ACCESS_SESSION) {
 request.getSession(false);
 }
 

Modified: tomcat/trunk/webapps/docs/config/systemprops.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/config/systemprops.xml?rev=951922&r1=951921&r2=951922&view=diff
==
--- tomcat/trunk/webapps/docs/config/systemprops.xml (original)
+++ tomcat/trunk/webapps/docs/config/systemprops.xml Sun Jun  6 18:29:26 2010
@@ -228,10 +228,6 @@
   dispatcher will be checked to ensure that it has wrapped the original
   request or response. (SRV.8.2 / SRV.14.2.5.1)
   
-  every request that is associated with a session will cause the
-  session's last accessed time to be updated regardless of whether or not
-  the request explicitly accesses the session. (SRV.7.6)  
-  
   
 ServletContext.getResource/getResourceAsStream must start 
with "/"
 if set to false, code like 
getResource("myfolder/myresource.txt") will work
@@ -239,6 +235,7 @@
   
 The default value will be changed for:
 
+
org.apache.catalina.core.StandardHostValve.ACCESS_SESSION
 
org.apache.catalina.session.StandardSession.ACTIVITY_CHECK
 
org.apache.tomcat.util.http.ServerCookie.ALWAYS_ADD_EXPIRES.
 
org.apache.tomcat.util.http.ServerCookie.FWD_SLASH_IS_SEPARATOR.
@@ -343,9 +340,18 @@
   JSESSIONIDSSO.
 
 
+
+  If this is true, every request that is associated with a
+  session will cause the session's last accessed time to be updated
+  regardless of whether or not the request explicitly accesses the session.
+  If org.apache.catalina.STRICT_SERVLET_COMPLIANCE is set to
+  true, the default of this setting will be true,
+  else the default value will be false.
+
+
 
-  If this is true Tomcat will track the number of active
+  If this is true, Tomcat will track the number of active
   requests for each session. When determining if a session is valid, any
   session with at least one active request will always be considered valid.
   If org.apache.catalina.STRICT_SERVLET_COMPLIANCE is set to



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



svn commit: r951923 - in /tomcat/trunk: java/org/apache/catalina/core/ApplicationContext.java webapps/docs/config/systemprops.xml

2010-06-06 Thread markt
Author: markt
Date: Sun Jun  6 18:39:33 2010
New Revision: 951923

URL: http://svn.apache.org/viewvc?rev=951923&view=rev
Log:
Part of refactoring STRICT_SERVLET_COMPLIANCE so it just changes defaults
New GET_RESOURCE_REQUIRE_SLASH property

Modified:
tomcat/trunk/java/org/apache/catalina/core/ApplicationContext.java
tomcat/trunk/webapps/docs/config/systemprops.xml

Modified: tomcat/trunk/java/org/apache/catalina/core/ApplicationContext.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/ApplicationContext.java?rev=951923&r1=951922&r2=951923&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/core/ApplicationContext.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/core/ApplicationContext.java Sun Jun  
6 18:39:33 2010
@@ -93,6 +93,24 @@ import org.apache.catalina.Globals;
 public class ApplicationContext
 implements ServletContext {
 
+protected static final boolean STRICT_SERVLET_COMPLIANCE;
+
+protected static final boolean GET_RESOURCE_REQUIRE_SLASH;
+
+
+static {
+STRICT_SERVLET_COMPLIANCE = Globals.STRICT_SERVLET_COMPLIANCE;
+
+String activityCheck = System.getProperty(
+
"org.apache.catalina.core.ApplicationContext.GET_RESOURCE_REQUIRE_SLASH");
+if (activityCheck == null) {
+GET_RESOURCE_REQUIRE_SLASH = STRICT_SERVLET_COMPLIANCE;
+} else {
+GET_RESOURCE_REQUIRE_SLASH =
+Boolean.valueOf(activityCheck).booleanValue();
+}
+}
+
 // --- Constructors
 
 
@@ -497,7 +515,7 @@ public class ApplicationContext
 if (path == null)
 throw new 
MalformedURLException(sm.getString("applicationContext.requestDispatcher.iae", 
path));
 
-if (!path.startsWith("/") && Globals.STRICT_SERVLET_COMPLIANCE)
+if (!path.startsWith("/") && GET_RESOURCE_REQUIRE_SLASH)
 throw new 
MalformedURLException(sm.getString("applicationContext.requestDispatcher.iae", 
path));
 
 
@@ -541,7 +559,7 @@ public class ApplicationContext
 if (path == null)
 return (null);
 
-if (!path.startsWith("/") && Globals.STRICT_SERVLET_COMPLIANCE)
+if (!path.startsWith("/") && GET_RESOURCE_REQUIRE_SLASH)
 return null;
 
 String normalizedPath = RequestUtil.normalize(path);

Modified: tomcat/trunk/webapps/docs/config/systemprops.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/config/systemprops.xml?rev=951923&r1=951922&r2=951923&view=diff
==
--- tomcat/trunk/webapps/docs/config/systemprops.xml (original)
+++ tomcat/trunk/webapps/docs/config/systemprops.xml Sun Jun  6 18:39:33 2010
@@ -229,12 +229,9 @@
   request or response. (SRV.8.2 / SRV.14.2.5.1)
   
   
-ServletContext.getResource/getResourceAsStream must start 
with "/"
-if set to false, code like 
getResource("myfolder/myresource.txt") will work
-  
-  
 The default value will be changed for:
 
+
rg.apache.catalina.core.ApplicationContext.GET_RESOURCE_REQUIRE_SLASH
 
org.apache.catalina.core.StandardHostValve.ACCESS_SESSION
 
org.apache.catalina.session.StandardSession.ACTIVITY_CHECK
 
org.apache.tomcat.util.http.ServerCookie.ALWAYS_ADD_EXPIRES.
@@ -274,6 +271,17 @@
   true will be used.
 
 
+
+  If this is true then the path passed to
+  ServletContext.getResource() or
+  ServletContext.getResourceAsStream() must start with
+  "/". If false, code like
+  getResource("myfolder/myresource.txt") will work. If
+  org.apache.catalina.STRICT_SERVLET_COMPLIANCE is set to
+  true, the default of this setting will be true,
+  else the default value will be false.
+
+
 
   If this is true Tomcat will allow =



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



svn commit: r951924 - /tomcat/trunk/webapps/docs/config/systemprops.xml

2010-06-06 Thread markt
Author: markt
Date: Sun Jun  6 18:39:48 2010
New Revision: 951924

URL: http://svn.apache.org/viewvc?rev=951924&view=rev
Log:
Typo

Modified:
tomcat/trunk/webapps/docs/config/systemprops.xml

Modified: tomcat/trunk/webapps/docs/config/systemprops.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/config/systemprops.xml?rev=951924&r1=951923&r2=951924&view=diff
==
--- tomcat/trunk/webapps/docs/config/systemprops.xml (original)
+++ tomcat/trunk/webapps/docs/config/systemprops.xml Sun Jun  6 18:39:48 2010
@@ -231,7 +231,7 @@
   
 The default value will be changed for:
 
-
rg.apache.catalina.core.ApplicationContext.GET_RESOURCE_REQUIRE_SLASH
+
org.apache.catalina.core.ApplicationContext.GET_RESOURCE_REQUIRE_SLASH
 
org.apache.catalina.core.StandardHostValve.ACCESS_SESSION
 
org.apache.catalina.session.StandardSession.ACTIVITY_CHECK
 
org.apache.tomcat.util.http.ServerCookie.ALWAYS_ADD_EXPIRES.



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



svn commit: r951925 - /tomcat/trunk/java/org/apache/catalina/core/ApplicationContext.java

2010-06-06 Thread markt
Author: markt
Date: Sun Jun  6 18:42:12 2010
New Revision: 951925

URL: http://svn.apache.org/viewvc?rev=951925&view=rev
Log:
Copy/paste error. Sorry for the noise.

Modified:
tomcat/trunk/java/org/apache/catalina/core/ApplicationContext.java

Modified: tomcat/trunk/java/org/apache/catalina/core/ApplicationContext.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/ApplicationContext.java?rev=951925&r1=951924&r2=951925&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/core/ApplicationContext.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/core/ApplicationContext.java Sun Jun  
6 18:42:12 2010
@@ -101,13 +101,13 @@ public class ApplicationContext
 static {
 STRICT_SERVLET_COMPLIANCE = Globals.STRICT_SERVLET_COMPLIANCE;
 
-String activityCheck = System.getProperty(
+String requireSlash = System.getProperty(
 
"org.apache.catalina.core.ApplicationContext.GET_RESOURCE_REQUIRE_SLASH");
-if (activityCheck == null) {
+if (requireSlash == null) {
 GET_RESOURCE_REQUIRE_SLASH = STRICT_SERVLET_COMPLIANCE;
 } else {
 GET_RESOURCE_REQUIRE_SLASH =
-Boolean.valueOf(activityCheck).booleanValue();
+Boolean.valueOf(requireSlash).booleanValue();
 }
 }
 



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



svn commit: r951926 - in /tomcat/trunk: java/org/apache/catalina/core/ApplicationDispatcher.java webapps/docs/config/systemprops.xml

2010-06-06 Thread markt
Author: markt
Date: Sun Jun  6 18:47:14 2010
New Revision: 951926

URL: http://svn.apache.org/viewvc?rev=951926&view=rev
Log:
Part of refactoring STRICT_SERVLET_COMPLIANCE so it just changes defaults
New WRAP_SAME_OBJECT property

Modified:
tomcat/trunk/java/org/apache/catalina/core/ApplicationDispatcher.java
tomcat/trunk/webapps/docs/config/systemprops.xml

Modified: tomcat/trunk/java/org/apache/catalina/core/ApplicationDispatcher.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/ApplicationDispatcher.java?rev=951926&r1=951925&r2=951926&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/core/ApplicationDispatcher.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/core/ApplicationDispatcher.java Sun 
Jun  6 18:47:14 2010
@@ -70,6 +70,25 @@ final class ApplicationDispatcher
 
 private static final Log log = 
LogFactory.getLog(ApplicationDispatcher.class);
 
+protected static final boolean STRICT_SERVLET_COMPLIANCE;
+
+protected static final boolean WRAP_SAME_OBJECT;
+
+
+static {
+STRICT_SERVLET_COMPLIANCE = Globals.STRICT_SERVLET_COMPLIANCE;
+
+String wrapSameObject = System.getProperty(
+
"org.apache.catalina.core.ApplicationDispatcher.WRAP_SAME_OBJECT");
+if (wrapSameObject == null) {
+WRAP_SAME_OBJECT = STRICT_SERVLET_COMPLIANCE;
+} else {
+WRAP_SAME_OBJECT =
+Boolean.valueOf(wrapSameObject).booleanValue();
+}
+}
+
+
 protected class PrivilegedForward
 implements PrivilegedExceptionAction {
 private ServletRequest request;
@@ -329,7 +348,7 @@ final class ApplicationDispatcher
 // Set up to handle the specified request and response
 State state = new State(request, response, false);
 
-if (Globals.STRICT_SERVLET_COMPLIANCE) {
+if (WRAP_SAME_OBJECT) {
 // Check SRV.8.2 / SRV.14.2.5.1 compliance
 checkSameObjects(request, response);
 }
@@ -504,7 +523,7 @@ final class ApplicationDispatcher
 // Set up to handle the specified request and response
 State state = new State(request, response, true);
 
-if (Globals.STRICT_SERVLET_COMPLIANCE) {
+if (WRAP_SAME_OBJECT) {
 // Check SRV.8.2 / SRV.14.2.5.1 compliance
 checkSameObjects(request, response);
 }

Modified: tomcat/trunk/webapps/docs/config/systemprops.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/config/systemprops.xml?rev=951926&r1=951925&r2=951926&view=diff
==
--- tomcat/trunk/webapps/docs/config/systemprops.xml (original)
+++ tomcat/trunk/webapps/docs/config/systemprops.xml Sun Jun  6 18:47:14 2010
@@ -222,31 +222,23 @@
   
 
 
-  If this is true the following actions will occur:
+  If this is true the default values will be changed for:
   
-  any wrapped request or response object passed to an application
-  dispatcher will be checked to ensure that it has wrapped the original
-  request or response. (SRV.8.2 / SRV.14.2.5.1)
-  
-  
-The default value will be changed for:
-
-
org.apache.catalina.core.ApplicationContext.GET_RESOURCE_REQUIRE_SLASH
-
org.apache.catalina.core.StandardHostValve.ACCESS_SESSION
-
org.apache.catalina.session.StandardSession.ACTIVITY_CHECK
-
org.apache.tomcat.util.http.ServerCookie.ALWAYS_ADD_EXPIRES.
-
org.apache.tomcat.util.http.ServerCookie.FWD_SLASH_IS_SEPARATOR.
-
org.apache.tomcat.util.http.ServerCookie.STRICT_NAMING.
-The tldNamespaceAware attribute of any
-Context element.
-The tldValidation attribute of any
-Context element.
-The xmlNamespaceAware attribute of any
-Context element.
-The xmlValidation attribute of any
-Context element.
-
-  
+  
org.apache.catalina.core.ApplicationContext.GET_RESOURCE_REQUIRE_SLASH
+  
org.apache.catalina.core.ApplicationDispatcher.WRAP_SAME_OBJECT
+  
org.apache.catalina.core.StandardHostValve.ACCESS_SESSION
+  
org.apache.catalina.session.StandardSession.ACTIVITY_CHECK
+  
org.apache.tomcat.util.http.ServerCookie.ALWAYS_ADD_EXPIRES.
+  
org.apache.tomcat.util.http.ServerCookie.FWD_SLASH_IS_SEPARATOR.
+  
org.apache.tomcat.util.http.ServerCookie.STRICT_NAMING.
+  The tldNamespaceAware attribute of any
+  Context element.
+  The tldValidation attribute of any
+  Context element.
+  The xmlNamespaceAware attribute of any
+  Context element.
+  The xmlValidation attribute of any
+  Context element.
   
   
   Note that changing a number of the above defaults is likely to break
@@ -282,6 +274,15 @@
   else the default val

svn commit: r951930 - in /tomcat/tc6.0.x/trunk: ./ java/org/apache/catalina/core/ java/org/apache/catalina/loader/ webapps/docs/ webapps/docs/config/

2010-06-06 Thread kkolinko
Author: kkolinko
Date: Sun Jun  6 19:17:58 2010
New Revision: 951930

URL: http://svn.apache.org/viewvc?rev=951930&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48971
Make stopping of leaking Timer threads optional and disabled by default.
It is merge of r950164, r950614 from trunk.

Modified:
tomcat/tc6.0.x/trunk/   (props changed)
tomcat/tc6.0.x/trunk/STATUS.txt
tomcat/tc6.0.x/trunk/java/org/apache/catalina/core/StandardContext.java
tomcat/tc6.0.x/trunk/java/org/apache/catalina/loader/WebappClassLoader.java
tomcat/tc6.0.x/trunk/java/org/apache/catalina/loader/WebappLoader.java
tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml
tomcat/tc6.0.x/trunk/webapps/docs/config/context.xml

Propchange: tomcat/tc6.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Sun Jun  6 19:17:58 2010
@@ -1 +1 @@
-/tomcat/trunk:601180,606992,612607,630314,640888,652744,653247,666232,673796,673820,677910,683969,683982,684001,684081,684234,684269-684270,685177,687503,687645,689402,690781,691392,691805,692748,693378,694992,695053,695311,696780,696782,698012,698227,698236,698613,699427,699634,701355,709294,709811,709816,710063,710066,710125,710205,711126,711600,712461,712467,713953,714002,718360,719119,719124,719602,719626,719628,720046,720069,721040,721286,721708,721886,723404,723738,726052,727303,728032,728768,728947,729057,729567,729569,729571,729681,729809,729815,729934,730250,730590,731651,732859,732863,734734,740675,740684,742677,742697,742714,744160,744238,746321,746384,746425,747834,747863,748344,750258,750291,750921,751286-751287,751289,751295,752323,753039,757335,757774,758249,758365,758596,758616,758664,759074,761601,762868,762929,762936-762937,763166,763183,763193,763228,763262,763298,763302,763325,763599,763611,763654,763681,763706,764985,764997,765662,768335,769979,770716,77
 
0809,770876,772872,776921,776924,776935,776945,777464,777466,777576,777625,778379,778523-778524,781528,781779,782145,782791,783316,783696,783724,783756,783762,783766,783863,783934,784453,784602,784614,785381,785688,785768,785859,786468,786487,786490,786496,786667,787627,787770,787985,789389,790405,791041,791184,791194,791224,791243,791326,791328,791789,792740,793372,793757,793882,793981,794082,794673,794822,795043,795152,795210,795457,795466,797168,797425,797596,797607,802727,802940,804462,804544,804734,805153,809131,809603,810916,810977,812125,812137,812432,813001,813013,813866,814180,814708,814876,815972,816252,817442,817822,819339,819361,820110,820132,820874,820954,821397,828196,828201,828210,828225,828759,830378-830379,830999,831106,831774,831785,831828,831850,831860,832214,832218,833121,833545,834047,835036,835336,836405,881396,881412,883130,883134,883146,883165,883177,883362,883565,884341,885038,885231,885241,885260,885901,885991,886019,888072,889363,889606,889716,8901
 
39,890265,890349-890350,890417,891185-891187,891583,892198,892341,892415,892464,892555,892812,892814,892817,892843,892887,893321,893493,894580,894586,894805,894831,895013,895045,895057,895191,895392,895703,896370,896384,897380-897381,897776,898126,898256,898468,898527,898555,898558,898718,898836,898906,899284,899348,899420,899653,899769-899770,899783,899788,899792,899916,899918-899919,899935,899949,903916,905020,905151,905722,905728,905735,907311,907513,907538,907652,907819,907825,907864,908002,908721,908754,908759,909097,909206,909212,909525,909636,909869,909875,909887,910266,910370,910442,910471,910485,910974,915226,915737,915861,916097,916141,916157,916170,917598,917633,918093,918489,918594,918684,918787,918792,918799,918803,918885,919851,919914,920025,920055,920298,920449,920596,920824,920840,921444,922010,926716,927062,927621,928482,928695,928732,928798,931709,932357,932967,935105,935983,939491,939551,940064,941356,941463,944409,944416,945808,945835,945841,946686,950596
 ,951615
+/tomcat/trunk:601180,606992,612607,630314,640888,652744,653247,666232,673796,673820,677910,683969,683982,684001,684081,684234,684269-684270,685177,687503,687645,689402,690781,691392,691805,692748,693378,694992,695053,695311,696780,696782,698012,698227,698236,698613,699427,699634,701355,709294,709811,709816,710063,710066,710125,710205,711126,711600,712461,712467,713953,714002,718360,719119,719124,719602,719626,719628,720046,720069,721040,721286,721708,721886,723404,723738,726052,727303,728032,728768,728947,729057,729567,729569,729571,729681,729809,729815,729934,730250,730590,731651,732859,732863,734734,740675,740684,742677,742697,742714,744160,744238,746321,746384,746425,747834,747863,748344,750258,750291,750921,751286-751287,751289,751295,752323,753039,757335,757774,758249,758365,758596,758616,758664,759074,761601,762868,762929,762936-762937,763166,763183,763193,763228,763262,763298,763302,763325,763599,763611,763654,763681,763706,764985,764997,765662,768335,769979,770716,77
 
0809,770876,772872,776921,776924,776935,776945,777464,777466,777576,777625,778379,7785

DO NOT REPLY [Bug 48971] memory leak protection : stopping TimeThreads should be optional and disabled by default

2010-06-06 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=48971

Konstantin Kolinko  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

--- Comment #12 from Konstantin Kolinko  2010-06-06 
15:22:26 EDT ---
The fix was applied to 6.0 in r951930 and will be in 6.0.27 onwards.
Stopping the java.util.TimerThread threads is now optional and disabled by
default.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

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



svn commit: r951941 - in /tomcat/tc6.0.x/trunk: ./ STATUS.txt java/javax/servlet/jsp/PageContext.java webapps/docs/changelog.xml

2010-06-06 Thread kkolinko
Author: kkolinko
Date: Sun Jun  6 19:56:59 2010
New Revision: 951941

URL: http://svn.apache.org/viewvc?rev=951941&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49196
Avoid NullPointerException in PageContext.getErrorData()

Modified:
tomcat/tc6.0.x/trunk/   (props changed)
tomcat/tc6.0.x/trunk/STATUS.txt
tomcat/tc6.0.x/trunk/java/javax/servlet/jsp/PageContext.java
tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc6.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Sun Jun  6 19:56:59 2010
@@ -1 +1 @@
-/tomcat/trunk:601180,606992,612607,630314,640888,652744,653247,666232,673796,673820,677910,683969,683982,684001,684081,684234,684269-684270,685177,687503,687645,689402,690781,691392,691805,692748,693378,694992,695053,695311,696780,696782,698012,698227,698236,698613,699427,699634,701355,709294,709811,709816,710063,710066,710125,710205,711126,711600,712461,712467,713953,714002,718360,719119,719124,719602,719626,719628,720046,720069,721040,721286,721708,721886,723404,723738,726052,727303,728032,728768,728947,729057,729567,729569,729571,729681,729809,729815,729934,730250,730590,731651,732859,732863,734734,740675,740684,742677,742697,742714,744160,744238,746321,746384,746425,747834,747863,748344,750258,750291,750921,751286-751287,751289,751295,752323,753039,757335,757774,758249,758365,758596,758616,758664,759074,761601,762868,762929,762936-762937,763166,763183,763193,763228,763262,763298,763302,763325,763599,763611,763654,763681,763706,764985,764997,765662,768335,769979,770716,77
 
0809,770876,772872,776921,776924,776935,776945,777464,777466,777576,777625,778379,778523-778524,781528,781779,782145,782791,783316,783696,783724,783756,783762,783766,783863,783934,784453,784602,784614,785381,785688,785768,785859,786468,786487,786490,786496,786667,787627,787770,787985,789389,790405,791041,791184,791194,791224,791243,791326,791328,791789,792740,793372,793757,793882,793981,794082,794673,794822,795043,795152,795210,795457,795466,797168,797425,797596,797607,802727,802940,804462,804544,804734,805153,809131,809603,810916,810977,812125,812137,812432,813001,813013,813866,814180,814708,814876,815972,816252,817442,817822,819339,819361,820110,820132,820874,820954,821397,828196,828201,828210,828225,828759,830378-830379,830999,831106,831774,831785,831828,831850,831860,832214,832218,833121,833545,834047,835036,835336,836405,881396,881412,883130,883134,883146,883165,883177,883362,883565,884341,885038,885231,885241,885260,885901,885991,886019,888072,889363,889606,889716,8901
 
39,890265,890349-890350,890417,891185-891187,891583,892198,892341,892415,892464,892555,892812,892814,892817,892843,892887,893321,893493,894580,894586,894805,894831,895013,895045,895057,895191,895392,895703,896370,896384,897380-897381,897776,898126,898256,898468,898527,898555,898558,898718,898836,898906,899284,899348,899420,899653,899769-899770,899783,899788,899792,899916,899918-899919,899935,899949,903916,905020,905151,905722,905728,905735,907311,907513,907538,907652,907819,907825,907864,908002,908721,908754,908759,909097,909206,909212,909525,909636,909869,909875,909887,910266,910370,910442,910471,910485,910974,915226,915737,915861,916097,916141,916157,916170,917598,917633,918093,918489,918594,918684,918787,918792,918799,918803,918885,919851,919914,920025,920055,920298,920449,920596,920824,920840,921444,922010,926716,927062,927621,928482,928695,928732,928798,931709,932357,932967,935105,935983,939491,939551,940064,941356,941463,944409,944416,945808,945835,945841,946686,950164
 ,950596,950614,951615
+/tomcat/trunk:601180,606992,612607,630314,640888,652744,653247,666232,673796,673820,677910,683969,683982,684001,684081,684234,684269-684270,685177,687503,687645,689402,690781,691392,691805,692748,693378,694992,695053,695311,696780,696782,698012,698227,698236,698613,699427,699634,701355,709294,709811,709816,710063,710066,710125,710205,711126,711600,712461,712467,713953,714002,718360,719119,719124,719602,719626,719628,720046,720069,721040,721286,721708,721886,723404,723738,726052,727303,728032,728768,728947,729057,729567,729569,729571,729681,729809,729815,729934,730250,730590,731651,732859,732863,734734,740675,740684,742677,742697,742714,744160,744238,746321,746384,746425,747834,747863,748344,750258,750291,750921,751286-751287,751289,751295,752323,753039,757335,757774,758249,758365,758596,758616,758664,759074,761601,762868,762929,762936-762937,763166,763183,763193,763228,763262,763298,763302,763325,763599,763611,763654,763681,763706,764985,764997,765662,768335,769979,770716,77
 
0809,770876,772872,776921,776924,776935,776945,777464,777466,777576,777625,778379,778523-778524,781528,781779,782145,782791,783316,783696,783724,783756,783762,783766,783863,783934,784453,784602,784614,785381,785688,785768,785859,786468,786487,786490,786496,786667,787627,787770,787985,789389,790405,791041,791184,791194,791224,791243,791326,791328,791789,

svn commit: r951943 - /tomcat/tc5.5.x/trunk/servletapi/jsr152/src/share/javax/servlet/jsp/PageContext.java

2010-06-06 Thread kkolinko
Author: kkolinko
Date: Sun Jun  6 20:01:56 2010
New Revision: 951943

URL: http://svn.apache.org/viewvc?rev=951943&view=rev
Log:
CTR. Tabs to spaces.

Modified:

tomcat/tc5.5.x/trunk/servletapi/jsr152/src/share/javax/servlet/jsp/PageContext.java

Modified: 
tomcat/tc5.5.x/trunk/servletapi/jsr152/src/share/javax/servlet/jsp/PageContext.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/servletapi/jsr152/src/share/javax/servlet/jsp/PageContext.java?rev=951943&r1=951942&r2=951943&view=diff
==
--- 
tomcat/tc5.5.x/trunk/servletapi/jsr152/src/share/javax/servlet/jsp/PageContext.java
 (original)
+++ 
tomcat/tc5.5.x/trunk/servletapi/jsr152/src/share/javax/servlet/jsp/PageContext.java
 Sun Jun  6 20:01:56 2010
@@ -512,12 +512,12 @@ abstract public class PageContext 
  * @since 2.0
  */
 public ErrorData getErrorData() {
-   return new ErrorData( 
-   (Throwable)getRequest().getAttribute( 
"javax.servlet.error.exception" ),
-   ((Integer)getRequest().getAttribute( 
-   "javax.servlet.error.status_code" )).intValue(),
-   (String)getRequest().getAttribute( 
"javax.servlet.error.request_uri" ),
-   (String)getRequest().getAttribute( 
"javax.servlet.error.servlet_name" ) );
+return new ErrorData( 
+(Throwable)getRequest().getAttribute( 
"javax.servlet.error.exception" ),
+((Integer)getRequest().getAttribute( 
+"javax.servlet.error.status_code" )).intValue(),
+(String)getRequest().getAttribute( 
"javax.servlet.error.request_uri" ),
+(String)getRequest().getAttribute( 
"javax.servlet.error.servlet_name" ) );
 }
 
 }



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



DO NOT REPLY [Bug 49196] NPE in PageContext.getErrorData()

2010-06-06 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49196

--- Comment #2 from Konstantin Kolinko  2010-06-06 
16:07:12 EDT ---
Created an attachment (id=25534)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=25534)
2010-06-07_tc55_bug49196.patch

Patch for 5.5. It is backport of r948057 from trunk.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

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



svn commit: r951950 - /tomcat/tc5.5.x/trunk/STATUS.txt

2010-06-06 Thread kkolinko
Author: kkolinko
Date: Sun Jun  6 20:12:34 2010
New Revision: 951950

URL: http://svn.apache.org/viewvc?rev=951950&view=rev
Log:
proposal

Modified:
tomcat/tc5.5.x/trunk/STATUS.txt

Modified: tomcat/tc5.5.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/STATUS.txt?rev=951950&r1=951949&r2=951950&view=diff
==
--- tomcat/tc5.5.x/trunk/STATUS.txt (original)
+++ tomcat/tc5.5.x/trunk/STATUS.txt Sun Jun  6 20:12:34 2010
@@ -66,3 +66,11 @@ PATCHES PROPOSED TO BACKPORT:
   (Easier to review if you ignore whitespaces during the diff)
   +1: kkolinko
   -1:
+
+* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49196
+  Avoid NullPointerException in PageContext.getErrorData() if an
+  error-handling JSP page is called directly.
+  It is backport of r948057.
+  https://issues.apache.org/bugzilla/attachment.cgi?id=25534
+  +1: kkolinko
+  -1: 



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



DO NOT REPLY [Bug 49196] NPE in PageContext.getErrorData()

2010-06-06 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49196

Konstantin Kolinko  changed:

   What|Removed |Added

  Component|Catalina|Catalina
Version|6.0.26  |5.5.28
Product|Tomcat 6|Tomcat 5
   Target Milestone|default |---

--- Comment #3 from Konstantin Kolinko  2010-06-06 
16:15:26 EDT ---
The fix was applied to 6.0 in r951941 and will be in 6.0.27.

Proposed for backport to 5.5, so I am changing Product and Version to track
this issue further.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

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



svn commit: r951951 - /tomcat/trunk/test/webapp-3.0/bug49196.jsp

2010-06-06 Thread kkolinko
Author: kkolinko
Date: Sun Jun  6 20:20:02 2010
New Revision: 951951

URL: http://svn.apache.org/viewvc?rev=951951&view=rev
Log:
correct a typo in a comment

Modified:
tomcat/trunk/test/webapp-3.0/bug49196.jsp

Modified: tomcat/trunk/test/webapp-3.0/bug49196.jsp
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/test/webapp-3.0/bug49196.jsp?rev=951951&r1=951950&r2=951951&view=diff
==
--- tomcat/trunk/test/webapp-3.0/bug49196.jsp (original)
+++ tomcat/trunk/test/webapp-3.0/bug49196.jsp Sun Jun  6 20:20:02 2010
@@ -15,7 +15,7 @@
   limitations under the License.
 --%>
 <%
-// As nonsensical as this is, in't shouldn't throw an NPE
+// As nonsensical as this is, it shouldn't throw an NPE
 pageContext.getErrorData();
 %>
 



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



[Tomcat Wiki] Update of "MemoryLeakProtection" by Sylva inLaurent

2010-06-06 Thread Apache Wiki
Dear Wiki user,

You have subscribed to a wiki page or wiki category on "Tomcat Wiki" for change 
notification.

The "MemoryLeakProtection" page has been changed by SylvainLaurent.
http://wiki.apache.org/tomcat/MemoryLeakProtection?action=diff&rev1=10&rev2=11

--

  
  = Summary matrix =
  ||'''Leak cause'''||'''Detected by tomcat'''||'''Fixed by 
tomcat'''||'''Possible enhancements'''||
- ||[[#customThreadLocal|Custom ThreadLocal class]]||>=6.0.24||>=6.0.24|| 
[[#48837|Detect child classloaders]], [[#48895|improve thread safety]] ||
- ||[[#webappClassInstanceAsThreadLocalValue|Webapp class instance as 
ThreadLocal value]]||>=6.0.24||>=6.0.24|| [[#48837|Detect child classloaders]], 
[[#48895|improve thread safety]] ||
+ ||[[#customThreadLocal|Custom ThreadLocal class]]||>=6.0.24||6.0.24-6.0.26 
but is unsafe. Made optional from 6.0.27|| [[#48837|Detect child 
classloaders]], [[#49159|BZ 49159]] ||
+ ||[[#webappClassInstanceAsThreadLocalValue|Webapp class instance as 
ThreadLocal value]]||>=6.0.24||6.0.24-6.0.26 but is unsafe. Made optional from 
6.0.27|| [[#48837|Detect child classloaders]], [[#49159|BZ 49159]] ||
- ||[[#webappClassInstanceAsThreadLocalIndirectValue|Webapp class instance 
indirectly held through a ThreadLocal value]]||no||no||Renew threads in the 
worker pool||
+ ||[[#webappClassInstanceAsThreadLocalIndirectValue|Webapp class instance 
indirectly held through a ThreadLocal value]]||no||no||[[#49159|Renew threads 
in the worker pool]]||
- ||[[#threadLocalPseudoLeak|ThreadLocal 
pseudo-leak]]||>=6.0.24||>=6.0.24||[[#48895|improve thread safety]]||
- ||[[#cclThreadSpawnedByWebApp|ContextClassLoader / Threads spawned by 
webapps]]||>=6.0.24||{{{TimerThread}}} are stopped. Other threads may be 
stopped if {{{clearReferencesStopThreads}}} is set to true on the Context 
||[[#48837|Detect child classloaders]], [[#48971|don't stop TimerThread by 
default]]||
- ||[[#cclThreadSpawnedByCommonClassLoader|ContextClassLoader / Threads spawned 
by classes loaded by the common classloader]]||>=6.0.24||{{{TimerThread}}} are 
stopped. Other threads may be stopped if {{{clearReferencesStopThreads}}} is 
set to true on the Context ||[[#48837|Detect child classloaders]], 
[[#48971|don't stop TimerThread by default]]||
+ ||[[#threadLocalPseudoLeak|ThreadLocal pseudo-leak]]||>=6.0.24||6.0.24-6.0.26 
but is unsafe. Made optional from 6.0.27||[[#49159|BZ 49159]]||
+ ||[[#cclThreadSpawnedByWebApp|ContextClassLoader / Threads spawned by 
webapps]]||>=6.0.24||In 6.0.24-6.0.26 {{{TimerThread}}} are stopped but it may 
lead to problems. Optional from 6.0.27 with the 
{{{clearReferencesStopTimerThreads}}} flag. Other threads may be stopped with 
the {{{clearReferencesStopThreads}}} flag, but it's unsafe. ||[[#48837|Detect 
child classloaders]], Fix the application to stop the thread when the 
application is stopped||
+ ||[[#cclThreadSpawnedByCommonClassLoader|ContextClassLoader / Threads spawned 
by classes loaded by the common classloader]]||>=6.0.24||In 6.0.24-6.0.26 
{{{TimerThread}}} are stopped but it may lead to problems. Optional from 6.0.27 
with the {{{clearReferencesStopTimerThreads}}} flag. Other threads may be 
stopped with the {{{clearReferencesStopThreads}}} flag, but it's unsafe. 
||[[#48837|Detect child classloaders]], fix the offending code (set the correct 
CCL when spawning the thread)||
  ||[[#cclThreadSpawnedByJRE|ContextClassLoader / Threads spawned by JRE 
classes]]|| no || >=6.0.24 pre-spawns some known offenders || ||
  ||[[#staticClassVariables|static class variables]]|| no ||> 6.0.? || ||
  ||[[#LogFactory|LogFactory]]||  ||> 6.0.? || ||
@@ -84, +84 @@

  
  Hopefully tomcat 6.0.24 can detect the leak when the application is stopped: 
each Thread in the JVM is examined, and the internal structures of the Thread 
and {{{ThreadLocal}}} classes are introspected to see if either the 
{{{ThreadLocal}}} instance or the value bound to it were loaded by the 
{{{WebAppClassLoader}}} of the application being stopped.
  
- In this particular case, the leak is detected, a message is logged and 
internal structures of the JDK ({{{ThreadLocalMap}}}) are modified to remove 
the reference to the {{{ThreadLocal}}} instance.
+ In this particular case, the leak is detected and a message is logged. Tomcat 
6.0.24 to 6.0.26 modify internal structures of the JDK ({{{ThreadLocalMap}}}) 
to remove the reference to the {{{ThreadLocal}}} instance, but this is unsafe 
(see [[#48895]]) so that it became optional and disabled by default from 6.0.27.
  
  {{{
  Mar 16, 2010 11:47:24 PM org.apache.catalina.loader.WebappClassLoader 
clearThreadLocalMap
@@ -145, +145 @@

  
  If the servlet is invoked at least once, the webapp classloader would not be 
GCed when the app is stopped: since the classloader of {{{ThreadScopedHolder}}} 
is the common classloader, it remains forever which is as expected. But its 
{{{ThreadLocal}}} instance has a value bound to it (for the non-terminate

svn commit: r951980 - /tomcat/trunk/java/org/apache/catalina/core/ApplicationFilterChain.java

2010-06-06 Thread markt
Author: markt
Date: Sun Jun  6 21:21:51 2010
New Revision: 951980

URL: http://svn.apache.org/viewvc?rev=951980&view=rev
Log:
Additional fix required for http://svn.apache.org/viewvc?rev=951926&view=rev

Modified:
tomcat/trunk/java/org/apache/catalina/core/ApplicationFilterChain.java

Modified: tomcat/trunk/java/org/apache/catalina/core/ApplicationFilterChain.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/ApplicationFilterChain.java?rev=951980&r1=951979&r2=951980&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/core/ApplicationFilterChain.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/core/ApplicationFilterChain.java Sun 
Jun  6 21:21:51 2010
@@ -60,7 +60,7 @@ final class ApplicationFilterChain imple
 private final static ThreadLocal lastServicedResponse;
 
 static {
-if (Globals.STRICT_SERVLET_COMPLIANCE) {
+if (ApplicationDispatcher.WRAP_SAME_OBJECT) {
 lastServicedRequest = new ThreadLocal();
 lastServicedResponse = new ThreadLocal();
 } else {
@@ -271,7 +271,7 @@ final class ApplicationFilterChain imple
 
 // We fell off the end of the chain -- call the servlet instance
 try {
-if (Globals.STRICT_SERVLET_COMPLIANCE) {
+if (ApplicationDispatcher.WRAP_SAME_OBJECT) {
 lastServicedRequest.set(request);
 lastServicedResponse.set(response);
 }
@@ -325,7 +325,7 @@ final class ApplicationFilterChain imple
 throw new ServletException
   (sm.getString("filterChain.servlet"), e);
 } finally {
-if (Globals.STRICT_SERVLET_COMPLIANCE) {
+if (ApplicationDispatcher.WRAP_SAME_OBJECT) {
 lastServicedRequest.set(null);
 lastServicedResponse.set(null);
 }



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



svn commit: r951981 - /tomcat/trunk/java/org/apache/catalina/core/StandardHostValve.java

2010-06-06 Thread markt
Author: markt
Date: Sun Jun  6 21:22:25 2010
New Revision: 951981

URL: http://svn.apache.org/viewvc?rev=951981&view=rev
Log:
Use the constant rather than re-reading

Modified:
tomcat/trunk/java/org/apache/catalina/core/StandardHostValve.java

Modified: tomcat/trunk/java/org/apache/catalina/core/StandardHostValve.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/StandardHostValve.java?rev=951981&r1=951980&r2=951981&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/core/StandardHostValve.java (original)
+++ tomcat/trunk/java/org/apache/catalina/core/StandardHostValve.java Sun Jun  
6 21:22:25 2010
@@ -66,9 +66,7 @@ final class StandardHostValve
 protected static final boolean ACCESS_SESSION;
 
 static {
-STRICT_SERVLET_COMPLIANCE = Boolean.valueOf(System.getProperty(
-"org.apache.catalina.STRICT_SERVLET_COMPLIANCE",
-"false")).booleanValue();
+STRICT_SERVLET_COMPLIANCE = Globals.STRICT_SERVLET_COMPLIANCE;
 
 String accessSession = System.getProperty(
 "org.apache.catalina.core.StandardHostValve.ACCESS_SESSION");



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



svn commit: r951985 - in /tomcat/trunk: java/org/apache/catalina/session/StandardSession.java webapps/docs/config/systemprops.xml

2010-06-06 Thread markt
Author: markt
Date: Sun Jun  6 21:24:09 2010
New Revision: 951985

URL: http://svn.apache.org/viewvc?rev=951985&view=rev
Log:
Final part of refactoring STRICT_SERVLET_COMPLIANCE so it just changes defaults
New property LAST_ACCESS_AT_START

Modified:
tomcat/trunk/java/org/apache/catalina/session/StandardSession.java
tomcat/trunk/webapps/docs/config/systemprops.xml

Modified: tomcat/trunk/java/org/apache/catalina/session/StandardSession.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/session/StandardSession.java?rev=951985&r1=951984&r2=951985&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/session/StandardSession.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/session/StandardSession.java Sun Jun  
6 21:24:09 2010
@@ -88,11 +88,10 @@ public class StandardSession
 
 protected static final boolean ACTIVITY_CHECK;
 
+protected static final boolean LAST_ACCESS_AT_START;
 
 static {
-STRICT_SERVLET_COMPLIANCE = Boolean.valueOf(System.getProperty(
-"org.apache.catalina.STRICT_SERVLET_COMPLIANCE",
-"false")).booleanValue();
+STRICT_SERVLET_COMPLIANCE = Globals.STRICT_SERVLET_COMPLIANCE;
 
 String activityCheck = System.getProperty(
 "org.apache.catalina.session.StandardSession.ACTIVITY_CHECK");
@@ -102,6 +101,15 @@ public class StandardSession
 ACTIVITY_CHECK =
 Boolean.valueOf(activityCheck).booleanValue();
 }
+
+String lastAccessAtStart = System.getProperty(
+
"org.apache.catalina.session.StandardSession.LAST_ACCESS_AT_START");
+if (lastAccessAtStart == null) {
+LAST_ACCESS_AT_START = STRICT_SERVLET_COMPLIANCE;
+} else {
+LAST_ACCESS_AT_START =
+Boolean.valueOf(lastAccessAtStart).booleanValue();
+}
 }
 
 
@@ -614,7 +622,7 @@ public class StandardSession
 if (maxInactiveInterval >= 0) { 
 long timeNow = System.currentTimeMillis();
 int timeIdle;
-if (Globals.STRICT_SERVLET_COMPLIANCE) {
+if (LAST_ACCESS_AT_START) {
 timeIdle = (int) ((timeNow - lastAccessedTime) / 1000L);
 } else {
 timeIdle = (int) ((timeNow - thisAccessedTime) / 1000L);
@@ -668,7 +676,7 @@ public class StandardSession
  * The servlet spec mandates to ignore request handling time
  * in lastAccessedTime.
  */
-if (Globals.STRICT_SERVLET_COMPLIANCE) {
+if (LAST_ACCESS_AT_START) {
 this.lastAccessedTime = this.thisAccessedTime;
 this.thisAccessedTime = System.currentTimeMillis();
 } else {

Modified: tomcat/trunk/webapps/docs/config/systemprops.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/config/systemprops.xml?rev=951985&r1=951984&r2=951985&view=diff
==
--- tomcat/trunk/webapps/docs/config/systemprops.xml (original)
+++ tomcat/trunk/webapps/docs/config/systemprops.xml Sun Jun  6 21:24:09 2010
@@ -228,6 +228,7 @@
   
org.apache.catalina.core.ApplicationDispatcher.WRAP_SAME_OBJECT
   
org.apache.catalina.core.StandardHostValve.ACCESS_SESSION
   
org.apache.catalina.session.StandardSession.ACTIVITY_CHECK
+  
org.apache.catalina.session.StandardSession.LAST_ACCESS_AT_START
   
org.apache.tomcat.util.http.ServerCookie.ALWAYS_ADD_EXPIRES.
   
org.apache.tomcat.util.http.ServerCookie.FWD_SLASH_IS_SEPARATOR.
   
org.apache.tomcat.util.http.ServerCookie.STRICT_NAMING.
@@ -368,6 +369,18 @@
   else the default value will be false.
 
 
+
+  If this is true, the last accessed time for sessions will
+  be calculated from the beginning of the previous request. If
+  false, the last accessed time for sessions will be 
calculated
+  from the end of the previous request. This also affects how the idle time
+  is calculated. If
+  org.apache.catalina.STRICT_SERVLET_COMPLIANCE is set to
+  true, the default of this setting will be true,
+  else the default value will be false.
+
+
   
 
 



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



svn commit: r951989 - /tomcat/tc6.0.x/trunk/STATUS.txt

2010-06-06 Thread markt
Author: markt
Date: Sun Jun  6 21:27:41 2010
New Revision: 951989

URL: http://svn.apache.org/viewvc?rev=951989&view=rev
Log:
Vote for better patch

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=951989&r1=951988&r2=951989&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Sun Jun  6 21:27:41 2010
@@ -165,13 +165,8 @@ PATCHES PROPOSED TO BACKPORT:
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49095
   AprEndpoint does not wakeup accepts with deferred accept or BSD filters
   Based on a patch provided by Ruediger Pluem
-  https://issues.apache.org/bugzilla/attachment.cgi?id=25266
-  +1: markt
-  -1: 
-  +0: kkolinko: I updated the above patch to use explicit encoding when
-  creating the writer:
   https://issues.apache.org/bugzilla/attachment.cgi?id=25510
-  +1: kkolinko, rjung
+  +1: kkolinko, rjung, markt
   -1:
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49343



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



svn commit: r951992 - in /tomcat/trunk/java/org/apache/catalina/ha: authenticator/ClusterSingleSignOn.java authenticator/ClusterSingleSignOnListener.java session/ClusterSessionListener.java

2010-06-06 Thread markt
Author: markt
Date: Sun Jun  6 21:31:20 2010
New Revision: 951992

URL: http://svn.apache.org/viewvc?rev=951992&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49376
Correct package names in info properties

Modified:

tomcat/trunk/java/org/apache/catalina/ha/authenticator/ClusterSingleSignOn.java

tomcat/trunk/java/org/apache/catalina/ha/authenticator/ClusterSingleSignOnListener.java
tomcat/trunk/java/org/apache/catalina/ha/session/ClusterSessionListener.java

Modified: 
tomcat/trunk/java/org/apache/catalina/ha/authenticator/ClusterSingleSignOn.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ha/authenticator/ClusterSingleSignOn.java?rev=951992&r1=951991&r2=951992&view=diff
==
--- 
tomcat/trunk/java/org/apache/catalina/ha/authenticator/ClusterSingleSignOn.java 
(original)
+++ 
tomcat/trunk/java/org/apache/catalina/ha/authenticator/ClusterSingleSignOn.java 
Sun Jun  6 21:31:20 2010
@@ -66,7 +66,7 @@ public class ClusterSingleSignOn
  * Descriptive information about this Valve implementation.
  */
 protected static String info =
-"org.apache.catalina.cluster.authenticator.ClusterSingleSignOn";
+"org.apache.catalina.ha.authenticator.ClusterSingleSignOn";
 
 protected int messageNumber = 0;
 

Modified: 
tomcat/trunk/java/org/apache/catalina/ha/authenticator/ClusterSingleSignOnListener.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ha/authenticator/ClusterSingleSignOnListener.java?rev=951992&r1=951991&r2=951992&view=diff
==
--- 
tomcat/trunk/java/org/apache/catalina/ha/authenticator/ClusterSingleSignOnListener.java
 (original)
+++ 
tomcat/trunk/java/org/apache/catalina/ha/authenticator/ClusterSingleSignOnListener.java
 Sun Jun  6 21:31:20 2010
@@ -40,7 +40,7 @@ public class ClusterSingleSignOnListener
 /**
  * The descriptive information about this implementation.
  */
-protected static final String info = 
"org.apache.catalina.session.ClusterSingleSignOnListener/1.0";
+protected static final String info = 
"org.apache.catalina.ha.authenticator.ClusterSingleSignOnListener/1.0";
 
 // - Properties
 

Modified: 
tomcat/trunk/java/org/apache/catalina/ha/session/ClusterSessionListener.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/ha/session/ClusterSessionListener.java?rev=951992&r1=951991&r2=951992&view=diff
==
--- 
tomcat/trunk/java/org/apache/catalina/ha/session/ClusterSessionListener.java 
(original)
+++ 
tomcat/trunk/java/org/apache/catalina/ha/session/ClusterSessionListener.java 
Sun Jun  6 21:31:20 2010
@@ -39,7 +39,7 @@ public class ClusterSessionListener exte
 /**
  * The descriptive information about this implementation.
  */
-protected static final String info = 
"org.apache.catalina.session.ClusterSessionListener/1.1";
+protected static final String info = 
"org.apache.catalina.ha.session.ClusterSessionListener/1.1";
 
 //--Constructor-
 



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



DO NOT REPLY [Bug 49376] Minor corrections to info properties.

2010-06-06 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49376

Mark Thomas  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED
 OS/Version||All

--- Comment #2 from Mark Thomas  2010-06-06 17:31:38 EDT ---
Thanks for the patch. It has been applied to trunk and will be in 7.0.0
onwards.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

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



svn commit: r952000 - /tomcat/trunk/java/org/apache/catalina/util/LifecycleBase.java

2010-06-06 Thread markt
Author: markt
Date: Sun Jun  6 21:50:11 2010
New Revision: 952000

URL: http://svn.apache.org/viewvc?rev=952000&view=rev
Log:
Make syncs consistent

Modified:
tomcat/trunk/java/org/apache/catalina/util/LifecycleBase.java

Modified: tomcat/trunk/java/org/apache/catalina/util/LifecycleBase.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/util/LifecycleBase.java?rev=952000&r1=951999&r2=952000&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/util/LifecycleBase.java (original)
+++ tomcat/trunk/java/org/apache/catalina/util/LifecycleBase.java Sun Jun  6 
21:50:11 2010
@@ -134,7 +134,12 @@ public abstract class LifecycleBase impl
 
 setState(LifecycleState.STARTING_PREP);
 
-startInternal();
+try {
+startInternal();
+} catch (LifecycleException e) {
+setState(LifecycleState.FAILED);
+throw e;
+}
 
 if (state.equals(LifecycleState.FAILED) ||
 state.equals(LifecycleState.MUST_STOP)) {
@@ -296,7 +301,7 @@ public abstract class LifecycleBase impl
  * @param state The new state for this component
  * @param data  The data to pass to the associated {...@link Lifecycle} 
event
  */
-protected void setState(LifecycleState state, Object data) {
+protected synchronized void setState(LifecycleState state, Object data) {
 
 if (log.isDebugEnabled()) {
 log.debug(sm.getString("lifecycleBase.setState", this, state));



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




svn commit: r952001 - /tomcat/trunk/java/org/apache/catalina/core/StandardService.java

2010-06-06 Thread markt
Author: markt
Date: Sun Jun  6 21:51:30 2010
New Revision: 952001

URL: http://svn.apache.org/viewvc?rev=952001&view=rev
Log:
Remainder of fix for https://issues.apache.org/bugzilla/show_bug.cgi?id=49372
Avoid invalid Lifecycle transitions on failed start due to already bound 
shutdown port

Modified:
tomcat/trunk/java/org/apache/catalina/core/StandardService.java

Modified: tomcat/trunk/java/org/apache/catalina/core/StandardService.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/StandardService.java?rev=952001&r1=952000&r2=952001&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/core/StandardService.java (original)
+++ tomcat/trunk/java/org/apache/catalina/core/StandardService.java Sun Jun  6 
21:51:30 2010
@@ -479,7 +479,12 @@ public class StandardService extends Lif
 // Stop our defined Connectors first
 synchronized (connectors) {
 for (int i = 0; i < connectors.length; i++) {
-((Lifecycle) connectors[i]).stop();
+// If Service fails to start, connectors may not have been
+// started
+if (!LifecycleState.INITIALIZED.equals(
+((Lifecycle) connectors[i]).getState())) {
+((Lifecycle) connectors[i]).stop();
+}
 }
 }
 



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



svn propchange: r952000 - svn:log

2010-06-06 Thread markt
Author: markt
Revision: 952000
Modified property: svn:log

Modified: svn:log at Sun Jun  6 21:52:34 2010
--
--- svn:log (original)
+++ svn:log Sun Jun  6 21:52:34 2010
@@ -1 +1,3 @@
 Make syncs consistent
+Partial fix for https://issues.apache.org/bugzilla/show_bug.cgi?id=49372
+Avoid invalid Lifecycle transitions on failed start due to already bound 
shutdown port


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



DO NOT REPLY [Bug 49372] 7.0.0RC: Starting second instance of Tomcat on the same port results in "An invalid Lifecycle transition was attempted"

2010-06-06 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49372

Mark Thomas  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

--- Comment #1 from Mark Thomas  2010-06-06 17:53:16 EDT ---
Fixed in trunk. Will be in 7.0.0 onwards

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

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



DO NOT REPLY [Bug 49395] New: manager.findLeaks : display the date when the leaking context was started

2010-06-06 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49395

   Summary: manager.findLeaks : display the date when the leaking
context was started
   Product: Tomcat 7
   Version: trunk
  Platform: Macintosh
Status: NEW
  Severity: enhancement
  Priority: P2
 Component: Catalina
AssignedTo: dev@tomcat.apache.org
ReportedBy: sylvain.laur...@gmail.com


Created an attachment (id=25535)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=25535)
patch for tomcat 6

It would be nice if the "find leaks" feature of the manager servlet provided
the date and time at which the leaking context was started.

I provide patches for tomcat 7 and 6 to do that.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

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



DO NOT REPLY [Bug 49395] manager.findLeaks : display the date when the leaking context was started

2010-06-06 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49395

--- Comment #1 from sylvain.laur...@gmail.com 2010-06-06 17:57:55 EDT ---
Created an attachment (id=25536)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=25536)
patch for tomcat 7

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

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



DO NOT REPLY [Bug 49395] manager.findLeaks : display the date when the leaking context was started

2010-06-06 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49395

Mark Thomas  changed:

   What|Removed |Added

 OS/Version||All

--- Comment #2 from Mark Thomas  2010-06-06 17:58:58 EDT ---
What is the use case for this? I can't see what problem you are trying to
solve.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

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



DO NOT REPLY [Bug 49395] manager.findLeaks : display the date when the leaking context was started

2010-06-06 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49395

--- Comment #3 from Konstantin Kolinko  2010-06-06 
19:23:55 EDT ---
Interesting, though there are some issues

1) Host.findReloadedContextMemoryLeaks is callable through JMX.
I need to test it, but I suppose that ContextLeakInfo cannot be rendered by
JConsole.

2) It looks like an API change for me.

3) I would like to see not only when the context was started, but also when it
was stopped.

4) ContextLeakInfo.java needs the Apache license header.

5) i18n.

6) As a further enhancement I think that the result returned by
StandardHost.findReloadedContextMemoryLeaks() might be sorted by contextPath
and then by startDate.

7) The documentation has to be updated: manager-howto.xml,
html-manager-howto.xml


I think that the use cases for this improvement are the following:
Knowing the times will allow someone to
a) distinguish new results from the old ones,
which is needed because the information on the leaked contexts accumulates over
time.

b) match the leaked application instance to a certain time frame in the
deployment history of that web application, its logs, GC logs. E.g. if not each
start/stop cycle exposes the leak, or if GC cannot be forced and we must wait
some time before it occurs naturally.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

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



svn commit: r952021 - /tomcat/tc6.0.x/trunk/STATUS.txt

2010-06-06 Thread markt
Author: markt
Date: Sun Jun  6 23:39:53 2010
New Revision: 952021

URL: http://svn.apache.org/viewvc?rev=952021&view=rev
Log:
Comment

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=952021&r1=952020&r2=952021&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Sun Jun  6 23:39:53 2010
@@ -95,6 +95,7 @@ PATCHES PROPOSED TO BACKPORT:
 Though you need to configure a VirtualWebappLoader to test that:
 without it the property is always a zero-length array.
Mark: Does your +1 stand including the second part of the path?
+   markt: Yes
 
   Add property "searchExternalFirst" to WebappLoader:
   If set, the external repositories will be searched before



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



svn commit: r952022 - in /tomcat/tc6.0.x/trunk: ./ java/org/apache/catalina/ha/authenticator/ java/org/apache/catalina/ha/session/ webapps/docs/

2010-06-06 Thread markt
Author: markt
Date: Sun Jun  6 23:43:17 2010
New Revision: 952022

URL: http://svn.apache.org/viewvc?rev=952022&view=rev
Log:
Fix missing message text and logging with unexpected logger issues in cluster 
listeners

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt

tomcat/tc6.0.x/trunk/java/org/apache/catalina/ha/authenticator/ClusterSingleSignOnListener.java

tomcat/tc6.0.x/trunk/java/org/apache/catalina/ha/session/ClusterSessionListener.java

tomcat/tc6.0.x/trunk/java/org/apache/catalina/ha/session/JvmRouteSessionIDBinderListener.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=952022&r1=952021&r2=952022&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Sun Jun  6 23:43:17 2010
@@ -135,12 +135,6 @@ PATCHES PROPOSED TO BACKPORT:
   +1: rjung, markt, kkolinko
   -1: 
 
-* Fix missing message text and logging with unexpected logger issues in cluster
-  listeners
-  http://people.apache.org/~markt/patches/2010-05-19-lost-messages.patch
-  +1: markt, rjung, kkolinko
-  -1: 
-
 * Fix ByteChunk.indexOf(String, ...) when the string is of length 1
   http://svn.apache.org/viewvc?rev=945231&view=rev
   +1: kkolinko, markt, rjung

Modified: 
tomcat/tc6.0.x/trunk/java/org/apache/catalina/ha/authenticator/ClusterSingleSignOnListener.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/catalina/ha/authenticator/ClusterSingleSignOnListener.java?rev=952022&r1=952021&r2=952022&view=diff
==
--- 
tomcat/tc6.0.x/trunk/java/org/apache/catalina/ha/authenticator/ClusterSingleSignOnListener.java
 (original)
+++ 
tomcat/tc6.0.x/trunk/java/org/apache/catalina/ha/authenticator/ClusterSingleSignOnListener.java
 Sun Jun  6 23:43:17 2010
@@ -24,6 +24,8 @@ import org.apache.catalina.Session;
 import org.apache.catalina.ha.ClusterManager;
 import org.apache.catalina.ha.ClusterMessage;
 import org.apache.catalina.ha.ClusterListener;
+import org.apache.juli.logging.Log;
+import org.apache.juli.logging.LogFactory;
 
 /**
  * Receive replicated SingleSignOnMessage form other cluster node.
@@ -32,6 +34,9 @@ import org.apache.catalina.ha.ClusterLis
  */
 public class ClusterSingleSignOnListener extends ClusterListener {
  
+private static final Log log =
+LogFactory.getLog(ClusterSingleSignOnListener.class);
+
 /**
  * The descriptive information about this implementation.
  */

Modified: 
tomcat/tc6.0.x/trunk/java/org/apache/catalina/ha/session/ClusterSessionListener.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/catalina/ha/session/ClusterSessionListener.java?rev=952022&r1=952021&r2=952022&view=diff
==
--- 
tomcat/tc6.0.x/trunk/java/org/apache/catalina/ha/session/ClusterSessionListener.java
 (original)
+++ 
tomcat/tc6.0.x/trunk/java/org/apache/catalina/ha/session/ClusterSessionListener.java
 Sun Jun  6 23:43:17 2010
@@ -19,9 +19,11 @@ package org.apache.catalina.ha.session;
 
 import java.util.Map;
 
+import org.apache.catalina.ha.ClusterListener;
 import org.apache.catalina.ha.ClusterManager;
 import org.apache.catalina.ha.ClusterMessage;
-import org.apache.catalina.ha.*;
+import org.apache.juli.logging.Log;
+import org.apache.juli.logging.LogFactory;
 
 /**
  * Receive replicated SessionMessage form other cluster node.
@@ -30,7 +32,10 @@ import org.apache.catalina.ha.*;
  * @version $Id$
  */
 public class ClusterSessionListener extends ClusterListener {
- 
+
+private static final Log log =
+LogFactory.getLog(ClusterSessionListener.class);
+
 /**
  * The descriptive information about this implementation.
  */

Modified: 
tomcat/tc6.0.x/trunk/java/org/apache/catalina/ha/session/JvmRouteSessionIDBinderListener.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/catalina/ha/session/JvmRouteSessionIDBinderListener.java?rev=952022&r1=952021&r2=952022&view=diff
==
--- 
tomcat/tc6.0.x/trunk/java/org/apache/catalina/ha/session/JvmRouteSessionIDBinderListener.java
 (original)
+++ 
tomcat/tc6.0.x/trunk/java/org/apache/catalina/ha/session/JvmRouteSessionIDBinderListener.java
 Sun Jun  6 23:43:17 2010
@@ -24,9 +24,12 @@ import org.apache.catalina.Context;
 import org.apache.catalina.Engine;
 import org.apache.catalina.LifecycleException;
 import org.apache.catalina.Session;
+import org.apache.catalina.ha.ClusterListener;
 import org.apache.catalina.ha.ClusterMessage;
 import org.apache.catalina.core.StandardEngine;
-import org.apache.catalina.ha.*;
+import org.apache.juli.logging.Log;
+import org.apache.juli.logging.LogFactory;
+import org.apache.tomcat

svn commit: r952023 - in /tomcat/tc6.0.x/trunk: STATUS.txt dist.xml webapps/docs/changelog.xml

2010-06-06 Thread markt
Author: markt
Date: Sun Jun  6 23:46:22 2010
New Revision: 952023

URL: http://svn.apache.org/viewvc?rev=952023&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48990
Fix the skip.installer build property so once set, the Windows installer is 
skipped but all the other artifacts are created

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt
tomcat/tc6.0.x/trunk/dist.xml
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=952023&r1=952022&r2=952023&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Sun Jun  6 23:46:22 2010
@@ -140,13 +140,6 @@ PATCHES PROPOSED TO BACKPORT:
   +1: kkolinko, markt, rjung
   -1:
 
-* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48990
-  Fix the skip.installer build property so once set, the Windows installer is
-  skipped but all the other artifacts are created
-  https://issues.apache.org/bugzilla/attachment.cgi?id=25477&action=edit
-  +1: markt, kkolinko, rjung
-  -1: 
-
 * Fix issues running AJP BIO coyote connector under a security manager
   http://svn.apache.org/viewvc?rev=947717&view=rev (less changes to 
loadUtilPackage())
   +1: markt, kkolinko, rjung

Modified: tomcat/tc6.0.x/trunk/dist.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/dist.xml?rev=952023&r1=952022&r2=952023&view=diff
==
--- tomcat/tc6.0.x/trunk/dist.xml (original)
+++ tomcat/tc6.0.x/trunk/dist.xml Sun Jun  6 23:46:22 2010
@@ -210,6 +210,28 @@
 
 
 
+   
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 
 
 
@@ -305,26 +327,6 @@
 
 
 
-
-
-
-
-
-
-
-
-
-
-
-
-
 
 
 

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=952023&r1=952022&r2=952023&view=diff
==
--- tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Sun Jun  6 23:46:22 2010
@@ -270,6 +270,10 @@
   
 Remove unused code from org.apache.tomcat.util.buf classes. (kkolinko)
   
+  
+Fix the skip.installer build property so if set, only the Windows
+installer is skipped. (markt)
+  
 
   
 



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



DO NOT REPLY [Bug 48990] Build fails on Linux when skip.installer set to true

2010-06-06 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=48990

Mark Thomas  changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution||FIXED

--- Comment #9 from Mark Thomas  2010-06-06 19:46:33 EDT ---
Fixed in 6.0.x and will be in 6.0.27 onwards

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

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



svn commit: r952026 - in /tomcat/tc6.0.x/trunk: ./ java/org/apache/catalina/security/ java/org/apache/coyote/http11/ webapps/docs/

2010-06-06 Thread markt
Author: markt
Date: Sun Jun  6 23:56:45 2010
New Revision: 952026

URL: http://svn.apache.org/viewvc?rev=952026&view=rev
Log:
Fix issues running AJP BIO coyote connector under a security manager
Additional clean-up

Modified:
tomcat/tc6.0.x/trunk/   (props changed)
tomcat/tc6.0.x/trunk/STATUS.txt

tomcat/tc6.0.x/trunk/java/org/apache/catalina/security/SecurityClassLoad.java
tomcat/tc6.0.x/trunk/java/org/apache/coyote/http11/Http11AprProcessor.java
tomcat/tc6.0.x/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java
tomcat/tc6.0.x/trunk/java/org/apache/coyote/http11/Http11Processor.java
tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc6.0.x/trunk/
--
--- svn:mergeinfo (original)
+++ svn:mergeinfo Sun Jun  6 23:56:45 2010
@@ -1 +1 @@
-/tomcat/trunk:601180,606992,612607,630314,640888,652744,653247,666232,673796,673820,677910,683969,683982,684001,684081,684234,684269-684270,685177,687503,687645,689402,690781,691392,691805,692748,693378,694992,695053,695311,696780,696782,698012,698227,698236,698613,699427,699634,701355,709294,709811,709816,710063,710066,710125,710205,711126,711600,712461,712467,713953,714002,718360,719119,719124,719602,719626,719628,720046,720069,721040,721286,721708,721886,723404,723738,726052,727303,728032,728768,728947,729057,729567,729569,729571,729681,729809,729815,729934,730250,730590,731651,732859,732863,734734,740675,740684,742677,742697,742714,744160,744238,746321,746384,746425,747834,747863,748344,750258,750291,750921,751286-751287,751289,751295,752323,753039,757335,757774,758249,758365,758596,758616,758664,759074,761601,762868,762929,762936-762937,763166,763183,763193,763228,763262,763298,763302,763325,763599,763611,763654,763681,763706,764985,764997,765662,768335,769979,770716,77
 
0809,770876,772872,776921,776924,776935,776945,777464,777466,777576,777625,778379,778523-778524,781528,781779,782145,782791,783316,783696,783724,783756,783762,783766,783863,783934,784453,784602,784614,785381,785688,785768,785859,786468,786487,786490,786496,786667,787627,787770,787985,789389,790405,791041,791184,791194,791224,791243,791326,791328,791789,792740,793372,793757,793882,793981,794082,794673,794822,795043,795152,795210,795457,795466,797168,797425,797596,797607,802727,802940,804462,804544,804734,805153,809131,809603,810916,810977,812125,812137,812432,813001,813013,813866,814180,814708,814876,815972,816252,817442,817822,819339,819361,820110,820132,820874,820954,821397,828196,828201,828210,828225,828759,830378-830379,830999,831106,831774,831785,831828,831850,831860,832214,832218,833121,833545,834047,835036,835336,836405,881396,881412,883130,883134,883146,883165,883177,883362,883565,884341,885038,885231,885241,885260,885901,885991,886019,888072,889363,889606,889716,8901
 
39,890265,890349-890350,890417,891185-891187,891583,892198,892341,892415,892464,892555,892812,892814,892817,892843,892887,893321,893493,894580,894586,894805,894831,895013,895045,895057,895191,895392,895703,896370,896384,897380-897381,897776,898126,898256,898468,898527,898555,898558,898718,898836,898906,899284,899348,899420,899653,899769-899770,899783,899788,899792,899916,899918-899919,899935,899949,903916,905020,905151,905722,905728,905735,907311,907513,907538,907652,907819,907825,907864,908002,908721,908754,908759,909097,909206,909212,909525,909636,909869,909875,909887,910266,910370,910442,910471,910485,910974,915226,915737,915861,916097,916141,916157,916170,917598,917633,918093,918489,918594,918684,918787,918792,918799,918803,918885,919851,919914,920025,920055,920298,920449,920596,920824,920840,921444,922010,926716,927062,927621,928482,928695,928732,928798,931709,932357,932967,935105,935983,939491,939551,940064,941356,941463,944409,944416,945808,945835,945841,946686,948057
 ,950164,950596,950614,951615
+/tomcat/trunk:601180,606992,612607,630314,640888,652744,653247,666232,673796,673820,677910,683969,683982,684001,684081,684234,684269-684270,685177,687503,687645,689402,690781,691392,691805,692748,693378,694992,695053,695311,696780,696782,698012,698227,698236,698613,699427,699634,701355,709294,709811,709816,710063,710066,710125,710205,711126,711600,712461,712467,713953,714002,718360,719119,719124,719602,719626,719628,720046,720069,721040,721286,721708,721886,723404,723738,726052,727303,728032,728768,728947,729057,729567,729569,729571,729681,729809,729815,729934,730250,730590,731651,732859,732863,734734,740675,740684,742677,742697,742714,744160,744238,746321,746384,746425,747834,747863,748344,750258,750291,750921,751286-751287,751289,751295,752323,753039,757335,757774,758249,758365,758596,758616,758664,759074,761601,762868,762929,762936-762937,763166,763183,763193,763228,763262,763298,763302,763325,763599,763611,763654,763681,763706,764985,764997,765662,768335,769979,770716,77
 
0809,770876,772872,776921,776924,776935,776945,777464,777466,777576,777625,778379,778523-778524,781528,781779,782145,782791,7833

svn commit: r952044 - /tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml

2010-06-06 Thread kkolinko
Author: kkolinko
Date: Mon Jun  7 01:07:14 2010
New Revision: 952044

URL: http://svn.apache.org/viewvc?rev=952044&view=rev
Log:
Add issue number
It is followup to r952023

Modified:
tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml

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=952044&r1=952043&r2=952044&view=diff
==
--- tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Mon Jun  7 01:07:14 2010
@@ -262,6 +262,10 @@
 Update to NSIS 2.46. (kkolinko)
   
   
+48990: Fix the skip.installer build property
+so if set, only the Windows installer is skipped. (markt)
+  
+  
 49178: Provide in catalina.policy an example of additional
 permissions that might be needed for code located in
 $CATALINA_BASE/lib. (markt)
@@ -273,10 +277,6 @@
   
 Remove unused code from org.apache.tomcat.util.buf classes. (kkolinko)
   
-  
-Fix the skip.installer build property so if set, only the Windows
-installer is skipped. (markt)
-  
 
   
 



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



svn commit: r952051 - /tomcat/tc6.0.x/trunk/dist.xml

2010-06-06 Thread kkolinko
Author: kkolinko
Date: Mon Jun  7 01:17:09 2010
New Revision: 952051

URL: http://svn.apache.org/viewvc?rev=952051&view=rev
Log:
replace tab with whitespace

Modified:
tomcat/tc6.0.x/trunk/dist.xml

Modified: tomcat/tc6.0.x/trunk/dist.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/dist.xml?rev=952051&r1=952050&r2=952051&view=diff
==
--- tomcat/tc6.0.x/trunk/dist.xml (original)
+++ tomcat/tc6.0.x/trunk/dist.xml Mon Jun  7 01:17:09 2010
@@ -210,7 +210,7 @@
 
 
 
-   
+
 
 



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



DO NOT REPLY [Bug 48990] Build fails on Linux when skip.installer set to true

2010-06-06 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=48990

--- Comment #10 from Konstantin Kolinko  2010-06-06 
22:24:03 EDT ---
Created an attachment (id=25537)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=25537)
2010-06-07_tc55_bug48990.patch

Looking at comment #3 and how the build file is implemented, it looks that it
is a problem for 5.5 as well.

I am attaching patch that fixes this issue in Tomcat 5.5 (backport of r952023)
AND introduces support for "skip.installer" property.

Defining the "skip.installer" property will allow to skip creating the
installer when running Tomcat 5.5 build on Windows.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

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



DO NOT REPLY [Bug 48990] Build fails on Linux when skip.installer set to true

2010-06-06 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=48990

Konstantin Kolinko  changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
  Component|Catalina|Catalina
Version|6.0.26  |5.5.28
 Resolution|FIXED   |
Product|Tomcat 6|Tomcat 5
   Target Milestone|default |---

--- Comment #11 from Konstantin Kolinko  2010-06-06 
23:50:58 EDT ---
Testing with just "skip.installer" part of the above tc5.5 patch, I should say
that I do not see any build failures.

I.e., the Unix builds of 5.5.28 should build successfully. The only thing is
that zip/tar.gz files created by such build will differ with the ones created
on Windows in that they do not include the following files and folders:

bin/tomcat5.exe
bin/tomcat5w.exe
bin/tcnative-1.dll
bin/i64/
bin/i64/tomcat5.exe
bin/i64/tcnative-1.dll
bin/x64/
bin/x64/tomcat5.exe
bin/x64/tcnative-1.dll

The effect of the above tc5.5 patch (attachment 25537) is that
A) it will align zip/tar.gz files created by building a release on Unix with
the ones on Windows.
The ones on Unix will not omit the above said exe/dll files.
B) it will provide support for "skip.installer" property


Reopening, changing Product, Version to track it in Tomcat 5.5.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

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



svn commit: r952078 - /tomcat/tc5.5.x/trunk/STATUS.txt

2010-06-06 Thread kkolinko
Author: kkolinko
Date: Mon Jun  7 03:57:01 2010
New Revision: 952078

URL: http://svn.apache.org/viewvc?rev=952078&view=rev
Log:
proposal

Modified:
tomcat/tc5.5.x/trunk/STATUS.txt

Modified: tomcat/tc5.5.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/STATUS.txt?rev=952078&r1=952077&r2=952078&view=diff
==
--- tomcat/tc5.5.x/trunk/STATUS.txt (original)
+++ tomcat/tc5.5.x/trunk/STATUS.txt Mon Jun  7 03:57:01 2010
@@ -74,3 +74,11 @@ PATCHES PROPOSED TO BACKPORT:
   https://issues.apache.org/bugzilla/attachment.cgi?id=25534
   +1: kkolinko
   -1: 
+
+* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48990#c11
+  Do not omit exe and dll files in release builds built on Unixes,
+  to align them with the official ones built on Windows.
+  Add support for "skip.installer" property.
+  https://issues.apache.org/bugzilla/attachment.cgi?id=25537
+  +1: kkolinko
+  -1:



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



DO NOT REPLY [Bug 48990] Build fails on Linux when skip.installer set to true

2010-06-06 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=48990

--- Comment #12 from Konstantin Kolinko  2010-06-06 
23:57:55 EDT ---
The patch was proposed for 5.5.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

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



Re: svn commit: r952000 - /tomcat/trunk/java/org/apache/catalina/util/LifecycleBase.java

2010-06-06 Thread Konstantin Kolinko
2010/6/7  :
> Author: markt
> Date: Sun Jun  6 21:50:11 2010
> New Revision: 952000
>
> URL: http://svn.apache.org/viewvc?rev=952000&view=rev
> Log:
> Make syncs consistent
> Partial fix for https://issues.apache.org/bugzilla/show_bug.cgi?id=49372
> Avoid invalid Lifecycle transitions on failed start due to already bound 
> shutdown port
>
> Modified:
>    tomcat/trunk/java/org/apache/catalina/util/LifecycleBase.java
>

> +++ tomcat/trunk/java/org/apache/catalina/util/LifecycleBase.java Sun Jun  6 
> 21:50:11 2010
> @@ -134,7 +134,12 @@ public abstract class LifecycleBase impl
>
>         setState(LifecycleState.STARTING_PREP);
>
> -        startInternal();
> +        try {
> +            startInternal();
> +        } catch (LifecycleException e) {
> +            setState(LifecycleState.FAILED);
> +            throw e;
> +        }
>
>         if (state.equals(LifecycleState.FAILED) ||
>                 state.equals(LifecycleState.MUST_STOP)) {

Probably RuntimeException and Error should also move it into the
FAILED state? E.g.

boolean startDone = false;
try {
  startInternal();
  startDone = true;
} finally {
  if (!startDone) {
setState(LifecycleState.FAILED);
  }
}

Best regards,
Konstantin Kolinko

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



svn commit: r952090 - in /tomcat/tc6.0.x/trunk: STATUS.txt java/org/apache/tomcat/util/net/AprEndpoint.java webapps/docs/changelog.xml

2010-06-06 Thread kkolinko
Author: kkolinko
Date: Mon Jun  7 04:34:08 2010
New Revision: 952090

URL: http://svn.apache.org/viewvc?rev=952090&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49095
AprEndpoint does not wakeup accepts with deferred accept or BSD filters
Based on a patch provided by Ruediger Pluem

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=952090&r1=952089&r2=952090&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Mon Jun  7 04:34:08 2010
@@ -140,13 +140,6 @@ PATCHES PROPOSED TO BACKPORT:
   +1: kkolinko, markt, rjung
   -1:
 
-* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49095
-  AprEndpoint does not wakeup accepts with deferred accept or BSD filters
-  Based on a patch provided by Ruediger Pluem
-  https://issues.apache.org/bugzilla/attachment.cgi?id=25510
-  +1: kkolinko, rjung, markt
-  -1:
-
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49343
   When ChannelException is thrown, remove listener from channel.
   http://svn.apache.org/viewvc?view=revision&revision=948402

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=952090&r1=952089&r2=952090&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 Mon 
Jun  7 04:34:08 2010
@@ -17,6 +17,7 @@
 
 package org.apache.tomcat.util.net;
 
+import java.io.OutputStreamWriter;
 import java.net.InetAddress;
 import java.net.InetSocketAddress;
 import java.util.ArrayList;
@@ -906,6 +907,19 @@ public class AprEndpoint {
 log.debug("About to unlock socket for: " + saddr);
 }
 s.connect(saddr, unlockTimeout);
+/*
+ * In the case of a deferred accept / accept filters we need to
+ * send data to wake up the accept. Send OPTIONS * to bypass even
+ * BSD accept filters. The Acceptor will discard it.
+ */
+if (deferAccept) {
+OutputStreamWriter sw;
+
+sw = new OutputStreamWriter(s.getOutputStream(), "ISO-8859-1");
+sw.write("OPTIONS * HTTP/1.0\r\n"
++ "User-Agent: Tomcat wakeup connection\r\n\r\n");
+sw.flush();
+}
 } catch(Exception e) {
 if (log.isDebugEnabled()) {
 log.debug(sm.getString("endpoint.debug.unlock", "" + port), e);
@@ -1153,6 +1167,15 @@ public class AprEndpoint {
 try {
 // Accept the next incoming connection from the server 
socket
 long socket = Socket.accept(serverSock);
+/*
+ * In the case of a deferred accept unlockAccept needs to
+ * send data. This data will be rubbish, so destroy the
+ * socket and don't process it.
+ */
+if (deferAccept && (paused || !running)) {
+Socket.destroy(socket);
+continue;
+}
 // Hand this socket off to an appropriate processor
 if (!processSocketWithOptions(socket)) {
 // Close socket and pool right away

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=952090&r1=952089&r2=952090&view=diff
==
--- tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Mon Jun  7 04:34:08 2010
@@ -166,6 +166,11 @@
 48917: Correct name of mod_jk module in ApacheConfig. Patch
 provided by Todd Hicks. (markt)
   
+  
+49095: AprEndpoint did not wakeup acceptors during shutdown
+when deferAccept option was enabled. Based on a patch provided by
+Ruediger Pluem. (kkolinko)
+  
   
 Use chunked encoding for http 1.1 requests with no content-length
 (regardless of keep-alive) so client can differentiate between complete



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



DO NOT REPLY [Bug 49095] org.apache.tomcat.util.net.AprEndpoint.unlockAccept does not wakeup accepts with deferred accept or BSD filters

2010-06-06 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=49095

Konstantin Kolinko  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

--- Comment #3 from Konstantin Kolinko  2010-06-07 
01:11:05 EDT ---
Fixed in 6.0 in r952090 and will be in 6.0.27 onwards.



Note, that all the affected lines are guarded by if(deferAccept) clauses.
It is easy to remove that clause and extend this implementation of
unlockAccept() that sends a valid "OPTIONS" request upon other connector
implementations. Is it worth doing?


Regarding the 'BSD filters': apparently the documentation on them is here: [1],
[2]

[1] http://www.freebsd.org/cgi/man.cgi?query=accept_filter&sektion=9
[2] http://www.freebsd.org/cgi/man.cgi?query=setsockopt&sektion=2  (see
SO_ACCEPTFILTER)

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

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