Fix bug 43117. Setting an empty workDir can delete all of CATALINA_HOME.
Patch provided by Takayuki Kaneko.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@653113 13f79535-47bb-0310-9956-ffa450edef68

A final bit of code clean up.
Actually no functional change.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@653112 13f79535-47bb-0310-9956-ffa450edef68

Fix various Eclipse warnings. Generics, unused code, etc
No functional change.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@653111 13f79535-47bb-0310-9956-ffa450edef68

Add Remy's NPE fix to trunk.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@653038 13f79535-47bb-0310-9956-ffa450edef68

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=43191
No way to turn off compression for some file types.
Based on a patch by Len Popp

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@653032 13f79535-47bb-0310-9956-ffa450edef68

Code clean up. Remove unused methods, unnecessary casts/code, fix generics warnings etc reported by Eclipse.
No functional change.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@652918 13f79535-47bb-0310-9956-ffa450edef68

Since not beeing able to build with JDK1.6 is so annoying - and commons-dbcp doesn't support jdk1.6 in the first place:
- let the compile work with jdk1.6, minus dbcp
- add a small warning at the end.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@652744 13f79535-47bb-0310-9956-ffa450edef68

Match build.xml - exclude webservices, missing deps
Not sure why it's checked in - if 'download' doesn't get the deps and build.xml doesn't build it.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@652743 13f79535-47bb-0310-9956-ffa450edef68

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=43150
# in installation path stops Tomcat starting

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@652669 13f79535-47bb-0310-9956-ffa450edef68

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=43333
Correct sendfile docs.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@652666 13f79535-47bb-0310-9956-ffa450edef68

Fix bug 43343. Correctly handle the case where a request arrives for a session we are in the middle of persisting.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@652662 13f79535-47bb-0310-9956-ffa450edef68

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=43142
If someone is daft enough to name their directory xxx.war, don't assume it is a war file.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@652441 13f79535-47bb-0310-9956-ffa450edef68

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=43366
Provide backwards compatibility for sessions command

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@651988 13f79535-47bb-0310-9956-ffa450edef68

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=44877
Prevent collisions in tag pool names

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@651984 13f79535-47bb-0310-9956-ffa450edef68

Further minor clean-up of unused/unnecessary code after I made Eclipse even more pedantic.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@651978 13f79535-47bb-0310-9956-ffa450edef68

Add some missing control checks.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@651977 13f79535-47bb-0310-9956-ffa450edef68

Code clean up o.a.t.util.net.
Generics and unused code.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@651976 13f79535-47bb-0310-9956-ffa450edef68

Code cleanup. o.a.t.util.res
No functional change

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@651968 13f79535-47bb-0310-9956-ffa450edef68

Reduce signoff error message to DEBUG, since it is harmless if mod_jk has hung up the phone

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@651792 13f79535-47bb-0310-9956-ffa450edef68

Fix bug https://issues.apache.org/bugzilla/show_bug.cgi?id=43425
Annotations not spec compliant.
Patch provided by Dain Sundstrom.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@651732 13f79535-47bb-0310-9956-ffa450edef68

Better fix https://issues.apache.org/bugzilla/show_bug.cgi?id=43617
Correctly handle quotes in attribute values for tag(x) files.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@651729 13f79535-47bb-0310-9956-ffa450edef68

Revert previous patch. It didn't cover all cases.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@651725 13f79535-47bb-0310-9956-ffa450edef68

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=43578
Tomcat doesn't start in linux if CATALINA_BASE contains a space
Patch provided by Ray Sauers

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@651713 13f79535-47bb-0310-9956-ffa450edef68

Fix compiler warnings for o.a.c.deploy.*
Note using generics would have highlighted bug 43470

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@651680 13f79535-47bb-0310-9956-ffa450edef68

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=43470
Based on a changes provided by Lucas Galfaso

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@651675 13f79535-47bb-0310-9956-ffa450edef68

Add missing access check.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@651662 13f79535-47bb-0310-9956-ffa450edef68

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=43617
Correctly handle quotes in attribute values for tag(x) files.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@651082 13f79535-47bb-0310-9956-ffa450edef68

Generics changes for o.a.t.util.res
No fucntional change

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@651076 13f79535-47bb-0310-9956-ffa450edef68

Generics changes for o.a.t.util.threads
No fucntional change

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@651075 13f79535-47bb-0310-9956-ffa450edef68

notify user of the actual error and add a todo behavior for buffer copying

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@650826 13f79535-47bb-0310-9956-ffa450edef68

new idea

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@650824 13f79535-47bb-0310-9956-ffa450edef68

Remy's better fix for 43683.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@650648 13f79535-47bb-0310-9956-ffa450edef68

Revert previous patch to fix obvious potential NPE as pointed out by Remy.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@650645 13f79535-47bb-0310-9956-ffa450edef68

Fix bug 31257. Quote endorsed dirs if they contain a space.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@649993 13f79535-47bb-0310-9956-ffa450edef68

Fix bug 29936. In some circumstances, Tomcat would use the parser from a webapp to parse web.xml and possibly context.xml files.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@649974 13f79535-47bb-0310-9956-ffa450edef68

Add license and props.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@649785 13f79535-47bb-0310-9956-ffa450edef68

More fixes for bug 43656 so new test case passes.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@649784 13f79535-47bb-0310-9956-ffa450edef68

Add test case for bug 43656. Based on a patch from Konstantin Kolinko.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@649783 13f79535-47bb-0310-9956-ffa450edef68

Fix bug 43656. coerceToType() modified some values. Patch provided by Nils Eckert.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@649638 13f79535-47bb-0310-9956-ffa450edef68

Remove unused code. No functional change.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@649637 13f79535-47bb-0310-9956-ffa450edef68

Patch for bug 43683. It could be more efficient if wrapper.started was visible but that would have made the patch quite invasive.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@649302 13f79535-47bb-0310-9956-ffa450edef68

update changelog

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@649253 13f79535-47bb-0310-9956-ffa450edef68

make the alwaysSend flag public, so that we can set it through config

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@649252 13f79535-47bb-0310-9956-ffa450edef68

update changelog

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@649248 13f79535-47bb-0310-9956-ffa450edef68

fix override of methods from standardsession

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@649238 13f79535-47bb-0310-9956-ffa450edef68

Align realm config doc and realm howto doc..

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@648161 13f79535-47bb-0310-9956-ffa450edef68

Fix some links so they work even if the ROOT app is renamed.
Minor clean-up.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@648017 13f79535-47bb-0310-9956-ffa450edef68

Remove second index page. It wasn't 100% in sync with the html one and I don't see why we need two.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@648014 13f79535-47bb-0310-9956-ffa450edef68

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=44380
Don't bother scanning non-file URLs for TLDs.
Patch provided by Florent BENOIT

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@647345 13f79535-47bb-0310-9956-ffa450edef68

Code clean up. Java5 and remove unused code. No functional change.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@647344 13f79535-47bb-0310-9956-ffa450edef68

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=44389
Make nested class static so it doesn't hold a reference to the ApplciationContext instance.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@647339 13f79535-47bb-0310-9956-ffa450edef68

Fix bug44391. Handling of escaped characters was incorrect.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@647316 13f79535-47bb-0310-9956-ffa450edef68

Fix for garbage test in bug 44494. Patch provided by Suzuki Yuichiro.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@647307 13f79535-47bb-0310-9956-ffa450edef68

Re-instate Rainer's code that fixes the 44494 test case for read with maxHttpHeaderSize > 8192.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@647304 13f79535-47bb-0310-9956-ffa450edef68

Fix bug 44392. HTML entities not resolved in SSI.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@647289 13f79535-47bb-0310-9956-ffa450edef68

Java 5 clean up. No functional change.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@646980 13f79535-47bb-0310-9956-ffa450edef68

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=44408
Avoid bottleneck by calling the synchronized method only once.
Patch provided by Robert Andersson.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@646889 13f79535-47bb-0310-9956-ffa450edef68

Fix bug 44428. Make sure m is not null to prevent NPE.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@646574 13f79535-47bb-0310-9956-ffa450edef68

Clean up type checking code. Patch provided by Konstantin Kolinko.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@646571 13f79535-47bb-0310-9956-ffa450edef68

Fix bug 44785. Correct deafult maxThreads for AJP connector.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@646559 13f79535-47bb-0310-9956-ffa450edef68

Additional permission needed to read JDK logging config.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@646550 13f79535-47bb-0310-9956-ffa450edef68

Handle case where we don't have permission to read context specific configuration.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@646543 13f79535-47bb-0310-9956-ffa450edef68

Better patch for 44766. I still can't see why the original code listed the classes individually.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@646106 13f79535-47bb-0310-9956-ffa450edef68

Revert previous fix as it needs correcting.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@646102 13f79535-47bb-0310-9956-ffa450edef68

This fixes 44766 but I can't see why the code was written as originally coded. Any light much appreciated. I'll give it a couple of days and then, assuming there are no objections, propose it for 6.0.x.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@646076 13f79535-47bb-0310-9956-ffa450edef68

All of the issues I was seeing with mark/reset were due to states that resulted in a call to CharChunk.flushBuffer().
I tried many different ways to fix it but by far the simplest was this patch that just increases the size of the CharChunk internal buffer when creating the mark sufficiently that flushBuffer() is never called.
With this patch I can't break mark/reset with bug 44494's test case in single or multibyte mode.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@645722 13f79535-47bb-0310-9956-ffa450edef68

Remy's resizing fix from bug 44494

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@645719 13f79535-47bb-0310-9956-ffa450edef68

Submitted by Ruediger Pluem. PR#44595

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@645428 13f79535-47bb-0310-9956-ffa450edef68

Fix for bug
https://issues.apache.org/bugzilla/show_bug.cgi
The ObjectInputStream.resolveProxyClass uses some funky native code base to retrieve what class loader to use, and it doesn't return the correct one. So we have to mimic the behavior in that class but use our class loaders instead

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@645184 13f79535-47bb-0310-9956-ffa450edef68

Implement asynchronous callbacks for CometEvent.close and CometEvent.setTimeout to avoid connections with very long timeouts getting stuck
CometProcessor implements servlet interface, or the code will fail deployment
if no keepalive, close the connection after comet transaction is complete

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@645175 13f79535-47bb-0310-9956-ffa450edef68

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=44759
Need source to generate javadocs
Patch provided by Stephen Bannasch.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@645169 13f79535-47bb-0310-9956-ffa450edef68

doco update for the latest change

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@644860 13f79535-47bb-0310-9956-ffa450edef68

fix sendfile, incorrect if else statements

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@644858 13f79535-47bb-0310-9956-ffa450edef68

Better fix for bug 44715. Document use of secret for AJP connector.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@643497 13f79535-47bb-0310-9956-ffa450edef68

Undo last commit - need to rework wording.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@643491 13f79535-47bb-0310-9956-ffa450edef68

Fix bug 44715. Document use of secret for AJP connector.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@643181 13f79535-47bb-0310-9956-ffa450edef68

Apply Remy's patch for 44494 for multi-byte character data.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@642819 13f79535-47bb-0310-9956-ffa450edef68

Remove unused code. No functional change.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@642816 13f79535-47bb-0310-9956-ffa450edef68

Tab police. Tabs -> 8 spaces. No functional change.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@642814 13f79535-47bb-0310-9956-ffa450edef68

https://issues.apache.org/bugzilla/show_bug.cgi?id=44529
No roles (deny all) trumps no auth-constraint (allow all)

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@642542 13f79535-47bb-0310-9956-ffa450edef68

Remove hack for getting env since JDK >= 1.5 provide a suitable method. Fixes issue with CGI, env and vista.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@642391 13f79535-47bb-0310-9956-ffa450edef68

Fix bug 44673. Throw IOE if stream is closed and a call is made to any read(), ready(), mark(), reset(), or skip() method as per javadocs for Reader.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@641076 13f79535-47bb-0310-9956-ffa450edef68

Code clean-up. Remove unused code. Fix generics related warnings. No functional change.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@641073 13f79535-47bb-0310-9956-ffa450edef68

Apply the patch for BZ44494 that was applied to 6.0.x

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@641051 13f79535-47bb-0310-9956-ffa450edef68

Revert original proposal for BZ44494 since an alternative patch was committed to 6.0.x

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@641043 13f79535-47bb-0310-9956-ffa450edef68

Fix bug 44541 - document packetSize for the AJP connector.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@640994 13f79535-47bb-0310-9956-ffa450edef68

Revert change from some work in progress that snuck in as part of an unrelated commit

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@640962 13f79535-47bb-0310-9956-ffa450edef68

SavedRequest is the only one using T_STR instead T_BYTES.
converting to T_BYTES allows to use utf8 encoded querystring with parameters and form based authentication.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@640897 13f79535-47bb-0310-9956-ffa450edef68

Add missing public on constructor.
Otherwise when using form based authentication:
+++
java.lang.NoClassDefFoundError: org/apache/tomcat/util/http/NamesEnumerator
        at org.apache.tomcat.util.http.MimeHeaders.names(MimeHeaders.java:201)
        at org.apache.catalina.connector.Request.getHeaderNames(Request.java:1873)
        at org.apache.catalina.authenticator.FormAuthenticator.saveRequest(FormAuthenticator.java:479)
+++

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@640888 13f79535-47bb-0310-9956-ffa450edef68

Fix bug 44611 https://issues.apache.org/bugzilla/show_bug.cgi?id=44611
DirContextURLConnection:
1) Does not implement the getHeaderFields() method
2) It's implementation of getHeaderField(String name) is case sensitive.
3) It returns an empty string "", rather than null, for header values which don't exist.
Patch provided by Chris Hubick.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@640584 13f79535-47bb-0310-9956-ffa450edef68

Fix BZ 44620 https://issues.apache.org/bugzilla/show_bug.cgi?id=44620
Move the code that handles writing more than the output buffer can accept in a single write to the method that does the writing. This then protects all calls to addToBB() from the issue described in BZ 44620 and does not require the socket.appWriteBufSize to be >= maxHttpHeaderSize

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@640572 13f79535-47bb-0310-9956-ffa450edef68

Use Rainer's improved patch for bug 44633

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@640559 13f79535-47bb-0310-9956-ffa450edef68

Clean-up unused import

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@640551 13f79535-47bb-0310-9956-ffa450edef68

Fix bug 44633. Tell use which class failed to load due to wrong version number.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@640451 13f79535-47bb-0310-9956-ffa450edef68

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=44646
The problem was wider than the issue described in the bug report. The session listener and the valve are different objects so the connection list wasn't visible to the session listener. I couldn't see an easy way to make them the same object so I used two lists - one for the valve (stored in the valve) and one for the session (stored in the session).
This valve now works for a simple comet app.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@640273 13f79535-47bb-0310-9956-ffa450edef68

maxActive setting for unlimited changed in commons-pool > 1.2

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@639842 13f79535-47bb-0310-9956-ffa450edef68

No longer need to rename jars for CGI or SSI.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@639771 13f79535-47bb-0310-9956-ffa450edef68

There is no admin app in 6.x

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@639198 13f79535-47bb-0310-9956-ffa450edef68

Only allow version switching on the "value" of the cookie, for the rest, just quote like we did in the past. this becomes the most backwards compatible with old behavior

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@638695 13f79535-47bb-0310-9956-ffa450edef68

Typo.

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@637817 13f79535-47bb-0310-9956-ffa450edef68