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

Fix intermittent test failures

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

isELIgnored depends on library version and web.xml declaration

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

Rename no-el to el-as-literal
Add tests for 2.3, 2.4 & 2.5 webapps with Tag libraries that require various JSP versions
These currently all work and I think some should fail - looking into that next

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

Test that tags from libraries that declare various JSP versions have EL and deferred EL enabled/disabled as appropriate.
This currently triggers a failure - fix to follow shortly.

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

Extend the no-el test to cover deferred expressions

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

Both TLD and web.xml determine if deferred EL syntax is treated as EL or as a literal

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

Line length

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

Make getVersion(), getMagorVersion() and getMinorVersion() return consistent values

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

Ensure result of getVersion() is consistent with getMajorVersion() and getMinorVersion()
With the current code this fails. Patch to fix this to follow.

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

Remove the TestAll suite. Ant doesn't use it, IDEs don't need it and it is often out of sync with the tests

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

Check all versions are correctly parsed

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

Add a test the EL doesn't work if the web-app is a 2.3 web-app

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

Rename webapp to allow tetsing with webapps with different servlet spec versions in web.xml

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48668
Fix remaining issues in BZ48668
The idea behind this change is to make ELParser aware about isDeferredAsLiteral option.
Before this change ELParser was used to parse an attribute regardless of isELIgnored or isDeferredSyntaxAllowedAsLiteral values. With this change we do not use ELParser when isELIgnored is true and ELParser does not parse '#{' in expressions when isDeferredSyntaxAllowedAsLiteral is true.
It simplified the code in many places.
Also, servlet specification version from web.xml and JSP specification version from TLD file are now taken into account when determining the default values for isELIgnored and isDeferredSyntaxAllowedAsLiteral. As far as I understand the code, previously only isELIgnored was determined by the servlet specification version.

TstParser.java, bug48668a.jsp:
I reenabled the tests that now pass with these changes applied.

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48668
Honor isELIgnored and isDeferredSyntaxAllowed in the Parser

Uncommented the test cases in TestParser/bug48668a.jsp that now are passing.
The ##12,13,16,17 remain commented, because they are still failing.

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

Additional tests for https://issues.apache.org/bugzilla/show_bug.cgi?id=48668
The TestCase currently passes:
the parts that do not pass currently are commented-out:
- in bug48668a.jsp: those that prevent JSP from being compiled
- in TestParser.java: also those that provide wrong output.

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

remove extra newline character

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

Shave a few seconds of the rebuild time if the manifests don't need to be updated

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

Add comment to the tomcat-users.xml template used by the Tomcat exe installer on Windows.

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

Add comment about the "manager" role (copied from TC 5.5)

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

Add a comment about comment to tomcat-users.xml

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

trivial: correct a link on an image on the jsp-examples index page

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

Lifecycle refactoring - Manager
Lifecycle.START event fired too early in a number of cases
Added toString() for use in Lifecycle error messages

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