Add in some debug, what is causing some TCK failures is a request/response pair not being recycled properly

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

Adjust startAsync().complete() behavior

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

Clean up
 - Eclipse warnings
 - Unnecessary code
 - Line length
 - i18n message names

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

Adjust examples to the latest implementation

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

Mark example servlets as async

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

Mark ResourceEntry.loadedClass as volatile.
That is because WebappClassLoader#findClassInternal() uses double-checked locking to initialize this field.

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

Missed a file in r927850

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

Improve log message when deploying from configBase to make it clearer where xml file is read from.
Javadoc tweak

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

spec says that "started" is defined as started until the thread has returned to the container.

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

Remove tests rendered unnecessary when context.xml files were moved from appBase to configBase

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48992
Fix XML tags in examples for CombinedRealm and LockOutRealm
Based on the patch by Fumiyuki Shimizu

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

Obey the rules on quoting

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

Re-fix https://issues.apache.org/bugzilla/show_bug.cgi?id=45015
Regression in recent parsing re-factoring

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

Add an additional test case for bug 45015.

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

Remove unused imports

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

prevent duplicate threads processing the same socket

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

Ignore setenv.* so I don't accidently check it in

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

Update project name

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

Address various class-loader deadlock / sync issues
https://issues.apache.org/bugzilla/show_bug.cgi?id=44041
https://issues.apache.org/bugzilla/show_bug.cgi?id=48694
https://issues.apache.org/bugzilla/show_bug.cgi?id=48903

Whilst parallel class-loading would be a nice feature, the various issues that have emerged have demonstrated that anything other than synchronized(this) is likely to cause issues.

Parallel class-loading will be explored for Tomcat 7 (disabled by default) and ported back to 6.0.x when proven to be stable.

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

Clean up. The setAsync can be done in a single bottleneck, instead mixed into the processor

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

TCK failures: Make valve Async aware - with some help from Filip

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

recycle properly

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

Enable async behavior for the AJP connector

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

Prevent looping by correctly setting the socket wrapper async state

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

Make sure start() and stop() are called on Wrappers. This was a regression in the recent Lifecycle changes.

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

Use context path in debug log message

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

Add in a slightly different behavior for dispatch.
dispatch() will return immediately, according to spec. But if dispatch is called from the tomcat worker thread, then the dispatch will not happen until the worker thread has backed out of the call. This prevents multiple threads being launched into the same request/response and causing potential concurrency issues.
The Async state machine guarantees that multiple dispatch can not be called.

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

Tracking down a memory leak. Add some debug logging to the mapper.

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

Add support for detecting Spring Security user names in the manager app

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

Simpler fix for https://issues.apache.org/bugzilla/show_bug.cgi?id=48790 based on a patch by kkolinko
Make maxActive thread safe. Probably unnecessary but technically a bug.

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

Revert fix for https://issues.apache.org/bugzilla/show_bug.cgi?id=48790 in preparation for a simpler solution

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

TCK failures: Fix session expiry on cross-context requests. Need to test the local session is valid before calling access() which will make it valid again.

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

TCK failures: Don't delegate dispatcher type to wrapped request as it may well be different.

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

TCK failures: Look for the right element

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

TCK failures: Make sure FilterDefs where async-support was not defined are set to the default of false before being added to the context

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

TCK failures: Servlets do not support async by default

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

SRV.9.3 says we must throw a FNFE

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

Provide the method expected by the digester for adding security-role-ref elements

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

More async stuff, implement timeout handling

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

Make method names consistent with the names the digester looks for

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

Fix some TCK issues. The behaviour is undefined in the spec, so go with the TCK requirements

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

Fire listeners in reverse order for requestDestroyed

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

Servlets added through the new ServletContext.addServlet(...) methods may not be initialised

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

It is valid to call startAsync() inside a dispatched request

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

Always use the InstanceManager to create listeners, filters & servlets

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

Arrays indexes start at zero so no need for "... + 1"

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

Doh! Return the newly created registrations rather than just creating work for the garbage collector.

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

Fix various exceptions that prevented one of the TCK web applications from starting

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

Fix a couple of ASync issues that were causing Servlet 3.0 TCK test failures.
1. <quote section="2.3.3.3" page="17">
The complete method can be invoked by the container if the request is dispatched to a servlet that does not support async processing, or the target servlet called by AsyncContext.dispatch does not do a subsequent call to startAsync. In this case, it is the container's responsibility to call complete() as soon as that servlet's service method is exited.
</quote>
2. <quote section="2.3.3.3" page ="13">
public boolean isAsyncStarted() - Returns true if async processing has started on this request, and false otherwise. If this request has been dispatched using one of the AsyncContext.dispatch methods since it was put in asynchronous mode, or a call to AsynContext.complete is made, this method returns false.
</quote>

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

More async stuff, only timeout left

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

https://issues.apache.org/bugzilla/show_bug.cgi?id=48934
The wrong sender was disconnected, should only be the sender holding the actual TCP connections.

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

Use better variable name.

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

Use TEMP environment for storing the autoresponse file, and make sure the file was created

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

correct a comment

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

Use real exit value if shutdown via stop

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

Suppress anoying Terminate batch job prompt when hitting CTRL+C. Note however that it leaves the file named yes in the bin directory

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

Re-work the script to make it more Gump friendly

No functional change for normal builds, but allows skipping a taskdef that Gump doesn't want or need when running tests.

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

Alternative fix for bug 48795. Add a new property to control if the next request always triggers recompilation after a compilation failure. Defaults to false and only applies in development mode

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

Mark the min(a,b) method as static.
Inspired by https://issues.apache.org/bugzilla/show_bug.cgi?id=44265

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

Revert 921464 for rework

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

Update to NSIS 2.46

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

Fix download of commons-daemon-1.0.2:
The "destfile" parameter has to be specified for "downloadzip" task.
It specifies the file which absence triggers the download of the archive.
Thus, without it the archive was redownloaded again and again with each build.

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48795
If compilation fails, always try to re-compile the next time around

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48793
Make catalina.sh more robust to different return values on different platforms
Patch provided by Thomas GL

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

Remainder of fix for https://issues.apache.org/bugzilla/show_bug.cgi?id=48379
Allow session cookie name to be configured per context
With this option, the servlet 3 options and system property there were just too many places this was being configured so the system property option has been removed for Tomcat 7.

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

Remove some changes that have since made it into 6.0.x

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

Partial fix for https://issues.apache.org/bugzilla/show_bug.cgi?id=48379
Allow session cookie path to be configured per context
With this option, the servlet 3 options and Connector.emptySessionPath there were just too many places this was being configured so the Connector option has been removed for Tomcat 7.
Based on a patch by Brane F. Gracnar

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48258 for TC7
Allow session cookie domain to be over-ridden by context configuration in the same way httpOnly may be
Based on a patch by Donn Aiken

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48790
Make maxActive thread safe. Technically a bug but seems like overkill.

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

Move the 48nnn bugs

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

Too many 48nnn bugs in context root

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

Bug 48701 test cases

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

Complete fix for https://issues.apache.org/bugzilla/show_bug.cgi?id=48701

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

Make StandardContext aliases work with ServletContext.getResourcePaths()

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

Re-org some JSPs

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

Create a directory for all the bug 45nnn JSPs to make things more manageable. I'll do the same for other groups of bugs as the numers grow.

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

https://issues.apache.org/bugzilla/show_bug.cgi?id=48007#c5
Improve exception processing in CustomObjectInputStream#resolveClass(),
to help find the cause behind BZ 48007.

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

Review feedback
1. getNameFromAttribute() returns the attribute name, not the variable name and
2. This is a translation time test so the attribute value is not known

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48701
Add a system property to allow disabling enforcement of a requirement of JSP.5.3.
The specification recommends, but does not require us to enforce it.

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

Correct description for STRICT_QUOTE_ESCAPING system property
C-T-R

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48760
Remove potential race condition that can result in multiple threads trying to use the same InputStream

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48737
Don't assume paths that start with /META-INF/... are always in JARs. This is not true for some IDEs
Patch provided by Fabrizio Giustina

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48729
Return roles defined by both userRoleName and roleName mechanisms
Patch provided by 'eric'

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

Remove unnecessary duplication

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

Eclipse warnings

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

Updated the tests because r920110 was reverted in r920608.

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

Revert r920110
Compatibility with JSP 1.2 tag libraries had to be covered by JSP 2.0 specification,
see "Backwards Compatibility with JSP 1.2" in the Preface part of JSP 2.0 specification,
and there is no provision for this feature.
Discussed in the Re: r920055 thread on dev@

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

Amendment for BZ 48668 fixes.
Use setter methods that accept String value to set pageInfo properties.
Throw an exception if tagInfo is not available for a tag file or requiredVersion is not parseable. (Both of that should not happen).

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

JavaDoc correction
C-T-R

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

Revisit https://issues.apache.org/bugzilla/show_bug.cgi?id=48701
Allow TagVariableInfo as well as VariableInfo to introduce objects later used by <jsp:getProperty> - JSP.5.3

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

Whitespace

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48661
If the response has been committed, include the error page like Jasper does

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48629
Make nested role search work with username as well as DN
Add roleNested to the docs
Patch provided by Felix Schumacher

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

Use the Lifecycle state to dtermine if the context is in the correct state to allow configuration

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

more work towards making the JIO connector ready for async

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

Add context name to leak detection log messages

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

Handle component failure without throwing a whole stack of exceptions
Adds a new permitted transition from NEW to STOPPED that does not fire any events

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

No need for separate ant download

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

Update change log

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

Lifecycle refactoring - ContainerBase
Adds a new CONFIGURE event to allow Context to fire the START event at the right time
Context fires START a little later
ReplicatedContext takes advantage of LifecycleBase ensuring start() is only called once

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