Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=51940
Don't limit saving of request bodies during FORM authentication to POST requests since any HTTP method may include a body.
Based on a patch by Nicholas Sushkin

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=51972
Add support for handling scheme/protocol relative URLs for redirects

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

Make protected to it is visible to unit tests

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

Add a performance test that demonstrates why using URI is currently poor for performance

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

After some testing with DIGEST auth, this approach is slower

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

Add initial implementation of a thread safe MessageDigest helper class

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

Focus DIGEST auth performance test on authentication process.

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

Fix various IDE warings
- add missing @Override
- remove unused code
- avoid auto-(un)boxing
- fix some generics issues

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

ignore *.iml

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=51952
Make inclusion of a response body for redirects optional as it may cause
issues and is only SHOULD in RFC2616. See also
https://issues.apache.org/bugzilla/show_bug.cgi?id=41718

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

Resolve a warning about unboxing operation. Perform unboxing explicitly.

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

Resolve "Redundant null check: the variable cannot be null at this location" warning.
The JspC constructor cannot throw a JasperException.

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

Fixed bug #51956: RemoteAddrFilter uses remote host (not address) in doFilterEvent

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

rat-excludes.txt file for Buildbot
INFRA-3948

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

Do not sort lists that have only one element

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

List jars in alphabetical order.

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=51630
Correct bug in async examples. Remove unnecessary call to
AsyncContext.complete() that triggered an ISE.

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=51881
Don't mark processors handling comet requests as non-comet too early.
Before this fix, finishing a comet request was processed as non-comet meaning the comet clean-up code was not executed which was likely to break processing of the next request on the connection.

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

Javadoc for CometEvent: Add <br> tags to make it more readable.

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

Correct markup that looks odd with <code> formatting.

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

https://issues.apache.org/bugzilla/show_bug.cgi?id=50923
Use more distinct color for <code> and add small spacing around it.
Do not color attribute names (the names in the first column in attribute lists in Configuration reference).

Commented-out unused xsl:variable.

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

Remove unused parameters

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

Removed parameter from get()

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

Remove unused method

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

Remove unused parameter

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

Remove unused parameter

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

Remove unused parameter

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

Remove unused parameter

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

Remove unused method

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

Remove unused throws declarations

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

Fix consequences of removing throws

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

Remove unnecessary throws

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

Merge branch 'bug51910' into trunk

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

Remove one more unneeded line. Correct punctuation.

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

Remove duplicate wrong line that should have been deleted in r535325. See the same text in the JavaDoc of the CometEvent class.

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

Remove unused code.
The code to close comet connections has changed over the years and this
bit no longer does anything.

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

A bit more memory-efficient code.

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=51912
Fix several small issues in Header processing in Nio HTTP connector:
- Split HEADER_VALUE state into HEADER_VALUE_START and HEADER_VALUE.
The first one skips leading whitespace, the second one - does not.
- Trim trailing tabs as well as spaces. RFC-2616 Ch.4.2 says that both are LWS and both can be ignored.
- Always trim whitespace from last line.
- Fix initial value of lastSignificantChar when calling skipLine(). It fixes BZ 51912.

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

No functional change.
Simplify code and add comments:
- unwrapped nested if().
- s/while()/if()/ where it has only one iteration.

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

Improve MimeHeaders.toString() (is used only for debugging):
if there are several headers with the same name, print all of them.

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

Improve TestNonBlockingCoordinator:
- Format code for better readbility.
- In testCoord1() change how equality is tested. Read expected value from 0th member, instead of (i-1)th. They all should be the same.
- Remove unneeded main() method.

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

Improve TestGroupChannelMemberArrival
1. Add synchronization when accessing TestMbrListener.members.
This should fix the failures observed by Gump.
2. Remove unused method #clear().
3. Improve messages printed to System.out during the test:
- Add timestamps
- Add hash code of threads (because thread names are not unique: They all are either "Membership-MemberAdded." or "Membership-MemberDisappeared.").
- Add current length of members list
4. Print messages when memberAdded()/memberDisappeared() performed no work.
That is if notification arrives twice for the same Member. I have not observed such events when running the tests.

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

In the examples application:
Add links to the FORM Auth, Servlet 3.0 Async and Comet examples.

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=51905
Skip attempt to shutdown AprEndpoint acceptor threads if the server socket has been already forcefully closed.
Reduce waiting timeout from 30s to 10s. I think if things go wrong 10s is enough to wait.

If it were waiting 30s
- the JVM itself might be forcefully closed by service wrapper as unresponsive.
- it must print something to the logs, otherwise it looks odd.

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=51872
Ensure access log always logs the correct remote IP.
Ensure requests with multiple errors do not result in multiple access log entries.

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

ContainerServlets are always restricted.

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

Add to the Tomcat 8 TODOs

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

Fix a FindBugs warning

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

Fix some FindBugs warnings

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

Remove unused code

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

Improve RUNNING.txt by adding the following:
- Mention that CATALINA_HOME should have a correct value or be absent (BZ 51806)
- Mention environment variables that are used by the scripts.
- Document what is left in $CATALINA_HOME when $CATALINA_BASE is used.
- Document setenv.* scripts.

Feedback and better wording are welcome.

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

Add thread name to OneLineFormatter.

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=50923
Use slightly different color for <code> tags

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

There is a caveat when using RemoteAddrValve with IPv6 addresses
- see thread "tomcat 7.0.21: bug in RemoteAddrValve?" of 2011-09-14 on users@
Document it and update configuration examples in manager and host-manager apps.
Add usage examples to valve.html, filter.html.

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

Use slightly more correct markup:
<h1> is block element, while <font> is inline element

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

Update version number for Tomcat 8.

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

Remove unused webapps/docs/tribes/tomcat-docs.xsl

The build.xml that does "xslt basedir="webapps/docs/tribes"" uses
the common webapps/docs/tomcat-docs.xsl instead.

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

Update version number to Tomcat 8, where it was added by r962466

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

Don't send 503 if endpoint is paused and we
just answered an AJP CPING.

If CPING works, mod_jk expects the main request
to work too. If we don't want to process the
main request we need to fail the CPING received
directly before.

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

Add 503 return in process() when endpoint is paused.
This is consistent with BIO and NIO.
Some more whitespace and other non-functional
changes to align with AjpNioProcessor.

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

Whitespace change to align with AjpAprProcessor.
No functional change.

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=51871
Wrong transitive dependency listed in tomcat-jdbc maven pom
It should depend on tomcat-juli, not on itself.

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

Cleanup temporary files created by tests:
register them and delete them in tearDown().
In TestRegistration also aligned mkdir calls with their usage elsewhere.

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

Reduce visibility of static field ManagerBase.name and make it final.
We already had a bug being introduced in r1174181 because of this field being assignable.

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

Correct a typo

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=51887
Use an insecure random source for session ID generation during tests for speed.
Based on kkolinko's suggestion.

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

Fix Javadoc typo.

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

Break from AJP CPING handling if endpoint is
already paused.

Experimenting shows, that we will otherwise
loose the following request. TC will not
answer it, but mod_jk doesn't resend if properly
configured. CPING is meant to detect this.

Not 100% sure about the call to recycle(),
but didn't find where else it would happen.

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

Do not try to delete the whole output/test-tmp subdirectory when running tests,
because that wouldn't succeed (at least on Windows).

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

Add Override annotation to recently added methods.

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

Use setters when cloning attributes.
Should be more safe in most cases.

Example: setting name directly was wrong,
because name in base class was something else
then in the sub classes.

Not using the setter for maxActiveSessions,
because it will also call property change
listeners.

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

Ignore warnings in classes generated with JavaCC/JJTree

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

Add links to @SuppressWarnings documentation for Eclipse IDE.

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

Fix SSL + BIO + Java 7
The implementation of InputStream.read(byte[0]) has changed so it always returns zero without checking for EOF. This broke the old way of doing things.

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

Use the StringManager for i18n

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

Fix FindBugs warning

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=51862
bug 51862: JreMemoryLeakPreventionListener enhancement to load configurable classes
https://issues.apache.org/bugzilla/show_bug.cgi?id=51862

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

Fix Eclipse warnings

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

Remove some unused code

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

Suppress warning that is hard (impossible?) to avoid

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

Remove unused code

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

Remove unused code

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

Remove unnecessary code

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

Revert r1174322 and apply the correct fix

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

Fix a small number of Eclipse warnings.
 - Unused code
 - Unnecessary null checks

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

Fix Eclipse warning - unused throws

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

Fix docs typo.

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

Fix running under a security manager

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

Replace explicit Tomcat version with a template - for Tomcat 8.

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

Followup to r1173630
Simplify code: methodName can be tested once per iteration

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=51860
Fix issues when using NIO, SSL and a custom SSLImplementation.
Based on a suggestion by Roman Tsirulnikov.

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=51852
Correct 2 issues in varargs handling
 - Incorrectly constructed varargs arguments (resulting in
ArrayIndexOutOfBoundsExceptions)
 - Incorrectly detected matching varargs methods
Patch (with a test case!) provided by Matt Benson

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

Simplify code in NioEndpoint$Poller.
The events.size() call is costly, because it is a loop that counts entries in a linked queue.
Actually it is not needed here at all, because we iterate the queue by ourselves.

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

followup r1173088 and r1173090.

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

Fixed bug #51580: Severe error deploying WAR application (ExpandWar error)
- Added error mesage when JarEntry InputStream cannot be found due to a broken WAR file

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

Fixed bug #51687: Improve JreMemoryLeakPreventionListener against leak caused by sun.java2d.Disposer
- Added (optional) protection against sun.java2d.Disposer thread pinning a WebappClassLoader into memory in the JreMemoryLeakPreventionListener.

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

Modify test to account for behaviours of different platforms and different connectors.

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

Ensure that when the connector is stopped that a Comet END event is processed to signal to the app and to close the socket

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

5s is plenty when things are working and 10s is too long when they aren't.

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

Revert accidental commit

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