Fix remaining warnings in javax package (with recently documented settings)

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

Document the Eclipse compiler errors/warnings settings I am using

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

Avoid a possible NPE that in turn was masking the root cause from JspC

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

Fix massive memory leak in NIO connector. Http11NioProcessor objects were not being recycled.

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

https://issues.apache.org/bugzilla/show_bug.cgi?id=13731
Make variables in <code>_jspService()</code> method final where possible.

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

Starting to work on maxConnections attribute for BIO/NIO connectors to allow administrators to throttle how accepting
connections is handled. Implement a CounterLatch to keep track of the
connection count while also allowing the acceptor
thread block while the max has been reached

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

bug 48822: Include contest name in reload and stop log statements
https://issues.apache.org/bugzilla/show_bug.cgi?id=48822

changelog + proposed for tomcat 6

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

bug 48822: Include contest name in reload and stop log statements
https://issues.apache.org/bugzilla/show_bug.cgi?id=48822

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

Fix Findbugs issues / Eclipse warnings.
Some genuine issues
Some changes to make code easier to read (for humans and analysis tools)

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

Fix a FindBugs nag in the unit tests

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

https://issues.apache.org/bugzilla/show_bug.cgi?id=10972
Improve error message if the className attribute is missing on an element in server.xml where it is required.

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=10526
Add alwaysUseSession option to authenticators

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

Whilst I am thinking about excluding stuff from FindBugs - add some exceptions for the generated code with issues and fix the remaining issues in the same package

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

Add a FindBugs filter config for known false positives

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

Fix various warnings (FindBugs, Eclipse etc.)

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

Fix another FindBugs niggle

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

Remove unnecessary override

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

Fix my stupid mistake.

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

Fix a typo

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

Added documentation to parseBodyMethods and parseBodyMethodSet members.

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

Fixed bug #49125: toString on byte[] array probably is not correct
- Added HexUtils.toHexString method
- Wrapped output of byte[] objects with a call to HexUtils.toHexString

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

Log mis-configuration in context.xml

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

Aliases were broken if the context was reloaded

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

Fix various FindBugs and Eclipse warnings

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

Using abstract classes means impl isn't clear in the startup log. Modify name so it is clear.

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

Improve the fix for https://issues.apache.org/bugzilla/show_bug.cgi?id=50201
- Use AtomicReference to guard against possible duplicate registration of
  listeners when logAccess() is called by several threads at the same time.
- Added install() and uninstall() method to AccessLogListener
  to help register/unregister the listeners.
- When defaultAccessLog is nulled, always unregister the listener as well.
  The listener does not need to be reused, because now we do create a listener
  when there is no log and it will take care of things.

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

changelog for r1044987

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=50460
Avoid leak caused by using a cached exception instance

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

Use i18n for the messages
It is followup to r1042022 which was a fix for BZ 48973

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

Add some tests for bug50449 but haven't reproduced it yet

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

Fix FindBugs issues in this class

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

Fix FindBugs issues in this class

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

i18n changes for r1044874

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

Use the newly refactored session id generator when generating sso session IDs

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

Re-factor session ID generation into a separate class so it can be re-used for SSO (patch for that to follow)

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

Fix FindBugs issues in this class

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

bug 50459: StandardContext.bindThread() and unbindThread() are not symmetrical and not limited to current thread
https://issues.apache.org/bugzilla/show_bug.cgi?id=50459

Fixed incorrect comment in code

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

bug 50459: StandardContext.bindThread() and unbindThread() are not symmetrical and not limited to current thread
https://issues.apache.org/bugzilla/show_bug.cgi?id=50459

This fixes the bug that was introduced by http://svn.apache.org/viewvc?rev=1044145&view=rev (introduced a dedicated thread when starting and stopping web applications to avoid mem leak)

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

TRy and sync up the deps in the POMs with what Checkstyle validates

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

Tribes should only depend on JULI

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

o.a.tomcat.jni should not have any external dependencies
OK for o.a.naming to have dependencies

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

o.a.tomcat.util.net should not depend on o.a.coyote

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

o.a.tomcat.util.net should not depend on o.a.catalina

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

Get the checkstyle configs to play nicely with Ant and the Eclipse plug-in

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

Add import-control checks to Checkstyle (these found the recent dependency issues)
Move checkstyle config to res/checkstyle to stop it cluttering up the root

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

javax.el should not depend on javax.servlet.jsp.el

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

Coyote should not depend on Catalina

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=50448
Regression caused by http://svn.apache.org/viewvc?view=revision&revision=1036281

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

Avoid duplicate constant definition.
Use Constants defined by the Servlet/JSP specs where appropriate
Use Constants in preference to local Strings

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=50413
Ensure 304s are not returned when using static files as error pages

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

Missed file in r1044255

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

Fix FindBugs / unused code warnings

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

Fix Eclipse warnings in package

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

Fix Eclipse warnings in package

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

Remove redundant interface definitions and fix other warnings (missing @Overrides) in same files

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

Grrr. Missed a couple of @Override markers

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

Fix missing @Override markers in Jasper and remove an unnecessary null check

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

bug 49159: Improve ThreadLocal memory leak clean-up
https://issues.apache.org/bugzilla/show_bug.cgi?id=49159

Use a dedicated thread when calling web application code when it is started and stopped (calls to Listeners, Filters, Servlets).

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

Register the correct object after the refactoring

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

Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=50360
Add an option to control when the socket is bound

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

mserver != null will never be true if the mbean was registered. Drop the check since the unregister method will handle things correctly anyway

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

Re-factoring in support of https://issues.apache.org/bugzilla/show_bug.cgi?id=50360
Log the right message at the right point

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

ignore work and logs directories as discussed on dev list

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

bug 49159: Improve ThreadLocal memory leak clean-up
https://issues.apache.org/bugzilla/show_bug.cgi?id=49159

some formatting after discussion over the list (operator at the end of line, unnecessary cast)

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

Re-factoring in support of https://issues.apache.org/bugzilla/show_bug.cgi?id=50360
Move ad-hoc attribute handling from the ProtocolHandler to the Endpoint and expose the Endpoint to ServerSocketFactory objects so they can retrieve additional configuration if required. All default components now use explicit configuration rather than relying on the ad-hoc mechanism.
Ensure the hooks for alternative SSL implementations are in place (were partially removed).
Remove the hooks for alternative ServerSocketFactory implementations (had been hard-coded / half implemented for some time).
Make the sslEnabledProtocols attribute work with BIO as well.

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

- Fixed brace and paren placement
- Changed Connector.isParseBodyMethod from public -> protected
- Relaxed language claiming that use of parseBodyMethods may "violate" the servlet spec
- Moved error message to LocalStrings.properties (Translations requested)

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

Removed a little more unused code

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

Re-factoring in support of https://issues.apache.org/bugzilla/show_bug.cgi?id=50360
ProtocolHandler should register itself with MBean server rather than have the connector do it.

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

Remove some unused code
Fix some FindBugs niggles

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

Re-factoring in support of https://issues.apache.org/bugzilla/show_bug.cgi?id=50360
Pull up start()

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

Re-factoring in support of https://issues.apache.org/bugzilla/show_bug.cgi?id=50360
Move MBean registration from start() to init(). More MBean refactoring expected

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

Fix copy/paste error

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

Re-factoring in support of https://issues.apache.org/bugzilla/show_bug.cgi?id=50360
Pull up init()

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

Align BIO+JSSE with NIO+JSSE

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

Fix indents

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

JSSE will always be available in Tomcat 7

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

Re-factoring in support of https://issues.apache.org/bugzilla/show_bug.cgi?id=50360
Align the init() methods (JSSE still to go)

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

Nice idea, but need i18n for the actions so each needs its own string

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

Re-factoring in support of https://issues.apache.org/bugzilla/show_bug.cgi?id=50360
Pull up stop()

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

Re-factoring in support of https://issues.apache.org/bugzilla/show_bug.cgi?id=50360
Pull up pause() and resume() methods

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

Remove unused welcome file handling code

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

Remove unused code

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

Remove unused code

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

Remove unused code.
Make spacing consistent.

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

Jasper should not depend on Catalina
Re-use constants where possible
Fix a FindBugs niggle

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

Another Findbugs niggle

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

Fix some FindBugs warnings - all unclosed streams. These could potentially lead to locked files and maybe memory leaks.
Fix some other simple FindBugs/Eclipse warnings in affected files.

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

Fix a couple of 'high priority' FindBugs warnings

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

Add checkstyle rule for modifier order and fix remaining issues in current code base

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

Fix modifier order in org.apache.catalina.*

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

Fix modifier order in org.apache.naming.*

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

Fix modifier order in org.apache.tomcat.*

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

Fix modifier order in org.apache.jasper.*

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

Fix modifier order in org.apache.el.*

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

Fix modifier order in javax.*

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

Another day, another check

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

Typo

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

Re-factoring in support of https://issues.apache.org/bugzilla/show_bug.cgi?id=50360
Pull up the concrete destroy() method - required a fair bit of refactoring
Pull up the abstract init() method()

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

Re-factoring in support of https://issues.apache.org/bugzilla/show_bug.cgi?id=50360
Add the necessary plumbing for the AbstractProtocolHandler to perform the JMX registration

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

Fix auto-completion snafu

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