Bug report for Tomcat 5 [2012/07/29]

2012-07-29 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  |
| |   |   |  |  |
|52579|New|Reg|2012-02-02|Tomcat5.5.35?Java1.5 cannot return proper value of|
|53050|New|Nor|2012-04-09|org.apache.catalina.session.ManagerBase has issues|
+-+---+---+--+--+
| Total2 bugs   |
+---+

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



Bug report for Tomcat 7 [2012/07/29]

2012-07-29 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  |
| |   |   |  |  |
|16579|New|Enh|2003-01-30|documentation page layout/style breaks wrapping to|
|18500|New|Enh|2003-03-30|Host aliases to match by regular expression   |
|28039|Opn|Enh|2004-03-30|Cluster Support for SingleSignOn  |
|40728|Inf|Enh|2006-10-11|Catalina MBeans use non-serializable classes  |
|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|
|43866|New|Enh|2007-11-14|add support for session attribute propagation with|
|43925|Opn|Enh|2007-11-21|org.apache.jasper.runtime.BodyContentImpl causing |
|44216|New|Enh|2008-01-11|Don't reuse session ID even if emptySessionPath=tr|
|48550|Inf|Enh|2010-01-14|Update examples and default server.xml to use UTF-|
|49395|New|Enh|2010-06-06|manager.findLeaks : display the date when the leak|
|49589|New|Enh|2010-07-12|Tag handlers with constant attribute values are al|
|49785|New|Enh|2010-08-19|Enabling TLS for JNDIRealm|
|49821|New|Enh|2010-08-25|Tomcat CLI|
|50019|New|Enh|2010-09-28|Adding JNDI lookup-name support In XML and Resou|
|50175|New|Enh|2010-10-28|Enhance memory leak detection by selectively apply|
|50234|New|Enh|2010-11-08|JspC use servlet 3.0 features |
|50504|New|Enh|2010-12-21|Allow setting query string character set trough re|
|50670|New|Enh|2011-01-27|Tribes | RpcChannel | Add option to specify extern|
|51195|New|Enh|2011-05-13|Find leaks reports a false positive memory/class|
|51294|Opn|Enh|2011-05-30|Since 7.0.12 do not work option unpackWARs=true fo|
|51423|Inf|Enh|2011-06-23|[Patch] to add a path and a version parameters to |
|51463|New|Enh|2011-07-01|Tomcat.setBaseDir  (package org.apache.catalina.st|
|51496|New|Enh|2011-07-11|NSIS - Warn that duplicate service name will resul|
|51497|New|Enh|2011-07-11|Use canonical IPv6 text representation in logs|
|51526|New|Enh|2011-07-18|Process web application context config with embedd|
|51587|New|Enh|2011-07-29|Implement status and uptime commands  |
|51953|New|Enh|2011-10-04|Proposal: netmask filtering valve and filter  |
|52092|New|Enh|2011-10-26|Please make AsyncFileHandler and OneLineFormatter |
|52235|New|Enh|2011-11-23|Please do a bit of SEO tuning for the web site|
|52236|New|Enh|2011-11-23|Idea: support 'overlays' shaped like Maven overlay|
|52323|New|Enh|2011-12-13|Cobertura test code coverage support for build.xml|
|52381|New|Enh|2011-12-22|Please add OSGi metadata  |
|52448|New|Enh|2012-01-11|Cache jar indexes in WebappClassLoader to speed up|
|52489|New|Enh|2012-01-19|Enhancement request for code signing of war files |
|52558|New|Enh|2012-01-30|CometConnectionManagerValve is adding non-serializ|
|52688|New|Enh|2012-02-16|Add ability to remove old access log files|
|52751|Opn|Enh|2012-02-23|Optimized configuration of the system info display|
|52777|New|Enh|2012-02-27|Automatically shut down old versions in parallel d|
|52952|New|Enh|2012-03-20|Improve ExtensionValidator handling for embedded s|
|53085|New|Enh|2012-04-16|[perf] [concurrency] DefaultInstanceManager.annota|
|53387|New|Enh|2012-06-08|SSI: Allow to use $1 to get result of regular expr|
|53411|Opn|Enh|2012-06-13|NullPointerException in org.apache.tomcat.util.buf|
|53469|Opn|Nor|2012-06-26|possible bug in Response.normalize(CharChunk cc)  |
|53492|New|Enh|2012-07-01|Make JspC shell multithreaded |
|53535|New|Nor|2012-07-11|Out of memory while performing ContextConfig.webCo|
|53541|New|Maj|2012-07-12|Bug in virtual webapp feature. Custom tags badly s|
|53545|New|Maj|2012-07-13|jsp:forward action to static resources causes Ille|
|53553|New|Enh|2012-07-16|[PATCH] Deploy uploaded WAR with context.xml from |
|53574|New|Nor|2012-07-20|metadata-complete=true and servlet defined as js|
|53602|New|Enh|2012-07-25|Support for HTTP status code 451  |

Bug report for Tomcat 6 [2012/07/29]

2012-07-29 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  |
| |   |   |  |  |
|41679|New|Enh|2007-02-22|SemaphoreValve should be able to filter on url pat|
|41883|Ass|Enh|2007-03-18|use abstract wrapper instead of plain X509Certific|
|43001|New|Enh|2007-07-30|JspC lacks setMappedFile and setDie for use in Ant|
|43400|New|Enh|2007-09-14|enum support for tag libs |
|43548|Opn|Enh|2007-10-04|xml schema for tomcat-users.xml   |
|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|
|43979|New|Enh|2007-11-27|Add abstraction for Java and Classfile output |
|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|
|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 |
|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|
|45428|New|Enh|2008-07-18|warn if the tomcat stop doesn't complete  |
|45832|New|Enh|2008-09-18|add DIGEST authentication support to Ant tasks|
|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|
|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.xml does not update do|
|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|
|46727|New|Enh|2009-02-17|DefaultServlet - serving multiple encodings   |
|46902|New|Enh|2009-03-24|LoginValve to bypass restrictions of j_security_ch|
|47214|New|Enh|2009-05-17|Inner classes that are explicitly referenced - sho|
|47242|New|Enh|2009-05-22|request for AJP command line client   |
|47281|New|Enh|2009-05-28|Efficiency of the JDBCStore   |
|47407|New|Enh|2009-06-23|HttpSessionListener doesn't operate in the session|
|47467|New|Enh|2009-07-02|Deployment of the war file by URL when contextpath|
|47785|Opn|Enh|2009-09-04|Cluster MBean not registered  |
|47834|New|Enh|2009-09-14|TldConfig throws Exception when exploring unpacked|
|47919|New|Enh|2009-09-30|Log Tomcat  Java environment variables in additio|
|48358|Opn|Enh|2009-12-09|JSP-unloading reloaded|
|48543|New|Enh|2010-01-14|[Patch] More flexibility in specifying -Dcatalina.|
|48672|New|Enh|2010-02-03|Tomcat Virtual Host Manager (/host-manager) have b|
|48674|New|Enh|2010-02-03|Tomcat Virtual Host Manager application doesn't pe|
|48743|New|Enh|2010-02-15|Make the SLEEP variable in catalina.sh settable fr|
|48899|New|Enh|2010-03-12|Guess URI charset should solve lot of problems|
|48922|New|Enh|2010-03-16|org.apache.catalina.connector.Request clone static|
|48928|New|Enh|2010-03-17|An alternative solution to preloading classes when|
|49161|New|Enh|2010-04-21|Unknown Publisher when installing tomcat 6.0.26   |
|49176|Opn|Enh|2010-04-23|Jasper in Dev Mode Is Memory Inefficient  |
|49464|New|Enh|2010-06-18|DefaultServlet and CharacterEncoding  |
|49531|New|Enh|2010-06-30|singlesignon failover not working on DeltaManager/|
|49804|New|Enh|2010-08-23|Allow Embedded.redirectStreams value to be configu|
|49939|New|Enh|2010-09-16|Expose a method via JMX which empties the webapp f|
|49943|New|Enh|2010-09-16|Logging (via juli) does not reread configuration c|

Bug report for Taglibs [2012/07/29]

2012-07-29 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  |
| |   |   |  |  |
|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|Enh|2007-05-14|[PATCH] Log Taglib enhancements   |
|46052|New|Nor|2008-10-21|SetLocaleSupport is slow to initialize when many l|
|48333|New|Enh|2009-12-02|TLD generator |
+-+---+---+--+--+
| Total5 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 [2012/07/29]

2012-07-29 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|
|43303|New|Enh|2007-09-04|Versioning under Windows not reported by many conn|
|43968|Inf|Enh|2007-11-26|[patch] support ipv6 with mod_jk  |
|44290|Inf|Nor|2008-01-24|mod_jk/1.2.26: retry is not useful for an importan|
|44349|Inf|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|
|46337|New|Nor|2008-12-04|real worker name is wrong |
|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|
|47327|New|Enh|2009-06-07|remote_user not logged in apache logfile  |
|47617|Inf|Enh|2009-07-31|include time spent doing ajp_get_endpoint() in err|
|47678|New|Cri|2009-08-11|Unable to allocate shared memory when using isapi_|
|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|
|48460|New|Nor|2009-12-30|mod_proxy_ajp document has three misleading portio|
|48490|New|Nor|2010-01-05|Changing a node to stopped in uriworkermap.propert|
|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|
|48891|Opn|Enh|2010-03-11|Missing EOL-style settings in tomcat/jk/trunk |
|49035|New|Maj|2010-04-01|data lost when post a multipart/form-data form|
|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 |
|49469|New|Enh|2010-06-19|Workers status page has negative number of connect|
|49732|Opn|Nor|2010-08-10|reply_timeout can't wait forever. |
|49822|New|Enh|2010-08-25|Add hash lb worker method |
|49903|New|Enh|2010-09-09|Make workers file reloadable  |
|50186|New|Nor|2010-10-31|Wrong documentation of connection_pool_timeout / c|
|52334|New|Maj|2011-12-14|recover_time is not properly used |
|52483|New|Enh|2012-01-18|Print JkOptions's options in log file and jkstatus|
|52651|New|Nor|2012-02-13|JKSHMFile size limitation |
|53324|Opn|Nor|2012-05-30|Starting with mod_jk 1.2.35 I cannot modify worker|
|53542|New|Min|2012-07-13|Spelling mistake on 503 service unavailable page  |
+-+---+---+--+--+
| Total   41 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 [2012/07/29]

2012-07-29 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  |
| |   |   |  |  |
|45392|New|Nor|2008-07-14|No OCSP support for client SSL verification   |
|46179|Opn|Maj|2008-11-10|apr ssl client authentication |
|48655|Inf|Nor|2010-02-02|Active multipart downloads prevent tomcat shutdown|
|49038|Inf|Nor|2010-04-02|Crash in tcnative |
|51655|New|Nor|2011-08-12|Index page does not say what native does  |
|51813|New|Cri|2011-09-14|Tomcat randomly crashes with [libtcnative-1.so.1+0|
|52153|New|Maj|2011-11-08|periodic JVM crash (access violation) on buffer fl|
|52231|New|Nor|2011-11-23|Ant Tasks need to reflect changes in manager comma|
|52319|New|Maj|2011-12-12|Tomcat 6 crashes with [libapr-1.so.0+0x196da]  sig|
|52627|New|Min|2012-02-08|Segmentation fault in org.apache.tomcat.jni.File.i|
|52856|New|Maj|2012-03-08|High CPU load when the underlying socket error in |
|53110|New|Cri|2012-04-20|Access Violation Error while creating SHM |
|53481|New|Nor|2012-06-28|Support SSL_OP_CIPHER_SERVER_PREFERENCE / SSLHonor|
|53605|New|Nor|2012-07-26|use tcnative-1.1.24 Tomcat shutdown still crash   |
+-+---+---+--+--+
| Total   14 bugs   |
+---+

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



Bug report for Tomcat Modules [2012/07/29]

2012-07-29 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  |
|48861|New|Nor|2010-03-04|Files without AL headers  |
|49685|New|Nor|2010-08-02|Unsafe synchronization in class ManagedBean   |
|49686|New|Nor|2010-08-02|Using an instance lock to protect static shared da|
|50571|Inf|Nor|2011-01-11|Tomcat 7 JDBC connection pool exception enhancemen|
|51595|Inf|Nor|2011-08-01|org.apache.tomcat.jdbc.pool.jmx.ConnectionPool sho|
|51879|Inf|Enh|2011-09-22|Improve access to Native Connection Methods   |
|52024|Inf|Enh|2011-10-13|Custom interceptor to support automatic failover o|
|52318|Inf|Cri|2011-12-11|Version in tomcat-jdbc POM is conflicted with Vers|
|53088|Opn|Min|2012-04-17|Give PoolCleaner TimerTask a better name  |
|53198|New|Cri|2012-05-07|'driverClassName' Data Source Property Being Manda|
|53199|Inf|Enh|2012-05-07|Refactor ConnectionPool to use ScheduledExecutorSe|
|53200|New|Enh|2012-05-07|Be able to use SlowQueryReport without reporting f|
|53565|New|Min|2012-07-18|deprecated|
+-+---+---+--+--+
| Total   15 bugs   |
+---+

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



Re: tomcat 7.0.29 startup time

2012-07-29 Thread Mark Struberg
Hi Mark,

thanks for the clarifications, highly appreciated!

As far as the empty absolute-ordering/ goes: After reading the spec again 
I've been down to that route as well. So far it didn't work out. Maybe I've 
just done something wrong - will revisit and try again. Should the 
absolute-ordering/ only affect the jars with web-fragments, or does it 
disable scanning of all jars then?



I also discovered another possible impact:

Our scenario is to use 1 tomcat installation as 'quasi EAR' container. We use 
the shared.loader in conf/catalina.properties and set it to our own 
${catalina.home}/applib directory which contains all our shared libraries 
(myfaces, openwebbeans, openjpa, ...).
For what I did understand by reading the servlet-3.0 spec is that only 
fragments and classes in either WEB-INF/classes or WEB-INF/lib/*.jar shall get 
scanned by tomcat, right? But it seems that also all our shared.loader jars 
will get scanned as well. I have not explicitly debugged thru, but from the 
startup times I see no other explanation as our WARs contain almost no jars.

If I set the jarToSkip=*.jar then the boot time is back to normal.


Is there an explanation in the servlet spec, or does tomcat scan a bit too much 
yet?


txs and LieGrue,
strub



- Original Message -
 From: Mark Thomas ma...@apache.org
 To: Tomcat Developers List dev@tomcat.apache.org
 Cc: 
 Sent: Saturday, July 28, 2012 2:36 AM
 Subject: Re: tomcat 7.0.29 startup time
 
 On 28/07/2012 00:25, Mark Thomas wrote:
  On 25/07/2012 17:00, Mark Struberg wrote:
  Hi Lords and Ladies!
 
  I'm currently wrangling with a doubled boot time on tomcat7.0.29 in
  comparison to 7.0.28 (12 webapps in my tc: 7.0.28  45s, 7.0.29 
  90s).
 
  I'm aware that 7.0.29 now does the scanning for
  ServletContainerInitializer even if version=2.5 is specified. But
  there shall no class scanning be performed if
  metadata-complete=true is set, right?
 
  Wrong. I don't like this but the intent of the Servlet 3.0 EG was:
  - ServletContainerInitializer cannot be disabled
  - If a ServletContainerInitializer is found, then class-scanning will
  take place
 
  Any ideas how we can ease the pain quickly?
 
  The only option I see is a custom (non-spec compliant) Tomcat specific
  feature that disables all of this.
 
 Ah. See the latest developments on
 http://java.net/jira/browse/SERVLET_SPEC-36
 
 Using an absolute ordering that specifies no fragments along with
 metadata-complete=true should do the trick.
 
 Mark
 
 
 -
 To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
 For additional commands, e-mail: dev-h...@tomcat.apache.org
 

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



svn commit: r1366822 - /tomcat/tc7.0.x/trunk/java/org/apache/catalina/connector/Response.java

2012-07-29 Thread kkolinko
Author: kkolinko
Date: Sun Jul 29 12:05:55 2012
New Revision: 1366822

URL: http://svn.apache.org/viewvc?rev=1366822view=rev
Log:
Add @Deprecate annotation, aligning code with javadoc, as prompted by Eclipse.
It is followup to r1360851.

Modified:
tomcat/tc7.0.x/trunk/java/org/apache/catalina/connector/Response.java

Modified: tomcat/tc7.0.x/trunk/java/org/apache/catalina/connector/Response.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/catalina/connector/Response.java?rev=1366822r1=1366821r2=1366822view=diff
==
--- tomcat/tc7.0.x/trunk/java/org/apache/catalina/connector/Response.java 
(original)
+++ tomcat/tc7.0.x/trunk/java/org/apache/catalina/connector/Response.java Sun 
Jul 29 12:05:55 2012
@@ -128,6 +128,7 @@ public class Response
  * Associated Catalina connector.
  * @deprecated  Unused
  */
+@Deprecated
 protected Connector connector;
 
 /**



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



Re: tomcat 7.0.29 startup time

2012-07-29 Thread Mark Struberg
tried the empty absolute ordering again and it doesn't seem to help. I fear 
that @HandlesTypes again has higher precedence than the web-fragment 
ordering/disabling.

LieGrue,
strub



- Original Message -
 From: Mark Struberg strub...@yahoo.de
 To: Tomcat Developers List dev@tomcat.apache.org
 Cc: 
 Sent: Sunday, July 29, 2012 1:14 PM
 Subject: Re: tomcat 7.0.29 startup time
 
 Hi Mark,
 
 thanks for the clarifications, highly appreciated!
 
 As far as the empty absolute-ordering/ goes: After reading the spec 
 again I've been down to that route as well. So far it didn't work out. 
 Maybe I've just done something wrong - will revisit and try again. Should 
 the absolute-ordering/ only affect the jars with web-fragments, or does 
 it disable scanning of all jars then?
 
 
 
 I also discovered another possible impact:
 
 Our scenario is to use 1 tomcat installation as 'quasi EAR' container. 
 We use the shared.loader in conf/catalina.properties and set it to our own 
 ${catalina.home}/applib directory which contains all our shared libraries 
 (myfaces, openwebbeans, openjpa, ...).
 For what I did understand by reading the servlet-3.0 spec is that only 
 fragments 
 and classes in either WEB-INF/classes or WEB-INF/lib/*.jar shall get scanned 
 by 
 tomcat, right? But it seems that also all our shared.loader jars will get 
 scanned as well. I have not explicitly debugged thru, but from the startup 
 times 
 I see no other explanation as our WARs contain almost no jars.
 
 If I set the jarToSkip=*.jar then the boot time is back to normal.
 
 
 Is there an explanation in the servlet spec, or does tomcat scan a bit too 
 much 
 yet?
 
 
 txs and LieGrue,
 strub
 
 
 
 - Original Message -
  From: Mark Thomas ma...@apache.org
  To: Tomcat Developers List dev@tomcat.apache.org
  Cc: 
  Sent: Saturday, July 28, 2012 2:36 AM
  Subject: Re: tomcat 7.0.29 startup time
 
  On 28/07/2012 00:25, Mark Thomas wrote:
   On 25/07/2012 17:00, Mark Struberg wrote:
   Hi Lords and Ladies!
 
   I'm currently wrangling with a doubled boot time on 
 tomcat7.0.29 in
   comparison to 7.0.28 (12 webapps in my tc: 7.0.28  45s, 7.0.29 
 
   90s).
 
   I'm aware that 7.0.29 now does the scanning for
   ServletContainerInitializer even if version=2.5 is specified. But
   there shall no class scanning be performed if
   metadata-complete=true is set, right?
 
   Wrong. I don't like this but the intent of the Servlet 3.0 EG was:
   - ServletContainerInitializer cannot be disabled
   - If a ServletContainerInitializer is found, then class-scanning will
   take place
 
   Any ideas how we can ease the pain quickly?
 
   The only option I see is a custom (non-spec compliant) Tomcat specific
   feature that disables all of this.
 
  Ah. See the latest developments on
  http://java.net/jira/browse/SERVLET_SPEC-36
 
  Using an absolute ordering that specifies no fragments along with
  metadata-complete=true should do the trick.
 
  Mark
 
 
  -
  To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
  For additional commands, e-mail: dev-h...@tomcat.apache.org
 
 
 -
 To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
 For additional commands, e-mail: dev-h...@tomcat.apache.org


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



Re: tomcat 7.0.29 startup time

2012-07-29 Thread Mark Thomas
Mark Struberg strub...@yahoo.de wrote:

tried the empty absolute ordering again and it doesn't seem to help. I
fear that @HandlesTypes again has higher precedence than the
web-fragment ordering/disabling.

The 'clarification' from the EG is post the last release. I'm pretty sure some 
code changes are required. Feel free to open a bz issue so this isn't forgotten.

Mark


LieGrue,
strub



- Original Message -
 From: Mark Struberg strub...@yahoo.de
 To: Tomcat Developers List dev@tomcat.apache.org
 Cc: 
 Sent: Sunday, July 29, 2012 1:14 PM
 Subject: Re: tomcat 7.0.29 startup time
 
 Hi Mark,
 
 thanks for the clarifications, highly appreciated!
 
 As far as the empty absolute-ordering/ goes: After reading the spec

 again I've been down to that route as well. So far it didn't work
out. 
 Maybe I've just done something wrong - will revisit and try again.
Should 
 the absolute-ordering/ only affect the jars with web-fragments, or
does 
 it disable scanning of all jars then?
 
 
 
 I also discovered another possible impact:
 
 Our scenario is to use 1 tomcat installation as 'quasi EAR'
container. 
 We use the shared.loader in conf/catalina.properties and set it to
our own 
 ${catalina.home}/applib directory which contains all our shared
libraries 
 (myfaces, openwebbeans, openjpa, ...).
 For what I did understand by reading the servlet-3.0 spec is that
only fragments 
 and classes in either WEB-INF/classes or WEB-INF/lib/*.jar shall get
scanned by 
 tomcat, right? But it seems that also all our shared.loader jars will
get 
 scanned as well. I have not explicitly debugged thru, but from the
startup times 
 I see no other explanation as our WARs contain almost no jars.
 
 If I set the jarToSkip=*.jar then the boot time is back to normal.
 
 
 Is there an explanation in the servlet spec, or does tomcat scan a
bit too much 
 yet?
 
 
 txs and LieGrue,
 strub
 
 
 
 - Original Message -
  From: Mark Thomas ma...@apache.org
  To: Tomcat Developers List dev@tomcat.apache.org
  Cc: 
  Sent: Saturday, July 28, 2012 2:36 AM
  Subject: Re: tomcat 7.0.29 startup time
 
  On 28/07/2012 00:25, Mark Thomas wrote:
   On 25/07/2012 17:00, Mark Struberg wrote:
   Hi Lords and Ladies!
 
   I'm currently wrangling with a doubled boot time on 
 tomcat7.0.29 in
   comparison to 7.0.28 (12 webapps in my tc: 7.0.28  45s, 7.0.29 
 
   90s).
 
   I'm aware that 7.0.29 now does the scanning for
   ServletContainerInitializer even if version=2.5 is specified.
But
   there shall no class scanning be performed if
   metadata-complete=true is set, right?
 
   Wrong. I don't like this but the intent of the Servlet 3.0 EG
was:
   - ServletContainerInitializer cannot be disabled
   - If a ServletContainerInitializer is found, then class-scanning
will
   take place
 
   Any ideas how we can ease the pain quickly?
 
   The only option I see is a custom (non-spec compliant) Tomcat
specific
   feature that disables all of this.
 
  Ah. See the latest developments on
  http://java.net/jira/browse/SERVLET_SPEC-36
 
  Using an absolute ordering that specifies no fragments along with
  metadata-complete=true should do the trick.
 
  Mark
 
 
 
-
  To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
  For additional commands, e-mail: dev-h...@tomcat.apache.org
 
 
 -
 To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
 For additional commands, e-mail: dev-h...@tomcat.apache.org


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



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



svn commit: r1366885 - in /tomcat/trunk/java/org/apache/catalina/tribes/io: BufferPool.java BufferPool15Impl.java DirectByteArrayOutputStream.java XByteBuffer.java

2012-07-29 Thread markt
Author: markt
Date: Sun Jul 29 18:26:51 2012
New Revision: 1366885

URL: http://svn.apache.org/viewvc?rev=1366885view=rev
Log:
Partial code clean-up for o.a.catalina.tribes
 - Add final prompted by UCDetector
 - Java 7 use of 

Modified:
tomcat/trunk/java/org/apache/catalina/tribes/io/BufferPool.java
tomcat/trunk/java/org/apache/catalina/tribes/io/BufferPool15Impl.java

tomcat/trunk/java/org/apache/catalina/tribes/io/DirectByteArrayOutputStream.java
tomcat/trunk/java/org/apache/catalina/tribes/io/XByteBuffer.java

Modified: tomcat/trunk/java/org/apache/catalina/tribes/io/BufferPool.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/io/BufferPool.java?rev=1366885r1=1366884r2=1366885view=diff
==
--- tomcat/trunk/java/org/apache/catalina/tribes/io/BufferPool.java (original)
+++ tomcat/trunk/java/org/apache/catalina/tribes/io/BufferPool.java Sun Jul 29 
18:26:51 2012
@@ -34,7 +34,7 @@ public class BufferPool {
 
 
 protected static volatile BufferPool instance = null;
-protected BufferPoolAPI pool = null;
+protected final BufferPoolAPI pool;
 
 private BufferPool(BufferPoolAPI pool) {
 this.pool = pool;

Modified: tomcat/trunk/java/org/apache/catalina/tribes/io/BufferPool15Impl.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/io/BufferPool15Impl.java?rev=1366885r1=1366884r2=1366885view=diff
==
--- tomcat/trunk/java/org/apache/catalina/tribes/io/BufferPool15Impl.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/tribes/io/BufferPool15Impl.java Sun 
Jul 29 18:26:51 2012
@@ -26,8 +26,9 @@ import java.util.concurrent.atomic.Atomi
  */
 class BufferPool15Impl implements BufferPool.BufferPoolAPI {
 protected int maxSize;
-protected AtomicInteger size = new AtomicInteger(0);
-protected ConcurrentLinkedQueueXByteBuffer queue = new 
ConcurrentLinkedQueueXByteBuffer();
+protected final AtomicInteger size = new AtomicInteger(0);
+protected final ConcurrentLinkedQueueXByteBuffer queue =
+new ConcurrentLinkedQueue();
 
 @Override
 public void setMaxSize(int bytes) {

Modified: 
tomcat/trunk/java/org/apache/catalina/tribes/io/DirectByteArrayOutputStream.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/io/DirectByteArrayOutputStream.java?rev=1366885r1=1366884r2=1366885view=diff
==
--- 
tomcat/trunk/java/org/apache/catalina/tribes/io/DirectByteArrayOutputStream.java
 (original)
+++ 
tomcat/trunk/java/org/apache/catalina/tribes/io/DirectByteArrayOutputStream.java
 Sun Jul 29 18:26:51 2012
@@ -28,7 +28,7 @@ import java.io.OutputStream;
  */
 public class DirectByteArrayOutputStream extends OutputStream {
 
-private XByteBuffer buffer;
+private final XByteBuffer buffer;
 
 public DirectByteArrayOutputStream(int size) {
 buffer = new XByteBuffer(size,false);

Modified: tomcat/trunk/java/org/apache/catalina/tribes/io/XByteBuffer.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/io/XByteBuffer.java?rev=1366885r1=1366884r2=1366885view=diff
==
--- tomcat/trunk/java/org/apache/catalina/tribes/io/XByteBuffer.java (original)
+++ tomcat/trunk/java/org/apache/catalina/tribes/io/XByteBuffer.java Sun Jul 29 
18:26:51 2012
@@ -554,7 +554,7 @@ public class XByteBuffer
 return deserialize(data,offset,length,null);
 }
 
-private static AtomicInteger invokecount = new AtomicInteger(0);
+private static final AtomicInteger invokecount = new AtomicInteger(0);
 
 public static Serializable deserialize(byte[] data, int offset, int 
length, ClassLoader[] cls)
 throws IOException, ClassNotFoundException, ClassCastException {



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



svn commit: r1366889 - /tomcat/trunk/java/org/apache/catalina/tribes/io/XByteBuffer.java

2012-07-29 Thread markt
Author: markt
Date: Sun Jul 29 19:03:46 2012
New Revision: 1366889

URL: http://svn.apache.org/viewvc?rev=1366889view=rev
Log:
Remove deprecated code

Modified:
tomcat/trunk/java/org/apache/catalina/tribes/io/XByteBuffer.java

Modified: tomcat/trunk/java/org/apache/catalina/tribes/io/XByteBuffer.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/io/XByteBuffer.java?rev=1366889r1=1366888r2=1366889view=diff
==
--- tomcat/trunk/java/org/apache/catalina/tribes/io/XByteBuffer.java (original)
+++ tomcat/trunk/java/org/apache/catalina/tribes/io/XByteBuffer.java Sun Jul 29 
19:03:46 2012
@@ -422,15 +422,7 @@ public class XByteBuffer
  * Converts a boolean to a 1-byte array
  * @param bool - the integer
  * @return - 1-byte array
- * @deprecated use toBytes(boolean,byte[],int)
  */
-@Deprecated
-public static byte[] toBytes(boolean bool) {
-byte[] b = new byte[1] ;
-return toBytes(bool,b,0);
-
-}
-
 public static byte[] toBytes(boolean bool, byte[] data, int offset) {
 data[offset] = (byte)(bool?1:0);
 return data;
@@ -451,13 +443,7 @@ public class XByteBuffer
  * Converts an integer to four bytes
  * @param n - the integer
  * @return - four bytes in an array
- * @deprecated use toBytes(int,byte[],int)
  */
-@Deprecated
-public static byte[] toBytes(int n) {
-return toBytes(n,new byte[4],0);
-}
-
 public static byte[] toBytes(int n,byte[] b, int offset) {
 b[offset+3] = (byte) (n);
 n = 8;
@@ -473,12 +459,7 @@ public class XByteBuffer
  * Converts an long to eight bytes
  * @param n - the long
  * @return - eight bytes in an array
- * @deprecated use toBytes(long,byte[],int)
  */
-@Deprecated
-public static byte[] toBytes(long n) {
-return toBytes(n,new byte[8],0);
-}
 public static byte[] toBytes(long n, byte[] b, int offset) {
 b[offset+7] = (byte) (n);
 n = 8;



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



svn commit: r1366896 - /tomcat/trunk/java/org/apache/catalina/tribes/io/BufferPool.java

2012-07-29 Thread markt
Author: markt
Date: Sun Jul 29 19:15:54 2012
New Revision: 1366896

URL: http://svn.apache.org/viewvc?rev=1366896view=rev
Log:
Simplify since minimum Java version is now = 1.5

Modified:
tomcat/trunk/java/org/apache/catalina/tribes/io/BufferPool.java

Modified: tomcat/trunk/java/org/apache/catalina/tribes/io/BufferPool.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/io/BufferPool.java?rev=1366896r1=1366895r2=1366896view=diff
==
--- tomcat/trunk/java/org/apache/catalina/tribes/io/BufferPool.java (original)
+++ tomcat/trunk/java/org/apache/catalina/tribes/io/BufferPool.java Sun Jul 29 
19:15:54 2012
@@ -55,27 +55,18 @@ public class BufferPool {
 
 
 public static BufferPool getBufferPool() {
-if (  (instance == null) ) {
+if (instance == null) {
 synchronized (BufferPool.class) {
-if ( instance == null ) {
-   BufferPoolAPI pool = null;
-   Class? clazz = null;
-   try {
-   // TODO Is this approach still required?
-   clazz = 
Class.forName(org.apache.catalina.tribes.io.BufferPool15Impl);
-   pool = (BufferPoolAPI)clazz.newInstance();
-   } catch ( Throwable x ) {
-   log.warn(Unable to initilize BufferPool, not pooling 
XByteBuffer objects:+x.getMessage());
-   if ( log.isDebugEnabled() ) log.debug(Unable to 
initilize BufferPool, not pooling XByteBuffer objects:,x);
-   }
-   if (pool != null) {
-   pool.setMaxSize(DEFAULT_POOL_SIZE);
-   log.info(Created a buffer pool with max 
size:+DEFAULT_POOL_SIZE+ bytes of 
type:+(clazz!=null?clazz.getName():null));
-   instance = new BufferPool(pool);
-   }
-}//end if
-}//sync
-}//end if
+if (instance == null) {
+   BufferPoolAPI pool = new BufferPool15Impl();
+   pool.setMaxSize(DEFAULT_POOL_SIZE);
+   log.info(Created a buffer pool with max size: +
+   DEFAULT_POOL_SIZE +  bytes of type:  +
+   pool.getClass().getName());
+   instance = new BufferPool(pool);
+}
+}
+}
 return instance;
 }
 



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



svn commit: r1366910 - /tomcat/trunk/java/org/apache/catalina/tribes/io/ReplicationStream.java

2012-07-29 Thread markt
Author: markt
Date: Sun Jul 29 19:26:41 2012
New Revision: 1366910

URL: http://svn.apache.org/viewvc?rev=1366910view=rev
Log:
FindBugs: Fix potential NPE

Modified:
tomcat/trunk/java/org/apache/catalina/tribes/io/ReplicationStream.java

Modified: tomcat/trunk/java/org/apache/catalina/tribes/io/ReplicationStream.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/io/ReplicationStream.java?rev=1366910r1=1366909r2=1366910view=diff
==
--- tomcat/trunk/java/org/apache/catalina/tribes/io/ReplicationStream.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/tribes/io/ReplicationStream.java Sun 
Jul 29 19:26:41 2012
@@ -104,7 +104,12 @@ public final class ReplicationStream ext
 protected Class? resolveProxyClass(String[] interfaces)
 throws IOException, ClassNotFoundException {
 
-ClassLoader latestLoader = (classLoaders!=null  
classLoaders.length==0)?null:classLoaders[0];
+ClassLoader latestLoader;
+if (classLoaders != null  classLoaders.length  0) {
+latestLoader = classLoaders[0];
+} else {
+latestLoader = null;
+}
 ClassLoader nonPublicLoader = null;
 boolean hasNonPublicInterface = false;
 



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



svn commit: r1366914 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/catalina/tribes/io/ReplicationStream.java webapps/docs/changelog.xml

2012-07-29 Thread markt
Author: markt
Date: Sun Jul 29 19:32:20 2012
New Revision: 1366914

URL: http://svn.apache.org/viewvc?rev=1366914view=rev
Log:
FindBugs: Fix potential NPE

Modified:
tomcat/tc7.0.x/trunk/   (props changed)

tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/io/ReplicationStream.java
tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc7.0.x/trunk/
--
  Merged /tomcat/trunk:r1366910

Modified: 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/io/ReplicationStream.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/io/ReplicationStream.java?rev=1366914r1=1366913r2=1366914view=diff
==
--- 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/io/ReplicationStream.java 
(original)
+++ 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/io/ReplicationStream.java 
Sun Jul 29 19:32:20 2012
@@ -105,7 +105,12 @@ public final class ReplicationStream ext
 protected Class? resolveProxyClass(String[] interfaces)
 throws IOException, ClassNotFoundException {
 
-ClassLoader latestLoader = (classLoaders!=null  
classLoaders.length==0)?null:classLoaders[0];
+ClassLoader latestLoader;
+if (classLoaders != null  classLoaders.length  0) {
+latestLoader = classLoaders[0];
+} else {
+latestLoader = null;
+}
 ClassLoader nonPublicLoader = null;
 boolean hasNonPublicInterface = false;
 

Modified: tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml?rev=1366914r1=1366913r2=1366914view=diff
==
--- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Sun Jul 29 19:32:20 2012
@@ -121,6 +121,14 @@
   /fix
 /changelog
   /subsection
+  subsection name=Tribes
+changelog
+  fix
+Avoid potential NPE identified by Find Bugs in
+codeorg.apache.catalina.tribes.io.ReplicationStream/code. (markt)
+  /fix
+/changelog
+  /subsection
 /section
 section name=Tomcat 7.0.29 (markt) rtext=released 2012-07-08
   subsection name=Catalina



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



svn commit: r1366945 - in /tomcat/trunk: java/org/apache/catalina/connector/Response.java test/org/apache/catalina/connector/TestResponse.java

2012-07-29 Thread markt
Author: markt
Date: Sun Jul 29 20:29:20 2012
New Revision: 1366945

URL: http://svn.apache.org/viewvc?rev=1366945view=rev
Log:
Partial fix for https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
If the relative URL can't be made absolute, do not encode it and return it as is

Modified:
tomcat/trunk/java/org/apache/catalina/connector/Response.java
tomcat/trunk/test/org/apache/catalina/connector/TestResponse.java

Modified: tomcat/trunk/java/org/apache/catalina/connector/Response.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/connector/Response.java?rev=1366945r1=1366944r2=1366945view=diff
==
--- tomcat/trunk/java/org/apache/catalina/connector/Response.java (original)
+++ tomcat/trunk/java/org/apache/catalina/connector/Response.java Sun Jul 29 
20:29:20 2012
@@ -1134,7 +1134,14 @@ public class Response
 @Override
 public String encodeURL(String url) {
 
-String absolute = toAbsolute(url);
+String absolute;
+try {
+absolute = toAbsolute(url);
+} catch (IllegalArgumentException iae) {
+// Relative URL
+return url;
+}
+
 if (isEncodeable(absolute)) {
 // W3c spec clearly said
 if (url.equalsIgnoreCase()) {
@@ -1702,7 +1709,7 @@ public class Response
 if (index  0) {
 break;
 }
-// Prevent from going outside our context
+// Can't go above the server root
 if (index == startIndex) {
 throw new IllegalArgumentException();
 }
@@ -1719,7 +1726,7 @@ public class Response
 index = index2;
 }
 
-// Add the query string (if present) back in
+// Add the query string and/or fragment (if present) back in
 if (truncateCC != null) {
 try {
 cc.append(truncateCC, 0, truncateCC.length);

Modified: tomcat/trunk/test/org/apache/catalina/connector/TestResponse.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/connector/TestResponse.java?rev=1366945r1=1366944r2=1366945view=diff
==
--- tomcat/trunk/test/org/apache/catalina/connector/TestResponse.java (original)
+++ tomcat/trunk/test/org/apache/catalina/connector/TestResponse.java Sun Jul 
29 20:29:20 2012
@@ -364,6 +364,30 @@ public class TestResponse extends Tomcat
 }
 
 
+@Test
+public void testBug53469a() throws Exception {
+Request req = new TesterMockRequest();
+Response resp = new Response();
+resp.setRequest(req);
+
+String result = resp.encodeURL(../bar.html);
+
+Assert.assertEquals(../bar.html, result);
+}
+
+
+@Test
+public void testBug53469b() throws Exception {
+Request req = new TesterMockRequest();
+Response resp = new Response();
+resp.setRequest(req);
+
+String result = resp.encodeURL(../../../../bar.html);
+
+Assert.assertEquals(../../../../bar.html, result);
+}
+
+
 private static final class Bug52811Servlet extends HttpServlet {
 private static final long serialVersionUID = 1L;
 



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



svn commit: r1366946 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/catalina/connector/Response.java test/org/apache/catalina/connector/TestResponse.java webapps/docs/changelog.xml

2012-07-29 Thread markt
Author: markt
Date: Sun Jul 29 20:33:16 2012
New Revision: 1366946

URL: http://svn.apache.org/viewvc?rev=1366946view=rev
Log:
Partial fix for https://issues.apache.org/bugzilla/show_bug.cgi?id=53469
If the relative URL can't be made absolute, do not encode it and return it as is

Modified:
tomcat/tc7.0.x/trunk/   (props changed)
tomcat/tc7.0.x/trunk/java/org/apache/catalina/connector/Response.java
tomcat/tc7.0.x/trunk/test/org/apache/catalina/connector/TestResponse.java
tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc7.0.x/trunk/
--
  Merged /tomcat/trunk:r1366945

Modified: tomcat/tc7.0.x/trunk/java/org/apache/catalina/connector/Response.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/catalina/connector/Response.java?rev=1366946r1=1366945r2=1366946view=diff
==
--- tomcat/tc7.0.x/trunk/java/org/apache/catalina/connector/Response.java 
(original)
+++ tomcat/tc7.0.x/trunk/java/org/apache/catalina/connector/Response.java Sun 
Jul 29 20:33:16 2012
@@ -1242,7 +1242,14 @@ public class Response
 @Override
 public String encodeURL(String url) {
 
-String absolute = toAbsolute(url);
+String absolute;
+try {
+absolute = toAbsolute(url);
+} catch (IllegalArgumentException iae) {
+// Relative URL
+return url;
+}
+
 if (isEncodeable(absolute)) {
 // W3c spec clearly said
 if (url.equalsIgnoreCase()) {
@@ -1810,7 +1817,7 @@ public class Response
 if (index  0) {
 break;
 }
-// Prevent from going outside our context
+// Can't go above the server root
 if (index == startIndex) {
 throw new IllegalArgumentException();
 }
@@ -1827,7 +1834,7 @@ public class Response
 index = index2;
 }
 
-// Add the query string (if present) back in
+// Add the query string and/or fragment (if present) back in
 if (truncateCC != null) {
 try {
 cc.append(truncateCC, 0, truncateCC.length);

Modified: 
tomcat/tc7.0.x/trunk/test/org/apache/catalina/connector/TestResponse.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/test/org/apache/catalina/connector/TestResponse.java?rev=1366946r1=1366945r2=1366946view=diff
==
--- tomcat/tc7.0.x/trunk/test/org/apache/catalina/connector/TestResponse.java 
(original)
+++ tomcat/tc7.0.x/trunk/test/org/apache/catalina/connector/TestResponse.java 
Sun Jul 29 20:33:16 2012
@@ -364,6 +364,30 @@ public class TestResponse extends Tomcat
 }
 
 
+@Test
+public void testBug53469a() throws Exception {
+Request req = new TesterMockRequest();
+Response resp = new Response();
+resp.setRequest(req);
+
+String result = resp.encodeURL(../bar.html);
+
+Assert.assertEquals(../bar.html, result);
+}
+
+
+@Test
+public void testBug53469b() throws Exception {
+Request req = new TesterMockRequest();
+Response resp = new Response();
+resp.setRequest(req);
+
+String result = resp.encodeURL(../../../../bar.html);
+
+Assert.assertEquals(../../../../bar.html, result);
+}
+
+
 private static final class Bug52811Servlet extends HttpServlet {
 private static final long serialVersionUID = 1L;
 

Modified: tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml?rev=1366946r1=1366945r2=1366946view=diff
==
--- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Sun Jul 29 20:33:16 2012
@@ -63,6 +63,13 @@
 when request processing completes. (kkolinko)
   /fix
   fix
+bug53469/bug: If a URL passed to
+codejavax.servlet.http.HttpServletResponse.encodeURL()/code cannot
+be made absolute, never encode it and return it unchanged. Previously,
+the fix for bug53062/bug meant than an
+codeIllegalArgumentException/code was thrown. (markt)
+  /fix
+  fix
 bug53498/bug: Fix atomicity bugs in use of concurrent collections.
 Based on a patch by Yu Lin. (markt)
   /fix



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



[Bug 53469] possible bug in Response.normalize(CharChunk cc)

2012-07-29 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=53469

--- Comment #16 from Mark Thomas ma...@apache.org ---
I have fixed the IAE in trunk and 7.0.x.

I am leaving this open while I wait for clarification from the Servlet EG as to
how relative URLs passed to encodeURL should be treated.

See http://java.net/jira/browse/SERVLET_SPEC-43

-- 
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: r1366953 - /tomcat/trunk/java/org/apache/catalina/startup/ContextConfig.java

2012-07-29 Thread markt
Author: markt
Date: Sun Jul 29 21:21:08 2012
New Revision: 1366953

URL: http://svn.apache.org/viewvc?rev=1366953view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=53535
Reduce memory footprint when performing class scanning on Context start.
Patch provided by Cedomir Igaly

Modified:
tomcat/trunk/java/org/apache/catalina/startup/ContextConfig.java

Modified: tomcat/trunk/java/org/apache/catalina/startup/ContextConfig.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/startup/ContextConfig.java?rev=1366953r1=1366952r2=1366953view=diff
==
--- tomcat/trunk/java/org/apache/catalina/startup/ContextConfig.java (original)
+++ tomcat/trunk/java/org/apache/catalina/startup/ContextConfig.java Sun Jul 29 
21:21:08 2012
@@ -2122,10 +2122,8 @@ public class ContextConfig implements Li
 private void populateSCIsForCacheEntry(JavaClassCacheEntry cacheEntry) {
 SetServletContainerInitializer result = new HashSet();
 
-JavaClass javaClass = cacheEntry.getJavaClass();
-
 // Super class
-String superClassName = javaClass.getSuperclassName();
+String superClassName = cacheEntry.getSuperclassName();
 JavaClassCacheEntry superClassCacheEntry =
 javaClassCache.get(superClassName);
 
@@ -2145,7 +2143,7 @@ public class ContextConfig implements Li
 result.addAll(getSCIsForClass(superClassName));
 
 // Interfaces
-for (String interfaceName : javaClass.getInterfaceNames()) {
+for (String interfaceName : cacheEntry.getInterfaceNames()) {
 JavaClassCacheEntry interfaceEntry =
 javaClassCache.get(interfaceName);
 // A null could mean that the class not present in application or
@@ -2589,15 +2587,23 @@ public class ContextConfig implements Li
 }
 
 private static class JavaClassCacheEntry {
-private final JavaClass javaClass;
+public final String superclassName;
+
+public final String[] interfaceNames;
+
 private SetServletContainerInitializer sciSet = null;
 
 public JavaClassCacheEntry(JavaClass javaClass) {
-this.javaClass = javaClass;
+superclassName = javaClass.getSuperclassName();
+interfaceNames = javaClass.getInterfaceNames();
+}
+
+public String getSuperclassName() {
+return superclassName;
 }
 
-public JavaClass getJavaClass() {
-return javaClass;
+public String[] getInterfaceNames() {
+return interfaceNames;
 }
 
 public SetServletContainerInitializer getSciSet() {



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



svn commit: r1366955 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/catalina/startup/ContextConfig.java webapps/docs/changelog.xml

2012-07-29 Thread markt
Author: markt
Date: Sun Jul 29 21:23:13 2012
New Revision: 1366955

URL: http://svn.apache.org/viewvc?rev=1366955view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=53535
Reduce memory footprint when performing class scanning on Context start.
Patch provided by Cedomir Igaly

Modified:
tomcat/tc7.0.x/trunk/   (props changed)
tomcat/tc7.0.x/trunk/java/org/apache/catalina/startup/ContextConfig.java
tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc7.0.x/trunk/
--
  Merged /tomcat/trunk:r1366953

Modified: 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/startup/ContextConfig.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/catalina/startup/ContextConfig.java?rev=1366955r1=1366954r2=1366955view=diff
==
--- tomcat/tc7.0.x/trunk/java/org/apache/catalina/startup/ContextConfig.java 
(original)
+++ tomcat/tc7.0.x/trunk/java/org/apache/catalina/startup/ContextConfig.java 
Sun Jul 29 21:23:13 2012
@@ -2211,10 +2211,8 @@ public class ContextConfig implements Li
 SetServletContainerInitializer result =
 new HashSetServletContainerInitializer();
 
-JavaClass javaClass = cacheEntry.getJavaClass();
-
 // Super class
-String superClassName = javaClass.getSuperclassName();
+String superClassName = cacheEntry.getSuperclassName();
 JavaClassCacheEntry superClassCacheEntry =
 javaClassCache.get(superClassName);
 
@@ -2234,7 +2232,7 @@ public class ContextConfig implements Li
 result.addAll(getSCIsForClass(superClassName));
 
 // Interfaces
-for (String interfaceName : javaClass.getInterfaceNames()) {
+for (String interfaceName : cacheEntry.getInterfaceNames()) {
 JavaClassCacheEntry interfaceEntry =
 javaClassCache.get(interfaceName);
 // A null could mean that the class not present in application or
@@ -2678,15 +2676,23 @@ public class ContextConfig implements Li
 }
 
 private static class JavaClassCacheEntry {
-private final JavaClass javaClass;
+public final String superclassName;
+
+public final String[] interfaceNames;
+
 private SetServletContainerInitializer sciSet = null;
 
 public JavaClassCacheEntry(JavaClass javaClass) {
-this.javaClass = javaClass;
+superclassName = javaClass.getSuperclassName();
+interfaceNames = javaClass.getInterfaceNames();
+}
+
+public String getSuperclassName() {
+return superclassName;
 }
 
-public JavaClass getJavaClass() {
-return javaClass;
+public String[] getInterfaceNames() {
+return interfaceNames;
 }
 
 public SetServletContainerInitializer getSciSet() {

Modified: tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml?rev=1366955r1=1366954r2=1366955view=diff
==
--- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Sun Jul 29 21:23:13 2012
@@ -85,6 +85,10 @@
   update
 Add some HTTP status codes registered at IANA. (rjung)
   /update
+  fix
+bug53535/bug: Reduce memory footprint when performing class 
scanning
+on Context start. Patch provided by Cedomir Igaly. (markt)
+  /fix
 /changelog
   /subsection
   subsection name=Coyote



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



[Bug 53615] java.lang.OutOfMemoryError: GC overhead limit exceeded after upgrading from 7.0.28 to 7.0.29

2012-07-29 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=53615

Bug 53615 depends on bug 53535, which changed state.

Bug 53535 Summary: Out of memory while performing ContextConfig.webConfig
https://issues.apache.org/bugzilla/show_bug.cgi?id=53535

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

-- 
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



[Bug 53535] Out of memory while performing ContextConfig.webConfig

2012-07-29 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=53535

Mark Thomas ma...@apache.org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #4 from Mark Thomas ma...@apache.org ---
Thanks for the patch. It has been applied to trunk and 7.0.x and will be
included in 7.0.30 onwards.

-- 
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: r1366956 - /tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml

2012-07-29 Thread markt
Author: markt
Date: Sun Jul 29 21:30:49 2012
New Revision: 1366956

URL: http://svn.apache.org/viewvc?rev=1366956view=rev
Log:
Add missing entry

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

Modified: tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml?rev=1366956r1=1366955r2=1366956view=diff
==
--- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Sun Jul 29 21:30:49 2012
@@ -340,6 +340,12 @@
 (markt)
   /fix
   fix
+bug53047/bug: If a JDBC Realm or DataSource Realm is configured for
+an all roles mode that only requires authorization (and no roles) and 
no
+role table or column is defined, don't populate the Principal's roles.
+(markt)
+  /fix
+  fix
 bug53056/bug: Add APR version number to tcnative version INFO log
 message. (schultz)
   /fix



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



[Bug 53047] JDBCRealm allRolesMode=authOnly still needs role table

2012-07-29 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=53047

--- Comment #12 from Mark Thomas ma...@apache.org ---
See comment #10

The changelog entry was missed. This has been fixed.

http://svn.apache.org/viewvc?view=revisionrevision=1348499
http://svn.apache.org/viewvc?view=revisionrevision=1366956

-- 
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



[Bug 53409] Request class actively restricts sending parameters in the body unless using POST

2012-07-29 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=53409

Mark Thomas ma...@apache.org changed:

   What|Removed |Added

 OS||All
   Severity|normal  |enhancement

--- Comment #1 from Mark Thomas ma...@apache.org ---
Note that the restriction is as a result of the Servlet specification (SRV
3.1).

-- 
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: r1366959 - /tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/TcpPingInterceptor.java

2012-07-29 Thread markt
Author: markt
Date: Sun Jul 29 21:47:24 2012
New Revision: 1366959

URL: http://svn.apache.org/viewvc?rev=1366959view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=53606
Fix possible NPE when sending pings between cluster nodes
Based on a patch by F.Arnoud

Modified:

tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/TcpPingInterceptor.java

Modified: 
tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/TcpPingInterceptor.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/TcpPingInterceptor.java?rev=1366959r1=1366958r2=1366959view=diff
==
--- 
tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/TcpPingInterceptor.java
 (original)
+++ 
tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/TcpPingInterceptor.java
 Sun Jul 29 21:47:24 2012
@@ -122,13 +122,16 @@ public class TcpPingInterceptor extends 
 }
 
 protected void sendPing() {
-if (failureDetector.get() != null) {
+TcpFailureDetector tcpFailureDetector =
+failureDetector != null ? failureDetector.get() : null;
+if (tcpFailureDetector != null) {
 // We have a reference to the failure detector
 // Piggy back on it
-failureDetector.get().checkMembers(true);
+tcpFailureDetector.checkMembers(true);
 } else {
-StaticMembershipInterceptor smi = staticMembers.get();
-if (staticOnly  smi != null) {
+StaticMembershipInterceptor smi =
+staticOnly  staticMembers != null ? staticMembers.get() 
: null;
+if (smi != null) {
 sendPingMessage(smi.getMembers());
 } else {
 sendPingMessage(getMembers());



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



svn commit: r1366961 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/catalina/tribes/group/interceptors/TcpPingInterceptor.java webapps/docs/changelog.xml

2012-07-29 Thread markt
Author: markt
Date: Sun Jul 29 21:51:06 2012
New Revision: 1366961

URL: http://svn.apache.org/viewvc?rev=1366961view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=53606
Fix possible NPE when sending pings between cluster nodes
Based on a patch by F.Arnoud

Modified:
tomcat/tc7.0.x/trunk/   (props changed)

tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/group/interceptors/TcpPingInterceptor.java
tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml

Propchange: tomcat/tc7.0.x/trunk/
--
  Merged /tomcat/trunk:r1366959

Modified: 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/group/interceptors/TcpPingInterceptor.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/group/interceptors/TcpPingInterceptor.java?rev=1366961r1=1366960r2=1366961view=diff
==
--- 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/group/interceptors/TcpPingInterceptor.java
 (original)
+++ 
tomcat/tc7.0.x/trunk/java/org/apache/catalina/tribes/group/interceptors/TcpPingInterceptor.java
 Sun Jul 29 21:51:06 2012
@@ -122,13 +122,17 @@ public class TcpPingInterceptor extends 
 }
 
 protected void sendPing() {
-if (failureDetector.get()!=null) {
-//we have a reference to the failure detector
-//piggy back on that dude
-failureDetector.get().checkMembers(true);
-}else {
-if (staticOnly  staticMembers.get()!=null) {
-sendPingMessage(staticMembers.get().getMembers());
+TcpFailureDetector tcpFailureDetector =
+failureDetector != null ? failureDetector.get() : null;
+if (tcpFailureDetector != null) {
+// We have a reference to the failure detector
+// Piggy back on it
+tcpFailureDetector.checkMembers(true);
+} else {
+StaticMembershipInterceptor smi =
+staticOnly  staticMembers != null ? staticMembers.get() 
: null;
+if (smi != null) {
+sendPingMessage(smi.getMembers());
 } else {
 sendPingMessage(getMembers());
 }

Modified: tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml?rev=1366961r1=1366960r2=1366961view=diff
==
--- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Sun Jul 29 21:51:06 2012
@@ -138,6 +138,10 @@
 Avoid potential NPE identified by Find Bugs in
 codeorg.apache.catalina.tribes.io.ReplicationStream/code. (markt)
   /fix
+  fix
+bug53606/bug: Fix potential NPE in codeTcpPingInterceptor/code.
+Based on a patch by F. Arnoud. (markt)
+  /fix
 /changelog
   /subsection
 /section



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



[Bug 53606] NullPointerException in TcpPingInterceptor

2012-07-29 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=53606

Mark Thomas ma...@apache.org changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED
   Severity|blocker |normal

--- Comment #5 from Mark Thomas ma...@apache.org ---
Fixed in trunk and 7.0.x and will be included in 7.0.30 onwards. Thanks for the
patch.

-- 
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



[Bug 53606] NullPointerException in TcpPingInterceptor

2012-07-29 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=53606

Mark Thomas ma...@apache.org changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
  Component|Cluster |Cluster
 Resolution|FIXED   |---
Product|Tomcat 7|Tomcat 6
   Target Milestone|--- |default

--- Comment #6 from Mark Thomas ma...@apache.org ---
Whoops. Re-open for Tomcat 6.

-- 
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



[Bug 53409] Request class actively restricts sending parameters in the body unless using POST

2012-07-29 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=53409

--- Comment #2 from Konstantin Kolinko knst.koli...@gmail.com ---
If you want to backport the parseBodyMethods Connectors' attribute to Tomcat
6, someone (you?) has to prepare a patch for Tomcat 6 in unified diff format
(as documented elsewhere), and attach it here or better to issue 48692 of which
this one is a duplicate.

The code to start is in r1041892, though there were minor changes to it later.
http://svn.apache.org/viewvc?view=revisionrevision=1041892

-- 
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: [Proposal] Preparatory refactoring for Resource handling refactoring

2012-07-29 Thread Konstantin Kolinko
2012/6/17 Mark Thomas ma...@apache.org:
 On 16/06/2012 19:18, Mark Thomas wrote:


 Konstantin Kolinko knst.koli...@gmail.com wrote:

 2012/6/16 Mark Thomas ma...@apache.org:

 URLs are needed per Servlet API, so they cannot be removed.
 Does our jndi schema need DirContext as underlying
 implementation?

 Our jndi scheme was used to provide access to resources. I
 believe
 all
 of that will now go.

 I noticed the following commit in archives:
 http://svn.apache.org/viewvc?view=revisionrevision=1137646 so
 we have to deal with such schema combinations as jar:jndi:.

 No we won't. We only hadf to deal with URLs like that because we
 generated them.


 How are you going to implement ServletContext.getResource(String):
 URL

 without a custom URL scheme  (be it named jndi or somehow else)?

 For file resources it might be possible to produce the actual
 URL pointing to a JAR entry or to a file (leaving aside the
 question of whether it exposes too much details),  but you cannot
 do so with directories,  as entries in a directory can be assembled
 from several sources.

 My intention was to use the URL for the actual resource. For
 directories, I'll use the first matching dir I find although I need
 to re-read the spec and Javadoc to make sure there aren't any nasty
 surprises waiting to trip me up.

 Having re-read the specification and Javadoc, I don't see anything of
 concern. Additional pairs of eyes wouldn't hurt though.

 How to handle getResource() for a directory that exists in one or more
 overlays and/or the main WAR is an interesting question. I'll be sure to
 raise it within the Servlet EG when we get back to that question.


If we remove JNDI stuff from resource handling,  one of challenges
might be to re-implement DefaultServlet using only Servlet API
methods. Well, if the former is not possible, it might use the new
resources API (that you are going to implement instead of jndi one)
and be thus still tied with Tomcat internals...

If one reimplements DefaultServlet, one of the tasks would be to
generate directory listings. Those include file size and file
timestamp. One needs to obtain URL of a resource, open its
URLConnection and ask those attributes.


One good thing with jndi: URLs returned via Servlet API is that they
are backed by an instance of ProxyDirContext class and it has a cache
(*).  If we change implementation and return true URLs, they will
bypass the cache.  I suspect that using a jar: URL directly (in case
of an unpacked WAR file) may have poor performance.

Other good thing is that you can create relative URLs as new URL(Url,
String), which inherits URLStreamHandler instance from the original
URL, and thus inherits access to ProxyDirContext instance.  If it is a
jndi URL it will have access to the full resources hierarchy of the
web application.  If it is a true URL, it will be limited to its
origin file system.

The above two are the reasons why I think that in Tomcat 8 we cannot
return true URLs from ServletContext.getResource(String) method and
must still support the jndi: or some other proprietary URL schema.


(*) for reference: TTL of entries in ProxyDirContext#cache is by
default 5 seconds (5000). If the time has elapsed, the resource is
revalidated by comparing its timestamp with original one  The TTL is
configured via ProxyDirContext constructor - BaseDirContext#cacheTTL
- StandardContext#cacheTTL and thus is configurable on a Context.
http://tomcat.apache.org/tomcat-7.0-doc/config/context.html

Best regards,
Konstantin Kolinko

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



[GUMP@vmgump]: Project tomcat-trunk-validate-eoln (in module tomcat-trunk) failed

2012-07-29 Thread Bill Barker
To whom it may engage...

This is an automated request, but not an unsolicited one. For 
more information please visit http://gump.apache.org/nagged.html, 
and/or contact the folk at gene...@gump.apache.org.

Project tomcat-trunk-validate-eoln has an issue affecting its community 
integration.
This issue affects 1 projects,
 and has been outstanding for 33 runs.
The current state of this project is 'Failed', with reason 'Build Failed'.
For reference only, the following projects are affected by this:
- tomcat-trunk-validate-eoln :  Tomcat 8.x, a web server implementing Java 
Servlet 3.1,
...


Full details are available at:

http://vmgump.apache.org/gump/public/tomcat-trunk/tomcat-trunk-validate-eoln/index.html

That said, some information snippets are provided here.

The following annotations (debug/informational/warning/error messages) were 
provided:
 -INFO- Failed with reason build failed



The following work was performed:
http://vmgump.apache.org/gump/public/tomcat-trunk/tomcat-trunk-validate-eoln/gump_work/build_tomcat-trunk_tomcat-trunk-validate-eoln.html
Work Name: build_tomcat-trunk_tomcat-trunk-validate-eoln (Type: Build)
Work ended in a state of : Failed
Elapsed: 1 sec
Command Line: /usr/lib/jvm/java-6-openjdk/bin/java -Djava.awt.headless=true 
-Dbuild.sysclasspath=only org.apache.tools.ant.Main 
-Dgump.merge=/srv/gump/public/gump/work/merge.xml validate-eoln 
[Working Directory: /srv/gump/public/workspace/tomcat-trunk]
CLASSPATH: 
/usr/lib/jvm/java-6-openjdk/lib/tools.jar:/srv/gump/public/workspace/tomcat-trunk/output/classes:/srv/gump/public/workspace/ant/dist/lib/ant.jar:/srv/gump/public/workspace/ant/dist/lib/ant-launcher.jar:/srv/gump/public/workspace/ant/dist/lib/ant-jmf.jar:/srv/gump/public/workspace/ant/dist/lib/ant-junit.jar:/srv/gump/public/workspace/ant/dist/lib/ant-swing.jar:/srv/gump/public/workspace/ant/dist/lib/ant-apache-resolver.jar:/srv/gump/public/workspace/ant/dist/lib/ant-apache-xalan2.jar:/srv/gump/public/workspace/xml-commons/java/build/resolver.jar
-
Buildfile: /srv/gump/public/workspace/tomcat-trunk/build.xml

build-prepare:
[mkdir] Created dir: /srv/gump/public/workspace/tomcat-trunk/output/classes
[mkdir] Created dir: /srv/gump/public/workspace/tomcat-trunk/output/build
[mkdir] Created dir: 
/srv/gump/public/workspace/tomcat-trunk/output/build/bin
[mkdir] Created dir: 
/srv/gump/public/workspace/tomcat-trunk/output/build/conf
[mkdir] Created dir: 
/srv/gump/public/workspace/tomcat-trunk/output/build/lib
[mkdir] Created dir: 
/srv/gump/public/workspace/tomcat-trunk/output/build/logs
[mkdir] Created dir: 
/srv/gump/public/workspace/tomcat-trunk/output/build/temp
[mkdir] Created dir: 
/srv/gump/public/workspace/tomcat-trunk/output/build/webapps

compile-prepare:
 [copy] Copying 1 file to 
/srv/gump/public/workspace/tomcat-trunk/java/org/apache/catalina/startup
 [copy] Copying 1 file to 
/srv/gump/public/workspace/tomcat-trunk/webapps/docs

validate-eoln:
[javac] Compiling 1 source file to 
/srv/gump/public/workspace/tomcat-trunk/output/classes
[javac] javac: invalid target release: 1.7
[javac] Usage: javac options source files
[javac] use -help for a list of possible options

BUILD FAILED
/srv/gump/public/workspace/tomcat-trunk/build.xml:523: Compile failed; see the 
compiler error output for details.

Total time: 1 second
-

To subscribe to this information via syndicated feeds:
- RSS: 
http://vmgump.apache.org/gump/public/tomcat-trunk/tomcat-trunk-validate-eoln/rss.xml
- Atom: 
http://vmgump.apache.org/gump/public/tomcat-trunk/tomcat-trunk-validate-eoln/atom.xml

== Gump Tracking Only ===
Produced by Apache Gump(TM) version 2.3.
Gump Run 1130072012, vmgump.apache.org:vmgump:1130072012
Gump E-mail Identifier (unique within run) #6.

--
Apache Gump
http://gump.apache.org/ [Instance: vmgump]

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



[GUMP@vmgump]: Project tomcat-trunk-dbcp (in module tomcat-trunk) failed

2012-07-29 Thread Bill Barker
To whom it may engage...

This is an automated request, but not an unsolicited one. For 
more information please visit http://gump.apache.org/nagged.html, 
and/or contact the folk at gene...@gump.apache.org.

Project tomcat-trunk-dbcp has an issue affecting its community integration.
This issue affects 3 projects,
 and has been outstanding for 33 runs.
The current state of this project is 'Failed', with reason 'Build Failed'.
For reference only, the following projects are affected by this:
- tomcat-trunk :  Tomcat 8.x, a web server implementing Java Servlet 3.1,
...
- tomcat-trunk-dbcp :  Tomcat 8.x, a web server implementing Java Servlet 
3.1,
...
- tomcat-trunk-test :  Tomcat 8.x, a web server implementing Java Servlet 
3.1,
...


Full details are available at:

http://vmgump.apache.org/gump/public/tomcat-trunk/tomcat-trunk-dbcp/index.html

That said, some information snippets are provided here.

The following annotations (debug/informational/warning/error messages) were 
provided:
 -INFO- Made directory [/srv/gump/public/workspace/tomcat-trunk/tomcat-deps]
 -INFO- Failed with reason build failed
 -INFO- Failed to extract fallback artifacts from Gump Repository



The following work was performed:
http://vmgump.apache.org/gump/public/tomcat-trunk/tomcat-trunk-dbcp/gump_work/build_tomcat-trunk_tomcat-trunk-dbcp.html
Work Name: build_tomcat-trunk_tomcat-trunk-dbcp (Type: Build)
Work ended in a state of : Failed
Elapsed: 2 secs
Command Line: /usr/lib/jvm/java-6-openjdk/bin/java -Djava.awt.headless=true 
-Dbuild.sysclasspath=only org.apache.tools.ant.Main 
-Dgump.merge=/srv/gump/public/gump/work/merge.xml 
-Dcommons-pool.home=/srv/gump/public/workspace/commons-pool-1.x 
-Dcommons-dbcp.home=/srv/gump/public/workspace/commons-dbcp-1.x 
-Dtomcat-dbcp.jar=/srv/gump/public/workspace/tomcat-trunk/tomcat-deps/tomcat-dbcp-30072012.jar
 -Dtomcat-dbcp.home=/srv/gump/public/workspace/tomcat-trunk/tomcat-deps 
build-tomcat-dbcp 
[Working Directory: /srv/gump/public/workspace/tomcat-trunk]
CLASSPATH: 
/usr/lib/jvm/java-6-openjdk/lib/tools.jar:/srv/gump/public/workspace/ant/dist/lib/ant.jar:/srv/gump/public/workspace/ant/dist/lib/ant-launcher.jar:/srv/gump/public/workspace/ant/dist/lib/ant-jmf.jar:/srv/gump/public/workspace/ant/dist/lib/ant-junit.jar:/srv/gump/public/workspace/ant/dist/lib/ant-swing.jar:/srv/gump/public/workspace/ant/dist/lib/ant-apache-resolver.jar:/srv/gump/public/workspace/ant/dist/lib/ant-apache-xalan2.jar:/srv/gump/public/workspace/xml-commons/java/build/resolver.jar:/srv/gump/packages/jta-spec1_0_1/jta-spec1_0_1.jar
-
Buildfile: /srv/gump/public/workspace/tomcat-trunk/build.xml

build-prepare:
   [delete] Deleting directory 
/srv/gump/public/workspace/tomcat-trunk/output/build/temp
[mkdir] Created dir: 
/srv/gump/public/workspace/tomcat-trunk/output/build/temp

build-manifests:
[mkdir] Created dir: 
/srv/gump/public/workspace/tomcat-trunk/output/manifests
 [copy] Copying 12 files to 
/srv/gump/public/workspace/tomcat-trunk/output/manifests

build-tomcat-dbcp:
 [copy] Copying 70 files to 
/srv/gump/public/workspace/tomcat-trunk/tomcat-deps
[patch] patching file 
src/java/org/apache/commons/dbcp/DelegatingCallableStatement.java
[patch] Hunk #1 succeeded at 661 (offset -113 lines).
[patch] patching file 
src/java/org/apache/commons/dbcp/cpdsadapter/DriverAdapterCPDS.java
[patch] patching file 
src/java/org/apache/commons/dbcp/DelegatingResultSet.java
[patch] Hunk #1 succeeded at 1079 (offset -195 lines).
[patch] patching file 
src/java/org/apache/commons/dbcp/PoolingDataSource.java
[patch] Hunk #1 succeeded at 437 (offset -52 lines).
[patch] patching file 
src/java/org/apache/commons/dbcp/DelegatingConnection.java
[patch] Hunk #1 succeeded at 678 (offset -126 lines).
[patch] patching file src/java/org/apache/commons/dbcp/PoolingDriver.java
[patch] Hunk #1 succeeded at 497 (offset -4 lines).
[patch] patching file 
src/java/org/apache/commons/dbcp/DelegatingStatement.java
[patch] Hunk #1 succeeded at 484 (offset -45 lines).
[patch] patching file 
src/java/org/apache/commons/dbcp/DelegatingDatabaseMetaData.java
[patch] Hunk #1 succeeded at 1204 (offset -173 lines).
[patch] patching file src/java/org/apache/commons/dbcp/BasicDataSource.java
[patch] Hunk #1 succeeded at 28 with fuzz 1.
[patch] Hunk #2 succeeded at 1782 (offset -19 lines).
[patch] patching file 
src/java/org/apache/commons/dbcp/datasources/InstanceKeyDataSource.java
[patch] Hunk #1 succeeded at 887 (offset -1 lines).
[mkdir] Created dir: 
/srv/gump/public/workspace/tomcat-trunk/tomcat-deps/src/java/org/apache/tomcat/dbcp
 [move] Moving 75 files to 
/srv/gump/public/workspace/tomcat-trunk/tomcat-deps/src/java/org/apache/tomcat/dbcp
[mkdir] Created dir: 
/srv/gump/public/workspace/tomcat-trunk/tomcat-deps/classes
[javac] Compiling 66 source files to 

[GUMP@vmgump]: Project tomcat-tc7.0.x-validate (in module tomcat-7.0.x) failed

2012-07-29 Thread Bill Barker
To whom it may engage...

This is an automated request, but not an unsolicited one. For 
more information please visit http://gump.apache.org/nagged.html, 
and/or contact the folk at gene...@gump.apache.org.

Project tomcat-tc7.0.x-validate has an issue affecting its community 
integration.
This issue affects 1 projects,
 and has been outstanding for 33 runs.
The current state of this project is 'Failed', with reason 'Build Failed'.
For reference only, the following projects are affected by this:
- tomcat-tc7.0.x-validate :  Tomcat 7.x, a web server implementing Java 
Servlet 3.0,
...


Full details are available at:

http://vmgump.apache.org/gump/public/tomcat-7.0.x/tomcat-tc7.0.x-validate/index.html

That said, some information snippets are provided here.

The following annotations (debug/informational/warning/error messages) were 
provided:
 -DEBUG- Dependency on checkstyle exists, no need to add for property 
checkstyle.jar.
 -INFO- Failed with reason build failed



The following work was performed:
http://vmgump.apache.org/gump/public/tomcat-7.0.x/tomcat-tc7.0.x-validate/gump_work/build_tomcat-7.0.x_tomcat-tc7.0.x-validate.html
Work Name: build_tomcat-7.0.x_tomcat-tc7.0.x-validate (Type: Build)
Work ended in a state of : Failed
Elapsed: 1 sec
Command Line: /usr/lib/jvm/java-6-openjdk/bin/java -Djava.awt.headless=true 
-Dbuild.sysclasspath=only org.apache.tools.ant.Main 
-Dgump.merge=/srv/gump/public/gump/work/merge.xml 
-Dcheckstyle.jar=/srv/gump/public/workspace/checkstyle/target/checkstyle-5.6-SNAPSHOT.jar
 -Dexecute.validate=true validate 
[Working Directory: /srv/gump/public/workspace/tomcat-7.0.x]
CLASSPATH: 
/usr/lib/jvm/java-6-openjdk/lib/tools.jar:/srv/gump/public/workspace/ant/dist/lib/ant.jar:/srv/gump/public/workspace/ant/dist/lib/ant-launcher.jar:/srv/gump/public/workspace/ant/dist/lib/ant-jmf.jar:/srv/gump/public/workspace/ant/dist/lib/ant-junit.jar:/srv/gump/public/workspace/ant/dist/lib/ant-swing.jar:/srv/gump/public/workspace/ant/dist/lib/ant-apache-resolver.jar:/srv/gump/public/workspace/ant/dist/lib/ant-apache-xalan2.jar:/srv/gump/public/workspace/xml-commons/java/build/resolver.jar:/srv/gump/public/workspace/checkstyle/target/checkstyle-5.6-SNAPSHOT.jar:/srv/gump/public/workspace/apache-commons/beanutils/dist/commons-beanutils-30072012.jar:/srv/gump/public/workspace/apache-commons/cli/target/commons-cli-1.3-SNAPSHOT.jar:/srv/gump/public/workspace/apache-commons/exec/target/commons-exec-1.1.1-SNAPSHOT.jar:/srv/gump/public/workspace/apache-commons/validator/dist/commons-validator-30072012.jar:/srv/gump/public/workspace/junit/dist/junit-30072012.jar:/srv/gump
 
/public/workspace/junit/dist/junit-dep-30072012.jar:/srv/gump/public/workspace/google-guava/guava/target/guava-*[0-9T].jar:/srv/gump/public/workspace/apache-commons/logging/target/commons-logging-30072012.jar:/srv/gump/public/workspace/apache-commons/logging/target/commons-logging-api-30072012.jar:/srv/gump/public/workspace/commons-collections-3.x/target/commons-collections-3.3-SNAPSHOT.jar:/srv/gump/packages/antlr/antlr-3.1.3.jar:/srv/gump/public/workspace/jdom/build/jdom.jar:/srv/gump/public/workspace/velocity-engine/bin/velocity-30072012.jar:/srv/gump/public/workspace/velocity-engine/bin/velocity-30072012-dep.jar
-
download-validate:

proxyflags:

setproxy:

testexist:
 [echo] Testing  for 
/srv/gump/public/workspace/checkstyle/target/checkstyle-5.6-SNAPSHOT.jar

downloadzip:

validate:
[mkdir] Created dir: 
/srv/gump/public/workspace/tomcat-7.0.x/output/res/checkstyle

BUILD FAILED
/srv/gump/public/workspace/tomcat-7.0.x/build.xml:450: Could not create type 
checkstyle due to java.lang.NoClassDefFoundError: 
com/google/common/collect/Lists
at 
com.puppycrawl.tools.checkstyle.CheckStyleTask.init(CheckStyleTask.java:78)
at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
at 
sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:57)
at 
sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
at java.lang.reflect.Constructor.newInstance(Constructor.java:532)
at 
org.apache.tools.ant.AntTypeDefinition.innerCreateAndSet(AntTypeDefinition.java:328)
at 
org.apache.tools.ant.AntTypeDefinition.createAndSet(AntTypeDefinition.java:274)
at 
org.apache.tools.ant.AntTypeDefinition.icreate(AntTypeDefinition.java:219)
at 
org.apache.tools.ant.AntTypeDefinition.create(AntTypeDefinition.java:206)
at 
org.apache.tools.ant.ComponentHelper.createComponent(ComponentHelper.java:286)
at 
org.apache.tools.ant.ComponentHelper.createComponent(ComponentHelper.java:264)
at 
org.apache.tools.ant.UnknownElement.makeObject(UnknownElement.java:417)
at 
org.apache.tools.ant.UnknownElement.maybeConfigure(UnknownElement.java:163)
at 

[GUMP@vmgump]: Project tomcat-trunk-validate (in module tomcat-trunk) failed

2012-07-29 Thread Bill Barker
To whom it may engage...

This is an automated request, but not an unsolicited one. For 
more information please visit http://gump.apache.org/nagged.html, 
and/or contact the folk at gene...@gump.apache.org.

Project tomcat-trunk-validate has an issue affecting its community integration.
This issue affects 1 projects,
 and has been outstanding for 33 runs.
The current state of this project is 'Failed', with reason 'Build Failed'.
For reference only, the following projects are affected by this:
- tomcat-trunk-validate :  Tomcat 8.x, a web server implementing Java 
Servlet 3.1,
...


Full details are available at:

http://vmgump.apache.org/gump/public/tomcat-trunk/tomcat-trunk-validate/index.html

That said, some information snippets are provided here.

The following annotations (debug/informational/warning/error messages) were 
provided:
 -DEBUG- Dependency on checkstyle exists, no need to add for property 
checkstyle.jar.
 -INFO- Failed with reason build failed



The following work was performed:
http://vmgump.apache.org/gump/public/tomcat-trunk/tomcat-trunk-validate/gump_work/build_tomcat-trunk_tomcat-trunk-validate.html
Work Name: build_tomcat-trunk_tomcat-trunk-validate (Type: Build)
Work ended in a state of : Failed
Elapsed: 1 sec
Command Line: /usr/lib/jvm/java-6-openjdk/bin/java -Djava.awt.headless=true 
-Dbuild.sysclasspath=only org.apache.tools.ant.Main 
-Dgump.merge=/srv/gump/public/gump/work/merge.xml 
-Dcheckstyle.jar=/srv/gump/public/workspace/checkstyle/target/checkstyle-5.6-SNAPSHOT.jar
 -Dexecute.validate=true validate 
[Working Directory: /srv/gump/public/workspace/tomcat-trunk]
CLASSPATH: 
/usr/lib/jvm/java-6-openjdk/lib/tools.jar:/srv/gump/public/workspace/ant/dist/lib/ant.jar:/srv/gump/public/workspace/ant/dist/lib/ant-launcher.jar:/srv/gump/public/workspace/ant/dist/lib/ant-jmf.jar:/srv/gump/public/workspace/ant/dist/lib/ant-junit.jar:/srv/gump/public/workspace/ant/dist/lib/ant-swing.jar:/srv/gump/public/workspace/ant/dist/lib/ant-apache-resolver.jar:/srv/gump/public/workspace/ant/dist/lib/ant-apache-xalan2.jar:/srv/gump/public/workspace/xml-commons/java/build/resolver.jar:/srv/gump/public/workspace/checkstyle/target/checkstyle-5.6-SNAPSHOT.jar:/srv/gump/public/workspace/apache-commons/beanutils/dist/commons-beanutils-30072012.jar:/srv/gump/public/workspace/apache-commons/cli/target/commons-cli-1.3-SNAPSHOT.jar:/srv/gump/public/workspace/apache-commons/exec/target/commons-exec-1.1.1-SNAPSHOT.jar:/srv/gump/public/workspace/apache-commons/validator/dist/commons-validator-30072012.jar:/srv/gump/public/workspace/junit/dist/junit-30072012.jar:/srv/gump
 
/public/workspace/junit/dist/junit-dep-30072012.jar:/srv/gump/public/workspace/google-guava/guava/target/guava-*[0-9T].jar:/srv/gump/public/workspace/apache-commons/logging/target/commons-logging-30072012.jar:/srv/gump/public/workspace/apache-commons/logging/target/commons-logging-api-30072012.jar:/srv/gump/public/workspace/commons-collections-3.x/target/commons-collections-3.3-SNAPSHOT.jar:/srv/gump/packages/antlr/antlr-3.1.3.jar:/srv/gump/public/workspace/jdom/build/jdom.jar:/srv/gump/public/workspace/velocity-engine/bin/velocity-30072012.jar:/srv/gump/public/workspace/velocity-engine/bin/velocity-30072012-dep.jar:/srv/gump/packages/javamail-1.4/mail.jar:/srv/gump/packages/javamail-1.4/lib/mailapi.jar:/srv/gump/packages/jaf-1.1ea/activation.jar
-
download-validate:

proxyflags:

setproxy:

testexist:
 [echo] Testing  for 
/srv/gump/public/workspace/checkstyle/target/checkstyle-5.6-SNAPSHOT.jar

downloadzip:

validate:
[mkdir] Created dir: 
/srv/gump/public/workspace/tomcat-trunk/output/res/checkstyle

BUILD FAILED
/srv/gump/public/workspace/tomcat-trunk/build.xml:461: Could not create type 
checkstyle due to java.lang.NoClassDefFoundError: 
com/google/common/collect/Lists
at 
com.puppycrawl.tools.checkstyle.CheckStyleTask.init(CheckStyleTask.java:78)
at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
at 
sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:57)
at 
sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
at java.lang.reflect.Constructor.newInstance(Constructor.java:532)
at 
org.apache.tools.ant.AntTypeDefinition.innerCreateAndSet(AntTypeDefinition.java:328)
at 
org.apache.tools.ant.AntTypeDefinition.createAndSet(AntTypeDefinition.java:274)
at 
org.apache.tools.ant.AntTypeDefinition.icreate(AntTypeDefinition.java:219)
at 
org.apache.tools.ant.AntTypeDefinition.create(AntTypeDefinition.java:206)
at 
org.apache.tools.ant.ComponentHelper.createComponent(ComponentHelper.java:286)
at 
org.apache.tools.ant.ComponentHelper.createComponent(ComponentHelper.java:264)
at 
org.apache.tools.ant.UnknownElement.makeObject(UnknownElement.java:417)
at