Re: GSOC 2010

2010-02-12 Thread Mark Thomas
On 12/02/2010 07:21, buddhika chamith wrote:
> I am Buddhika Chamith, a final year student from University of Moratuwa, Sri
> Lanka. I am hoping to participate in GSOC 2010 program. This would be my
> second GSOC. Last year I worked in and successfully completed Apache ODE
> project titled "Apache ODE integration in BPELUnit". [1]
> 
> I would be willing to work on Apache Tomcat project this year if you are
> planning put any ideas forward. I have experience in developing web
> applications with JSP(which I think would be beneficial) and have some
> understanding of tomcat internals. (I went through some of the architecture
> documentations).

Great! It is always good to see people interested in Tomact.

> So are there any project ideas being considered as yet for Tomcat this year?

Not yet. We need to kick off a discussion on this.

> I saw last year projects ideas (regarding JMX, Valves etc). Also are there
> any specific areas that I have to be aware in order to work with Tomcat?

I can't speak for any other potential mentors (and assuming that some of
the Tomcat ideas are accepted for GSOC), but I will not be accepting
applications from a student unless they have already provided an
acceptable patch for at least one and preferably more currently open,
non-trivial bugzilla items (bugs or enhancements).

Mark



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Tomcat Wiki] Update of "GSOC" by markt

2010-02-12 Thread Apache Wiki
Dear Wiki user,

You have subscribed to a wiki page or wiki category on "Tomcat Wiki" for change 
notification.

The "GSOC" page has been changed by markt.
http://wiki.apache.org/tomcat/GSOC?action=diff&rev1=2&rev2=3

--

  The Tomcat project participated / is participating in the following Google 
Summers of Code:
   * [[SummerOfCode2005]] - 2005
   * [[SummerOfCode2009]] - 2009
+  * [[SummerOfCode2010]] - 2010
  

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Minor Error in Documentation

2010-02-12 Thread Gurkan Erdogdu
Hello folks;

I think that there is a minor error in Tomcat 6 documentation (
http://tomcat.apache.org/tomcat-6.0-doc/monitoring.html). It says that to
setup JMX remote access you have to add

 set CATALINA_OPTS="-Dcom.sun.management.jmxremote \
-Dcom.sun.management.jmxremote.port=%my.jmx.port% \
-Dcom.sun.management.jmxremote.ssl=false \
-Dcom.sun.management.jmxremote.authenticate=false"

into tomcat startup script. Client does not connect to JMX after adding
above setting because quotations marks are put around CATALINA_OPTS=*"* bla
..bla *".  *If you remove "" everything works as expected.

Thanks;

--Gurkan


[Tomcat Wiki] Update of "SummerOfCode2010" by markt

2010-02-12 Thread Apache Wiki
Dear Wiki user,

You have subscribed to a wiki page or wiki category on "Tomcat Wiki" for change 
notification.

The "SummerOfCode2010" page has been changed by markt.
The comment on this change is: Add some initial ideas.
http://wiki.apache.org/tomcat/SummerOfCode2010

--

New page:
= Google Summer of Code 2010 =

== Important Notes ==
The application process has not yet started. This page is to record possible 
projects assuming the ASF and the Tomcat project does participate in this 
year's summer of code. The actual list of projects will be determined as
the application process progresses.

Anyone is free to add ideas to this list. If you are not a Tomcat committer 
please do not add a committer as a potential mentor for a project unless they 
have agreed to do so.

== Ideas ==

||Title||Mentor||Description||
||JMX||markt||Update the JMX descriptors to more accurately reflect the 
properties and methods of the underlying objects, taking particular care about 
which attributes should be read-only and which read-write. Then add additional 
functionality as necessary to enable a complete Tomcat instance to be 
configured entirely via JMX (ie start with a Server with no config and create 
everythins via JMX).||
||Security docs||markt||Update the security pages etc to include svn revisions 
for each fix, update the release notes to include the CVE reference with the 
fix, update the svn logs to include the CVE reference. This will require a lot 
of cross-checking to ensure that the correct CVEs and svn references are added, 
particularly for the older vulnerabilities. This would include the 5.5.x, 6.0.x 
and 7.0.x code-bases.||
||JSR196||markt||Provide a JSR196 implementation||

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: [Tomcat Wiki] Update of "SummerOfCode2010" by markt

2010-02-12 Thread Mark Thomas
On 12/02/2010 09:08, Apache Wiki wrote:
> Dear Wiki user,
> 
> You have subscribed to a wiki page or wiki category on "Tomcat Wiki" for 
> change notification.
> 
> The "SummerOfCode2010" page has been changed by markt.
> The comment on this change is: Add some initial ideas.

Folks - feel free to add your own ideas and to provide feedback on the
current ones. The sooner we start this discussion, the better prepared
for GSOC2010 we will be.

Cheers,

Mark



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r909321 - /tomcat/trunk/webapps/docs/monitoring.xml

2010-02-12 Thread markt
Author: markt
Date: Fri Feb 12 09:58:13 2010
New Revision: 909321

URL: http://svn.apache.org/viewvc?rev=909321&view=rev
Log:
Remove quotes from example - they break it

Modified:
tomcat/trunk/webapps/docs/monitoring.xml

Modified: tomcat/trunk/webapps/docs/monitoring.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/monitoring.xml?rev=909321&r1=909320&r2=909321&view=diff
==
--- tomcat/trunk/webapps/docs/monitoring.xml (original)
+++ tomcat/trunk/webapps/docs/monitoring.xml Fri Feb 12 09:58:13 2010
@@ -47,10 +47,10 @@
 The following is a quick configuration guide for Java 6:
 Add the following parameters to your Tomcat startup script:
 
-set CATALINA_OPTS="-Dcom.sun.management.jmxremote \
+set CATALINA_OPTS=-Dcom.sun.management.jmxremote \
 -Dcom.sun.management.jmxremote.port=%my.jmx.port% \
 -Dcom.sun.management.jmxremote.ssl=false \
--Dcom.sun.management.jmxremote.authenticate=false"
+-Dcom.sun.management.jmxremote.authenticate=false
 
 
 



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r909322 - /tomcat/tc6.0.x/trunk/webapps/docs/monitoring.xml

2010-02-12 Thread markt
Author: markt
Date: Fri Feb 12 09:58:40 2010
New Revision: 909322

URL: http://svn.apache.org/viewvc?rev=909322&view=rev
Log:
Remove quotes from example - they break it

Modified:
tomcat/tc6.0.x/trunk/webapps/docs/monitoring.xml

Modified: tomcat/tc6.0.x/trunk/webapps/docs/monitoring.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/webapps/docs/monitoring.xml?rev=909322&r1=909321&r2=909322&view=diff
==
--- tomcat/tc6.0.x/trunk/webapps/docs/monitoring.xml (original)
+++ tomcat/tc6.0.x/trunk/webapps/docs/monitoring.xml Fri Feb 12 09:58:40 2010
@@ -47,10 +47,10 @@
 The following is a quick configuration guide for Java 5:
 Add the following parameters to your Tomcat startup script:
 
-set CATALINA_OPTS="-Dcom.sun.management.jmxremote \
+set CATALINA_OPTS=-Dcom.sun.management.jmxremote \
 -Dcom.sun.management.jmxremote.port=%my.jmx.port% \
 -Dcom.sun.management.jmxremote.ssl=false \
--Dcom.sun.management.jmxremote.authenticate=false"
+-Dcom.sun.management.jmxremote.authenticate=false
 
 
 



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r909324 - /tomcat/tc5.5.x/trunk/container/webapps/docs/monitoring.xml

2010-02-12 Thread markt
Author: markt
Date: Fri Feb 12 09:59:09 2010
New Revision: 909324

URL: http://svn.apache.org/viewvc?rev=909324&view=rev
Log:
Remove quotes from example - they break it

Modified:
tomcat/tc5.5.x/trunk/container/webapps/docs/monitoring.xml

Modified: tomcat/tc5.5.x/trunk/container/webapps/docs/monitoring.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/container/webapps/docs/monitoring.xml?rev=909324&r1=909323&r2=909324&view=diff
==
--- tomcat/tc5.5.x/trunk/container/webapps/docs/monitoring.xml (original)
+++ tomcat/tc5.5.x/trunk/container/webapps/docs/monitoring.xml Fri Feb 12 
09:59:09 2010
@@ -47,10 +47,10 @@
 For quick installation you find here a short installation guide:
 Add the following parameters to your tomcat startup script:
 
-set CATALINA_OPTS="-Dcom.sun.management.jmxremote \
+set CATALINA_OPTS=-Dcom.sun.management.jmxremote \
 -Dcom.sun.management.jmxremote.port=%my.jmx.port% \
 -Dcom.sun.management.jmxremote.ssl=false \
--Dcom.sun.management.jmxremote.authenticate=false"
+-Dcom.sun.management.jmxremote.authenticate=false
 
 
 



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: Minor Error in Documentation

2010-02-12 Thread Mark Thomas
On 12/02/2010 08:57, Gurkan Erdogdu wrote:
> Hello folks;
> 
> I think that there is a minor error in Tomcat 6 documentation (
> http://tomcat.apache.org/tomcat-6.0-doc/monitoring.html). It says that to
> setup JMX remote access you have to add
> 
>  set CATALINA_OPTS="-Dcom.sun.management.jmxremote \
> -Dcom.sun.management.jmxremote.port=%my.jmx.port% \
> -Dcom.sun.management.jmxremote.ssl=false \
> -Dcom.sun.management.jmxremote.authenticate=false"
> 
> into tomcat startup script. Client does not connect to JMX after adding
> above setting because quotations marks are put around CATALINA_OPTS=*"* bla
> ..bla *".  *If you remove "" everything works as expected.

Thanks - fixed in trunk, 6.0.x and 5.5.x

Mark



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: Minor Error in Documentation

2010-02-12 Thread Konstantin Kolinko
2010/2/12 Mark Thomas :
> Thanks - fixed in trunk, 6.0.x and 5.5.x

That example is a mix of Windows and Unix syntaxes.

On Windows 1) no quotes, 2) "\" is not supported as continuation character

On Unix  1) "set " is not needed and is indeed a different command.
You just do the assignment. Quotes are needed. 2) %my.jmx.port% will
be ${my.jmx.port}


Best regards,
Konstantin Kolinko

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Tomcat Wiki] Update of "SummerOfCode2010" by JeanFrede ricClere

2010-02-12 Thread Apache Wiki
Dear Wiki user,

You have subscribed to a wiki page or wiki category on "Tomcat Wiki" for change 
notification.

The "SummerOfCode2010" page has been changed by JeanFredericClere.
http://wiki.apache.org/tomcat/SummerOfCode2010?action=diff&rev1=1&rev2=2

--

  ||JMX||markt||Update the JMX descriptors to more accurately reflect the 
properties and methods of the underlying objects, taking particular care about 
which attributes should be read-only and which read-write. Then add additional 
functionality as necessary to enable a complete Tomcat instance to be 
configured entirely via JMX (ie start with a Server with no config and create 
everythins via JMX).||
  ||Security docs||markt||Update the security pages etc to include svn 
revisions for each fix, update the release notes to include the CVE reference 
with the fix, update the svn logs to include the CVE reference. This will 
require a lot of cross-checking to ensure that the correct CVEs and svn 
references are added, particularly for the older vulnerabilities. This would 
include the 5.5.x, 6.0.x and 7.0.x code-bases.||
  ||JSR196||markt||Provide a JSR196 implementation||
+ ||SPDY||jfclere||Create Connector that supports the SPDY protocol 
(Seehttp://dev.chromium.org/spdy/spdy-whitepaper) ||
  

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: Minor Error in Documentation

2010-02-12 Thread Mark Thomas
On 12/02/2010 10:40, Konstantin Kolinko wrote:
> 2010/2/12 Mark Thomas :
>> Thanks - fixed in trunk, 6.0.x and 5.5.x
> 
> That example is a mix of Windows and Unix syntaxes.
> 
> On Windows 1) no quotes, 2) "\" is not supported as continuation character

My assumption was that the continuation character was used to indicate
that the line was broken up over several lines but should be on a single
one. A long single line is going to look horrible in a browser. I agree
this might not be clear to readers.

> On Unix  1) "set " is not needed and is indeed a different command.
> You just do the assignment. Quotes are needed. 2) %my.jmx.port% will
> be ${my.jmx.port}

I think the example is meant to be Windows.

Mark



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



[Tomcat Wiki] Update of "SummerOfCode2010" by Konstanti nKolinko

2010-02-12 Thread Apache Wiki
Dear Wiki user,

You have subscribed to a wiki page or wiki category on "Tomcat Wiki" for change 
notification.

The "SummerOfCode2010" page has been changed by KonstantinKolinko.
The comment on this change is: correct links.
http://wiki.apache.org/tomcat/SummerOfCode2010?action=diff&rev1=2&rev2=3

--

  == Ideas ==
  
  ||Title||Mentor||Description||
- ||JMX||markt||Update the JMX descriptors to more accurately reflect the 
properties and methods of the underlying objects, taking particular care about 
which attributes should be read-only and which read-write. Then add additional 
functionality as necessary to enable a complete Tomcat instance to be 
configured entirely via JMX (ie start with a Server with no config and create 
everythins via JMX).||
+ ||JMX||markt||Update the JMX descriptors to more accurately reflect the 
properties and methods of the underlying objects, taking particular care about 
which attributes should be read-only and which read-write. Then add additional 
functionality as necessary to enable a complete Tomcat instance to be 
configured entirely via JMX (ie start with a Server with no config and create 
everything via JMX).||
  ||Security docs||markt||Update the security pages etc to include svn 
revisions for each fix, update the release notes to include the CVE reference 
with the fix, update the svn logs to include the CVE reference. This will 
require a lot of cross-checking to ensure that the correct CVEs and svn 
references are added, particularly for the older vulnerabilities. This would 
include the 5.5.x, 6.0.x and 7.0.x code-bases.||
- ||JSR196||markt||Provide a JSR196 implementation||
+ ||JSR196||markt||Provide a [[http://jcp.org/en/jsr/detail?id=196|JSR196]] 
implementation||
- ||SPDY||jfclere||Create Connector that supports the SPDY protocol 
(Seehttp://dev.chromium.org/spdy/spdy-whitepaper) ||
+ ||SPDY||jfclere||Create Connector that supports the SPDY protocol (See 
[[http://dev.chromium.org/spdy/spdy-whitepaper]]) ||
  

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: svn commit: r909097 - in /tomcat/trunk/java/org/apache/catalina/loader: LocalStrings.properties WebappClassLoader.java

2010-02-12 Thread Mark Thomas
On 12/02/2010 02:52, Bill Barker wrote:
> 
> 
>  wrote in message
> news:20100211180956.bfd5f2388...@eris.apache.org...
>> Author: markt
>> Date: Thu Feb 11 18:09:08 2010
>> New Revision: 909097
>>
>> URL: http://svn.apache.org/viewvc?rev=909097&view=rev
>> Log:
>> Improved memory leak prevention for resource ResourceBundle



>> +private void clearReferencesResourceBundles() {
>> +// Get a reference to the cache
>> +try {
>> +Field cacheListField =
>> +ResourceBundle.class.getDeclaredField("cacheList");
>> +cacheListField.setAccessible(true);
>> +
> 
> I don't see anywhere in the documentation that says that there is a
> cacheList field.  Have you tried this with a non-Sun JVM?

It works with the 1.6.0 SR7 64-bit JDK on linux from IBM. I haven't
tested any further.

Granted it is undocumented, but most of the memory leak protection
involves using reflection to dig at internals that aren't normally
exposed there is always going to be a risk. This should be fairly same
as we don't need to dig too far to get to what we need.

Mark



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: svn commit: r909097 - in /tomcat/trunk/java/org/apache/catalina/loader: LocalStrings.properties WebappClassLoader.java

2010-02-12 Thread Mark Thomas
On 12/02/2010 12:32, Mark Thomas wrote:
> On 12/02/2010 02:52, Bill Barker wrote:
>>
>>
>>  wrote in message
>> news:20100211180956.bfd5f2388...@eris.apache.org...
>>> Author: markt
>>> Date: Thu Feb 11 18:09:08 2010
>>> New Revision: 909097
>>>
>>> URL: http://svn.apache.org/viewvc?rev=909097&view=rev
>>> Log:
>>> Improved memory leak prevention for resource ResourceBundle
> 
> 
> 
>>> +private void clearReferencesResourceBundles() {
>>> +// Get a reference to the cache
>>> +try {
>>> +Field cacheListField =
>>> +ResourceBundle.class.getDeclaredField("cacheList");
>>> +cacheListField.setAccessible(true);
>>> +
>>
>> I don't see anywhere in the documentation that says that there is a
>> cacheList field.  Have you tried this with a non-Sun JVM?
> 
> It works with the 1.6.0 SR7 64-bit JDK on linux from IBM. I haven't
> tested any further.

I take that back. It looks like I get an error. Not sure why my first
test missed it. I'll see what I can do to handle that more gracefully.

Mark



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



jk/cluster - intelligent systems load

2010-02-12 Thread Henri Gomez
Hi to all,

Some times ago, we discuss about mod_jk/cluster and more intelligent
strategy to be use for the dispatching system.

One point was to get system load informations about the various Tomcat
instances involved in a cluster, to send request to the lowest loaded
Tomcat.

Hyperic Sigar, http://www.hyperic.com/products/sigar.html, seems to
provide such informations for various OS.

Could it be interesting to add such 'extension' to Tomcat, so it could
be able to send back to jk/cluster so they could get the best
decisions ?

New commands could be added to AJP13, as done previously with
cping/cpong, http://tomcat.apache.org/connectors-doc/ajp/ajpv13ext.html.

What do you think about ?

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: jk/cluster - intelligent systems load

2010-02-12 Thread Mark Thomas
On 12/02/2010 15:20, Henri Gomez wrote:
> Hi to all,
> 
> Some times ago, we discuss about mod_jk/cluster and more intelligent
> strategy to be use for the dispatching system.
> 
> One point was to get system load informations about the various Tomcat
> instances involved in a cluster, to send request to the lowest loaded
> Tomcat.
> 
> Hyperic Sigar, http://www.hyperic.com/products/sigar.html, seems to
> provide such informations for various OS.
> 
> Could it be interesting to add such 'extension' to Tomcat, so it could
> be able to send back to jk/cluster so they could get the best
> decisions ?
> 
> New commands could be added to AJP13, as done previously with
> cping/cpong, http://tomcat.apache.org/connectors-doc/ajp/ajpv13ext.html.
> 
> What do you think about ?

I like the general idea.

My personal favourite in terms of implementation is to pass the
information via response headers as it allows interoperability with the
widest possible range of front-end load balancers. The info can be
passed back with every request or only for requests to a particular URL.

Then you get into how you measure load. I don't think that there will be
a one size fits all solution. I imagine that it would need to be fairly
easy for folks to implement their own load measurements. Using response
headers makes that simple to do providing we get the definition of that
header format right.

Mark



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r909525 - /tomcat/trunk/java/org/apache/catalina/loader/WebappClassLoader.java

2010-02-12 Thread markt
Author: markt
Date: Fri Feb 12 17:49:49 2010
New Revision: 909525

URL: http://svn.apache.org/viewvc?rev=909525&view=rev
Log:
Looks like the ResourceBundle leaks are triggered by a GC bug - only seems to 
affect Sun JVMs.
The fix is also Sun specific so only log a debug message if the internal field 
can't be found on a non-Sun JVM (it isn't there for IBM for example)

Modified:
tomcat/trunk/java/org/apache/catalina/loader/WebappClassLoader.java

Modified: tomcat/trunk/java/org/apache/catalina/loader/WebappClassLoader.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/loader/WebappClassLoader.java?rev=909525&r1=909524&r2=909525&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/loader/WebappClassLoader.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/loader/WebappClassLoader.java Fri Feb 
12 17:49:49 2010
@@ -1782,6 +1782,10 @@
 }
 
 // Clear the resource bundle cache
+// This shouldn't be necessary, the cache uses weak references but
+// it has caused leaks. Oddly, using the leak detection code in
+// standard host allows the class loader to be GC'd. This has been seen
+// on Sun but not IBM JREs. Maybe a bug in Sun's GC impl?
 clearReferencesResourceBundles();
 
 // Clear the classloader reference in the VM's bean introspector
@@ -2395,8 +2399,13 @@
 log.error(sm.getString(
 "webappClassLoader.clearReferencesResourceBundlesFail"), 
e);
 } catch (NoSuchFieldException e) {
-log.error(sm.getString(
-"webappClassLoader.clearReferencesResourceBundlesFail"), 
e);
+if (System.getProperty("java.vendor").startsWith("Sun")) {
+log.error(sm.getString(
+"webappClassLoader.clearReferencesResourceBundlesFail"), e);
+} else {
+log.debug(sm.getString(
+"webappClassLoader.clearReferencesResourceBundlesFail"), e);
+}
 } catch (IllegalArgumentException e) {
 log.error(sm.getString(
 "webappClassLoader.clearReferencesResourceBundlesFail"), 
e);



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r909526 - /tomcat/trunk/java/org/apache/catalina/core/StandardHost.java

2010-02-12 Thread markt
Author: markt
Date: Fri Feb 12 17:50:22 2010
New Revision: 909526

URL: http://svn.apache.org/viewvc?rev=909526&view=rev
Log:
Make class loader registration more robust

Modified:
tomcat/trunk/java/org/apache/catalina/core/StandardHost.java

Modified: tomcat/trunk/java/org/apache/catalina/core/StandardHost.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/StandardHost.java?rev=909526&r1=909525&r2=909526&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/core/StandardHost.java (original)
+++ tomcat/trunk/java/org/apache/catalina/core/StandardHost.java Fri Feb 12 
17:50:22 2010
@@ -30,7 +30,10 @@
 import org.apache.catalina.Container;
 import org.apache.catalina.Context;
 import org.apache.catalina.Host;
+import org.apache.catalina.Lifecycle;
+import org.apache.catalina.LifecycleEvent;
 import org.apache.catalina.LifecycleException;
+import org.apache.catalina.LifecycleListener;
 import org.apache.catalina.Valve;
 import org.apache.catalina.loader.WebappClassLoader;
 import org.apache.catalina.startup.HostConfig;
@@ -579,17 +582,33 @@
 throw new IllegalArgumentException
 (sm.getString("standardHost.notContext"));
 super.addChild(child);
-
-// Record a reference to the context's class loader to aid memory leak
-// detection
-if (child.getLoader() != null) {
-childClassLoaders.put(child.getLoader().getClassLoader(),
-child.getName());
+
+if (child instanceof Lifecycle) {
+((Lifecycle) child).addLifecycleListener(
+new MemoryLeakTrackingListener());
 }
 }
 
 
 /**
+ * Used to ensure the regardless of {...@link Context} implementation, a 
record
+ * is kept of the class loader used every time a context starts.
+ */
+private class MemoryLeakTrackingListener implements LifecycleListener {
+@Override
+public void lifecycleEvent(LifecycleEvent event) {
+if (event.getType().equals(Lifecycle.AFTER_START_EVENT)) {
+if (event.getSource() instanceof Context) {
+Context context = ((Context) event.getSource());
+childClassLoaders.put(context.getLoader().getClassLoader(),
+context.getServletContext().getContextPath());
+}
+}
+}
+}
+
+
+/**
  * Attempt to identify the contexts that have a class loader memory leak.
  * This is usually triggered on context reload. Note: This method attempts
  * to force a full garbage collection. This should be used with extreme



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r909528 - in /tomcat/trunk/java/org/apache/catalina/manager: HTMLManagerServlet.java LocalStrings.properties ManagerServlet.java

2010-02-12 Thread markt
Author: markt
Date: Fri Feb 12 17:51:06 2010
New Revision: 909528

URL: http://svn.apache.org/viewvc?rev=909528&view=rev
Log:
Add a find leaks option - with a suitable warning to the manager app (html & 
text)

Modified:
tomcat/trunk/java/org/apache/catalina/manager/HTMLManagerServlet.java
tomcat/trunk/java/org/apache/catalina/manager/LocalStrings.properties
tomcat/trunk/java/org/apache/catalina/manager/ManagerServlet.java

Modified: tomcat/trunk/java/org/apache/catalina/manager/HTMLManagerServlet.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/manager/HTMLManagerServlet.java?rev=909528&r1=909527&r2=909528&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/manager/HTMLManagerServlet.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/manager/HTMLManagerServlet.java Fri 
Feb 12 17:51:06 2010
@@ -218,6 +218,8 @@
 message = start(path);
 } else if (command.equals("/stop")) {
 message = stop(path);
+} else if (command.equals("/findleaks")) {
+message = findleaks();
 } else {
 // Try GET
 doGet(request,response);
@@ -616,6 +618,17 @@
 args[4] = newNonce;
 writer.print(MessageFormat.format(UPLOAD_SECTION, args));
 
+// Diagnostics section
+args = new Object[6];
+args[0] = sm.getString("htmlManagerServlet.diagnosticsTitle");
+args[1] = sm.getString("htmlManagerServlet.diagnosticsLeak");
+args[2] = response.encodeURL(
+request.getContextPath() + "/html/findleaks");
+args[3] = newNonce;
+args[4] = sm.getString("htmlManagerServlet.diagnosticsLeakWarning");
+args[5] = sm.getString("htmlManagerServlet.diagnosticsLeakButton");
+writer.print(MessageFormat.format(DIAGNOSTICS_SECTION, args));
+
 // Server Header Section
 args = new Object[7];
 args[0] = sm.getString("htmlManagerServlet.serverTitle");
@@ -751,6 +764,33 @@
 }
 
 /**
+ * Find potential memory leaks caused by web application reload.
+ *
+ * @see ManagerServlet#findleaks(PrintWriter) 
+ *
+ * @return message String
+ */
+protected String findleaks() {
+
+StringBuilder msg = new StringBuilder();
+
+StringWriter stringWriter = new StringWriter();
+PrintWriter printWriter = new PrintWriter(stringWriter);
+
+super.findleaks(printWriter);
+
+if (stringWriter.getBuffer().length() > 0) {
+msg.append(sm.getString("htmlManagerServlet.findleaksList"));
+msg.append(stringWriter.toString());
+} else {
+msg.append(sm.getString("htmlManagerServlet.findleaksNone"));
+}
+
+return msg.toString();
+}
+
+
+/**
  * @see javax.servlet.Servlet#getServletInfo()
  */
 @Override
@@ -1126,7 +1166,7 @@
 " \n" +
 "  \n" +
 "  \n" +
-"  " +
 "    {10}  {12} \n" +
 "  \n" +
 "  \n" +
@@ -1137,11 +1177,11 @@
 " \n" +
 "   {1} \n" +
 "  " +
-"  " +
 "  " +
 "  \n" +
 "  " +
-"  " +
 "  " +
 "  \n" +
 "  " +
@@ -1153,7 +1193,7 @@
 " \n" +
 "  \n" +
 "  \n" +
-"  " +
 "    {10}  {12} \n" +
 "  \n" +
 "  \n" +
@@ -1163,13 +1203,13 @@
 private static final String STOPPED_DEPLOYED_APPS_ROW_BUTTON_SECTION =
 " \n" +
 "  " +
-"  " +
 "  " +
 "  \n" +
 "   {3} \n" +
 "   {5} \n" +
 "  " +
-"  " +
 "  " +
 "  \n" +
 " \n" +
@@ -1179,11 +1219,11 @@
 " \n" +
 "   {1} \n" +
 "  " +
-"  " +
 "  " +
 "  \n" +
 "  " +
-"  " +
 "  " +
 "  \n" +
 "   {7} \n" +
@@ -1193,7 +1233,7 @@
 private static final String STOPPED_NONDEPLOYED_APPS_ROW_BUTTON_SECTION =
 " \n" +
 "  " +
-"  " +
 "  " +
 "  \n" +
 "   {3} \n" +
@@ -1215,7 +1255,7 @@
 "\n" +
 " \n" +
 "\n" +
-"" +
 "\n" +
 "\n" +
 " \n" +
@@ -1285,4 +1325,31 @@
 "\n" +
 "\n";
 
+private static final String DIAGNOSTICS_SECTION =
+"\n" +
+"\n" +
+" {0}\n" +
+"\n" +
+"\n" +
+" {1}\n" +
+"\n" +
+"\n" +
+" \n" +
+"\n" +
+"" +
+"\n" +
+"\n" +
+" \n" +
+"  \n" +
+" \n" +
+" \n" +
+"  {4}\n" +
+" \n" +
+"\n" +
+"\n" +
+"\n" +
+"\n" +
+"\n" +
+"\n" +
+"";
 }

Modified: tomcat/trunk/java/org/apache/catalina/manager/LocalStrings.properties
URL: 
http://svn

svn commit: r909542 - in /tomcat/trunk/webapps/docs: html-manager-howto.xml manager-howto.xml

2010-02-12 Thread markt
Author: markt
Date: Fri Feb 12 18:03:48 2010
New Revision: 909542

URL: http://svn.apache.org/viewvc?rev=909542&view=rev
Log:
Document new findleaks command

Modified:
tomcat/trunk/webapps/docs/html-manager-howto.xml
tomcat/trunk/webapps/docs/manager-howto.xml

Modified: tomcat/trunk/webapps/docs/html-manager-howto.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/html-manager-howto.xml?rev=909542&r1=909541&r2=909542&view=diff
==
--- tomcat/trunk/webapps/docs/html-manager-howto.xml (original)
+++ tomcat/trunk/webapps/docs/html-manager-howto.xml Fri Feb 12 18:03:48 2010
@@ -36,7 +36,7 @@
 Tomcat.  This document is for the HTML web interface to the web application
 manager.
 
-The interface is divided into five sections:
+The interface is divided into six sections:
 
   Message - Displays success and failure messages.
   Manager - General manager operations like list and
@@ -44,6 +44,7 @@
   Applications - List of web applications and
   commands.
   Deploy - Deploying web applications.
+  Diagnostocs - Identifying potential problems.
   Server Information - Information about the Tomcat
   server.
 
@@ -546,6 +547,19 @@
 
 
 
+
+
+The find leaks diagnostic triggers a full garbage collection. It
+should be used with extreme caution on production systems.
+
+The find leaks diagnostic attempts to identify web applications that have
+caused memory leaks when they were reloaded. Results should always be confirmed
+with a profiler. The diagnostic uses additional functionality provided by the
+StandardHost implementation. It will not work if a custom host is used that
+does not extend StandardHost.
+
+
+
 
 
 This section displays information about Tomcat, the operating system of

Modified: tomcat/trunk/webapps/docs/manager-howto.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/manager-howto.xml?rev=909542&r1=909541&r2=909542&view=diff
==
--- tomcat/trunk/webapps/docs/manager-howto.xml (original)
+++ tomcat/trunk/webapps/docs/manager-howto.xml Fri Feb 12 18:03:48 2010
@@ -51,6 +51,8 @@
 Stop an Existing Application
 
 Undeploy an Existing Application
+Server Status
+Finding memory leaks
 
 
 Executing Manager Commands With Ant
@@ -891,6 +893,35 @@
 
 
 
+
+
+
+http://localhost:8080/manager/text/findleaks
+
+
+The find leaks diagnostic triggers a full garbage collection. It
+should be used with extreme caution on production systems.
+
+The find leaks diagnostic attempts to identify web applications that have
+caused memory leaks when they were reloaded. Results should always be confirmed
+with a profiler. The diagnostic uses additional functionality provided by the
+StandardHost implementation. It will not work if a custom host is used that
+does not extend StandardHost.
+
+If this command succeeds, you will see a response like this:
+
+/leaking-webapp
+
+
+Each context path for a web application that is believed to have triggered a
+memory leak when it was reloaded will be listed on a new line. If an 
application
+has been reloaded several times, it may be listed several times.
+
+If the command does not succeed, the response will start with
+FAIL and include an error message.
+ 
+
+
 
 
 From this link , you can view information about the server.



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r909563 - /tomcat/trunk/java/org/apache/catalina/core/StandardHost.java

2010-02-12 Thread markt
Author: markt
Date: Fri Feb 12 19:15:32 2010
New Revision: 909563

URL: http://svn.apache.org/viewvc?rev=909563&view=rev
Log:
Add listener before child is added else listener won't get called on first load

Modified:
tomcat/trunk/java/org/apache/catalina/core/StandardHost.java

Modified: tomcat/trunk/java/org/apache/catalina/core/StandardHost.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/StandardHost.java?rev=909563&r1=909562&r2=909563&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/core/StandardHost.java (original)
+++ tomcat/trunk/java/org/apache/catalina/core/StandardHost.java Fri Feb 12 
19:15:32 2010
@@ -578,15 +578,16 @@
 @Override
 public void addChild(Container child) {
 
+if (child instanceof Lifecycle) {
+((Lifecycle) child).addLifecycleListener(
+new MemoryLeakTrackingListener());
+}
+
 if (!(child instanceof Context))
 throw new IllegalArgumentException
 (sm.getString("standardHost.notContext"));
 super.addChild(child);
 
-if (child instanceof Lifecycle) {
-((Lifecycle) child).addLifecycleListener(
-new MemoryLeakTrackingListener());
-}
 }
 
 



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r909573 - /tomcat/tc6.0.x/trunk/STATUS.txt

2010-02-12 Thread markt
Author: markt
Date: Fri Feb 12 19:31:07 2010
New Revision: 909573

URL: http://svn.apache.org/viewvc?rev=909573&view=rev
Log:
Proposal

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=909573&r1=909572&r2=909573&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Fri Feb 12 19:31:07 2010
@@ -313,7 +313,8 @@
   -1: 
 
 * More memory leak prevent - ResourceBundle
-  http://svn.apache.org/viewvc?rev=909097&view=rev
+  http://svn.apache.org/viewvc?rev=909097&view=rev (original)
+  http://svn.apache.org/viewvc?rev=909525&view=rev (improvements)
   +1: markt
   -1: 
 
@@ -323,3 +324,8 @@
   http://svn.apache.org/viewvc?rev=909212&view=rev
   +1: kkolinko
   -1:
+
+* Memory leak detection for JMX and manager
+  
http://people.apache.org/~markt/patches/2010-02-12-memory-leak-detection.patch
+  +1: markt
+  -1: 



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r909613 - /tomcat/trunk/.classpath

2010-02-12 Thread markt
Author: markt
Date: Fri Feb 12 21:34:20 2010
New Revision: 909613

URL: http://svn.apache.org/viewvc?rev=909613&view=rev
Log:
Update for new Eclipse lib
Align with new trunk build script

Modified:
tomcat/trunk/.classpath

Modified: tomcat/trunk/.classpath
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/.classpath?rev=909613&r1=909612&r2=909613&view=diff
==
--- tomcat/trunk/.classpath (original)
+++ tomcat/trunk/.classpath Fri Feb 12 21:34:20 2010
@@ -4,12 +4,11 @@



-   
-   





+   

 



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r909636 - /tomcat/trunk/java/org/apache/catalina/servlets/WebdavServlet.java

2010-02-12 Thread markt
Author: markt
Date: Fri Feb 12 22:12:48 2010
New Revision: 909636

URL: http://svn.apache.org/viewvc?rev=909636&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48318
Handle case where WebDAV resource is in directory listing but is not accessible

Modified:
tomcat/trunk/java/org/apache/catalina/servlets/WebdavServlet.java

Modified: tomcat/trunk/java/org/apache/catalina/servlets/WebdavServlet.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/servlets/WebdavServlet.java?rev=909636&r1=909635&r2=909636&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/servlets/WebdavServlet.java (original)
+++ tomcat/trunk/java/org/apache/catalina/servlets/WebdavServlet.java Fri Feb 
12 22:12:48 2010
@@ -2039,6 +2039,11 @@
 return;
 
 CacheEntry cacheEntry = resources.lookupCache(path);
+if (!cacheEntry.exists) {
+   // File is in directory listing but doesn't appear to exist
+   // Broken symlink or odd permission settings?
+   return;
+}
 
 generatedXML.writeElement(null, "response", XMLWriter.OPENING);
 String status = new String("HTTP/1.1 " + WebdavStatus.SC_OK + " "



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r909637 - /tomcat/tc6.0.x/trunk/STATUS.txt

2010-02-12 Thread markt
Author: markt
Date: Fri Feb 12 22:15:54 2010
New Revision: 909637

URL: http://svn.apache.org/viewvc?rev=909637&view=rev
Log:
Proposal

Modified:
tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=909637&r1=909636&r2=909637&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Fri Feb 12 22:15:54 2010
@@ -329,3 +329,10 @@
   
http://people.apache.org/~markt/patches/2010-02-12-memory-leak-detection.patch
   +1: markt
   -1: 
+
+* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48318
+  Handle case where WebDAV resource is in directory listing but is not
+  accessible
+  http://svn.apache.org/viewvc?rev=909636&view=rev
+  +1: markt
+  -1: 



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



DO NOT REPLY [Bug 48318] WebDAV servlet returns 500 if files not readable

2010-02-12 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=48318

--- Comment #1 from Mark Thomas  2010-02-12 22:16:32 UTC ---
This has been fixed in trunk and proposed for 6.0.x

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



DO NOT REPLY [Bug 48323] AccessControlException on AJP connector, in security mode

2010-02-12 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=48323

Mark Thomas  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

--- Comment #2 from Mark Thomas  2010-02-12 23:17:31 UTC ---
This works for me with the latest 6.0.x. I suspect the fix for bug47963
refactored the code in such a way as to avoid this issue.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r909652 - in /tomcat/site/trunk: docs/lists.html xdocs/lists.xml

2010-02-12 Thread kkolinko
Author: kkolinko
Date: Fri Feb 12 23:17:59 2010
New Revision: 909652

URL: http://svn.apache.org/viewvc?rev=909652&view=rev
Log:
Add a reference to the ASF Public Forum Archive Policy

Modified:
tomcat/site/trunk/docs/lists.html
tomcat/site/trunk/xdocs/lists.xml

Modified: tomcat/site/trunk/docs/lists.html
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/docs/lists.html?rev=909652&r1=909651&r2=909652&view=diff
==
--- tomcat/site/trunk/docs/lists.html (original)
+++ tomcat/site/trunk/docs/lists.html Fri Feb 12 23:17:59 2010
@@ -297,6 +297,8 @@
   The http://wiki.apache.org/tomcat/FAQ";>FAQ and
   http://wiki.apache.org/tomcat/HowTo";>How To pages
   are available at the http://wiki.apache.org/tomcat/";>Tomcat 
Wiki.
+  Usage of these mailing lists is subject to the
+  http://www.apache.org/foundation/public-archives.html";>Public Forum 
Archive Policy.
 
 
 

Modified: tomcat/site/trunk/xdocs/lists.xml
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/xdocs/lists.xml?rev=909652&r1=909651&r2=909652&view=diff
==
--- tomcat/site/trunk/xdocs/lists.xml (original)
+++ tomcat/site/trunk/xdocs/lists.xml Fri Feb 12 23:17:59 2010
@@ -86,6 +86,8 @@
   The http://wiki.apache.org/tomcat/FAQ";>FAQ and
   http://wiki.apache.org/tomcat/HowTo";>How To pages
   are available at the http://wiki.apache.org/tomcat/";>Tomcat 
Wiki.
+  Usage of these mailing lists is subject to the
+  http://www.apache.org/foundation/public-archives.html";>Public Forum 
Archive Policy.
 
 
 



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r909659 - in /tomcat/site/trunk/xdocs: getinvolved.xml lists.xml stylesheets/project.xml

2010-02-12 Thread kkolinko
Author: kkolinko
Date: Fri Feb 12 23:32:55 2010
New Revision: 909659

URL: http://svn.apache.org/viewvc?rev=909659&view=rev
Log:
Update links to Tomcat Wiki in the site menu and on several pages.
Those links should be not
http://wiki.apache.org/tomcat
but
http://wiki.apache.org/tomcat/

I am using a direct link to
http://wiki.apache.org/tomcat/FrontPage
instead of the above, because i18n is misconfigured in our MoinMoin and 
backfires - see https://issues.apache.org/jira/browse/INFRA-2251

Modified:
tomcat/site/trunk/xdocs/getinvolved.xml
tomcat/site/trunk/xdocs/lists.xml
tomcat/site/trunk/xdocs/stylesheets/project.xml

Modified: tomcat/site/trunk/xdocs/getinvolved.xml
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/xdocs/getinvolved.xml?rev=909659&r1=909658&r2=909659&view=diff
==
--- tomcat/site/trunk/xdocs/getinvolved.xml (original)
+++ tomcat/site/trunk/xdocs/getinvolved.xml Fri Feb 12 23:32:55 2010
@@ -18,7 +18,7 @@
   Answer questions posted to the mailing lists.
   
   Contribute to the http://wiki.apache.org/tomcat/FAQ";>FAQ or
-  http://wiki.apache.org/tomcat";>Wiki.
+  http://wiki.apache.org/tomcat/FrontPage";>Wiki.
   Contribute other documentation patches, either for the website or for the
   Apache Tomcat user documentation.
   http://issues.apache.org/bugzilla";>Investigate bugs that 
have

Modified: tomcat/site/trunk/xdocs/lists.xml
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/xdocs/lists.xml?rev=909659&r1=909658&r2=909659&view=diff
==
--- tomcat/site/trunk/xdocs/lists.xml (original)
+++ tomcat/site/trunk/xdocs/lists.xml Fri Feb 12 23:32:55 2010
@@ -85,7 +85,7 @@
   archives (links are below) are also searchable.
   The http://wiki.apache.org/tomcat/FAQ";>FAQ and
   http://wiki.apache.org/tomcat/HowTo";>How To pages
-  are available at the http://wiki.apache.org/tomcat/";>Tomcat 
Wiki.
+  are available at the http://wiki.apache.org/tomcat/FrontPage";>Tomcat Wiki.
   Usage of these mailing lists is subject to the
   http://www.apache.org/foundation/public-archives.html";>Public Forum 
Archive Policy.
 

Modified: tomcat/site/trunk/xdocs/stylesheets/project.xml
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/xdocs/stylesheets/project.xml?rev=909659&r1=909658&r2=909659&view=diff
==
--- tomcat/site/trunk/xdocs/stylesheets/project.xml (original)
+++ tomcat/site/trunk/xdocs/stylesheets/project.xml Fri Feb 12 23:32:55 2010
@@ -40,7 +40,7 @@
 
 
 
-http://wiki.apache.org/tomcat"/>
+http://wiki.apache.org/tomcat/FrontPage"/>
 
 
 



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r909660 - /tomcat/site/trunk/docs/

2010-02-12 Thread kkolinko
Author: kkolinko
Date: Fri Feb 12 23:38:27 2010
New Revision: 909660

URL: http://svn.apache.org/viewvc?rev=909660&view=rev
Log:
Followup to r909659 - updating the generated pages.

Modified:
tomcat/site/trunk/docs/bugreport.html
tomcat/site/trunk/docs/contact.html
tomcat/site/trunk/docs/download-55.html
tomcat/site/trunk/docs/download-60.html
tomcat/site/trunk/docs/download-connectors.html
tomcat/site/trunk/docs/download-native.html
tomcat/site/trunk/docs/findhelp.html
tomcat/site/trunk/docs/getinvolved.html
tomcat/site/trunk/docs/heritage.html
tomcat/site/trunk/docs/index.html
tomcat/site/trunk/docs/irc.html
tomcat/site/trunk/docs/legal.html
tomcat/site/trunk/docs/lists.html
tomcat/site/trunk/docs/migration.html
tomcat/site/trunk/docs/oldnews.html
tomcat/site/trunk/docs/resources.html
tomcat/site/trunk/docs/security-3.html
tomcat/site/trunk/docs/security-4.html
tomcat/site/trunk/docs/security-5.html
tomcat/site/trunk/docs/security-6.html
tomcat/site/trunk/docs/security-7.html
tomcat/site/trunk/docs/security-impact.html
tomcat/site/trunk/docs/security-jk.html
tomcat/site/trunk/docs/security-native.html
tomcat/site/trunk/docs/security.html
tomcat/site/trunk/docs/svn.html
tomcat/site/trunk/docs/whichversion.html
tomcat/site/trunk/docs/whoweare.html

Modified: tomcat/site/trunk/docs/bugreport.html
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/docs/bugreport.html?rev=909660&r1=909659&r2=909660&view=diff
==
--- tomcat/site/trunk/docs/bugreport.html (original)
+++ tomcat/site/trunk/docs/bugreport.html Fri Feb 12 23:38:27 2010
@@ -138,7 +138,7 @@
 Mailing Lists
 
 
-http://wiki.apache.org/tomcat";>Wiki
+http://wiki.apache.org/tomcat/FrontPage";>Wiki
 
 
 

Modified: tomcat/site/trunk/docs/contact.html
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/docs/contact.html?rev=909660&r1=909659&r2=909660&view=diff
==
--- tomcat/site/trunk/docs/contact.html (original)
+++ tomcat/site/trunk/docs/contact.html Fri Feb 12 23:38:27 2010
@@ -137,7 +137,7 @@
 Mailing Lists
 
 
-http://wiki.apache.org/tomcat";>Wiki
+http://wiki.apache.org/tomcat/FrontPage";>Wiki
 
 
 

Modified: tomcat/site/trunk/docs/download-55.html
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/docs/download-55.html?rev=909660&r1=909659&r2=909660&view=diff
==
--- tomcat/site/trunk/docs/download-55.html (original)
+++ tomcat/site/trunk/docs/download-55.html Fri Feb 12 23:38:27 2010
@@ -137,7 +137,7 @@
 Mailing Lists
 
 
-http://wiki.apache.org/tomcat";>Wiki
+http://wiki.apache.org/tomcat/FrontPage";>Wiki
 
 
 

Modified: tomcat/site/trunk/docs/download-60.html
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/docs/download-60.html?rev=909660&r1=909659&r2=909660&view=diff
==
--- tomcat/site/trunk/docs/download-60.html (original)
+++ tomcat/site/trunk/docs/download-60.html Fri Feb 12 23:38:27 2010
@@ -137,7 +137,7 @@
 Mailing Lists
 
 
-http://wiki.apache.org/tomcat";>Wiki
+http://wiki.apache.org/tomcat/FrontPage";>Wiki
 
 
 

Modified: tomcat/site/trunk/docs/download-connectors.html
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/docs/download-connectors.html?rev=909660&r1=909659&r2=909660&view=diff
==
--- tomcat/site/trunk/docs/download-connectors.html (original)
+++ tomcat/site/trunk/docs/download-connectors.html Fri Feb 12 23:38:27 2010
@@ -137,7 +137,7 @@
 Mailing Lists
 
 
-http://wiki.apache.org/tomcat";>Wiki
+http://wiki.apache.org/tomcat/FrontPage";>Wiki
 
 
 

Modified: tomcat/site/trunk/docs/download-native.html
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/docs/download-native.html?rev=909660&r1=909659&r2=909660&view=diff
==
--- tomcat/site/trunk/docs/download-native.html (original)
+++ tomcat/site/trunk/docs/download-native.html Fri Feb 12 23:38:27 2010
@@ -3,17 +3,17 @@
 
 
 Apache Tomcat - Tomcat Native Downloads
-
-
+
+
 
-
-
+
+
 
 
 
 
 http://tomcat.apache.org/";>
-
+
 
 
 
@@ -24,28 +24,28 @@
 
 
 http://www.apache.org/";>
-http://www.apache.org/images/asf-logo.gif"; />
+http://www.apache.org/images/asf-logo.gif"; align="right" alt="Apache 
Logo" border="0"/>
 
 
 
 
 
-http://www.google.com/search";>
-
-
-
+http://www.google.com/search"; method="get">
+
+
+
 
 
-
+
 
 
 
-
+
 
 
 
 
-
+
 
 Apache Tomcat
 
@@ -137,7 +137,7 @@
 Mailing Lists
 
 
-http://wiki.apache.org/tomcat";>Wiki
+http://wiki.apache.org/tomcat/FrontPage";>Wiki
 
 
 
@@ -171,11 +171,11 @@
 
 
 
-
-
+
+
 
 
-
+
 
 Tomcat Native Downloads
 
@@ -199,10 +199,10 @@
 
   Recent releases 

Re: svn commit: r907390 - in /tomcat/site/trunk: docs/whoweare.html xdocs/whoweare.xml

2010-02-12 Thread Konstantin Kolinko
2010/2/11 Mark Thomas :
> On 07/02/2010 07:33, t...@apache.org wrote:
>> Author: timw
>> Date: Sun Feb  7 07:33:25 2010
>> New Revision: 907390
>>
>> URL: http://svn.apache.org/viewvc?rev=907390&view=rev
>> Log:
>> Adding timw to committers list.
>
> Tim when you did svn up on people.a.o you didn't set your umask first.
> You are now the only person that can modify this file. Please can you do
> a chmod g+w on that file so anyone in the Tomcat group can update it.
>
> Cheers,
>
> Mark
>

SVN ignores those permissions.  I just updated that file (in r909660)
and it just let me to do so, regardless of -rw-r--r-- mode of it.
Note, that the .svn/entries file even has -r--r--r-- on it, but is
updated by svn.

Still, having
umask 002
in your .profile  is recommended.


Mark, please update the group owner for the  tomcat-5.5-doc symlink.
It is owned by your private group (markt) right now.


Best regards,
Konstantin Kolinko

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



svn commit: r909673 - /tomcat/trunk/java/org/apache/catalina/connector/Connector.java

2010-02-12 Thread fhanik
Author: fhanik
Date: Sat Feb 13 00:51:47 2010
New Revision: 909673

URL: http://svn.apache.org/viewvc?rev=909673&view=rev
Log:
fix compilation error

Modified:
tomcat/trunk/java/org/apache/catalina/connector/Connector.java

Modified: tomcat/trunk/java/org/apache/catalina/connector/Connector.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/connector/Connector.java?rev=909673&r1=909672&r2=909673&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/connector/Connector.java (original)
+++ tomcat/trunk/java/org/apache/catalina/connector/Connector.java Sat Feb 13 
00:51:47 2010
@@ -539,7 +539,7 @@
 
 /**
  * Return the port number on which we listen for requests.
- */
+ /
 public int getPort() {
 
 return (this.port);
@@ -558,6 +558,10 @@
 setProperty("port", String.valueOf(port));
 
 }
+
+public int getPort() {
+return this.port;
+}
 
 
 /**



-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org



Re: svn commit: r909673 - /tomcat/trunk/java/org/apache/catalina/connector/Connector.java

2010-02-12 Thread Konstantin Kolinko
2010/2/13  :
> Author: fhanik
> Date: Sat Feb 13 00:51:47 2010
> New Revision: 909673
>
> URL: http://svn.apache.org/viewvc?rev=909673&view=rev
> Log:
> fix compilation error
>
> Modified:
>    tomcat/trunk/java/org/apache/catalina/connector/Connector.java
>

What is this?  The new code is equivalent to the old one, but with a
broken javadoc comment.


> Modified: tomcat/trunk/java/org/apache/catalina/connector/Connector.java
> URL: 
> http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/connector/Connector.java?rev=909673&r1=909672&r2=909673&view=diff
> ==
> --- tomcat/trunk/java/org/apache/catalina/connector/Connector.java (original)
> +++ tomcat/trunk/java/org/apache/catalina/connector/Connector.java Sat Feb 13 
> 00:51:47 2010
> @@ -539,7 +539,7 @@
>
>     /**
>      * Return the port number on which we listen for requests.
> -     */
> +     /
>     public int getPort() {
>
>         return (this.port);
> @@ -558,6 +558,10 @@
>         setProperty("port", String.valueOf(port));
>
>     }
> +
> +    public int getPort() {
> +        return this.port;
> +    }
>
>
>     /**
>
>
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
>
>

-
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org