Re: Bayeux/Cometd API question

2008-12-19 Thread fl-00
Hi Filip,

ok, I'll take a look at the current implementation and if the effort seems 
reasonable for me, I'll try to allocate some days during the next (let's say 
3-6) weeks to contribute an implementation for the current API. I would then 
really propose to use the API interface package org.cometd.* as used by Jetty 
(and others). Would that be ok for you? 
If I have questions left, should I post them to the dev list here or should I 
contact you directly?

Regards,
Florian

 Original-Nachricht 
> Datum: Thu, 18 Dec 2008 15:01:26 -0700
> Von: Filip Hanik - Dev Lists 
> An: Tomcat Developers List 
> Betreff: Re: Bayeux/Cometd API question

> Feel free to contribute any API changes you feel need to be added
> 
> Filip
> 
> fl...@gmx.de wrote:
> > Hi together,
> >
> > I've developed a Bayeux/Cometd application using the Jetty
> implementation running in Tomcat 5. Development started several month back 
> with no
> Tomcat Bayeux implementation available.
> > With this setup the application works well for limited traffic but of
> course not for high-volume traffic (since that way it can't really scale).
> I'm following the Tomcat Bayeux development for several months now.
> Currently, I'm somewhat concerned about the Bayeux-API differences between 
> Jetty and
> Tomcat, and I can't see any sign/any attempt of unifying the API (this
> prevents us to switch to the Tomcat Bayeux implementation and we need to even
> think about switching completely to Jetty). 
> >
> > I know that org.cometd.Bayeux is not a real standardized thing and it's
> being hosted at mortbay.org. But are there any plans for the Tomcat
> implementation to catch up with the latest (at least half-"official") API? Or 
> even
> any vision of Tomcat/Jetty people pulling Bayeux/Cometd together towards a
> common direction/standardized API (or even JSR)? The current situation is
> somewhat unfortunate for a developer of a large Bayeux/Cometd application.
> >
> > Best Regards,
> > Florian
> >   
> 
> 
> -
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org

-- 
Pt! Schon vom neuen GMX MultiMessenger gehört? Der kann`s mit allen: 
http://www.gmx.net/de/go/multimessenger

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



svn commit: r728026 - in /tomcat/site/trunk: docs/security-4.html docs/security-5.html docs/security-6.html xdocs/security-4.xml xdocs/security-5.xml xdocs/security-6.xml

2008-12-19 Thread markt
Author: markt
Date: Fri Dec 19 06:10:33 2008
New Revision: 728026

URL: http://svn.apache.org/viewvc?rev=728026&view=rev
Log:
Update security pages with real cause of CVE-2008-2938

Modified:
tomcat/site/trunk/docs/security-4.html
tomcat/site/trunk/docs/security-5.html
tomcat/site/trunk/docs/security-6.html
tomcat/site/trunk/xdocs/security-4.xml
tomcat/site/trunk/xdocs/security-5.xml
tomcat/site/trunk/xdocs/security-6.xml

Modified: tomcat/site/trunk/docs/security-4.html
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/docs/security-4.html?rev=728026&r1=728025&r2=728026&view=diff
==
--- tomcat/site/trunk/docs/security-4.html (original)
+++ tomcat/site/trunk/docs/security-4.html Fri Dec 19 06:10:33 2008
@@ -326,28 +326,6 @@
 
 Affects: 4.1.0-4.1.37
 
-
-important: Directory traversal
-   http://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2008-2938";>
-   CVE-2008-2938
-
-
-If a context is configured with allowLinking="true" and the
-   connector is configured with URIEncoding="UTF-8" then a
-   malformed request may be used to access arbitrary files on the server.
-   If the connector is configured with URIEncoding="UTF-8" 
then
-   a malformed request may be used to access arbitrary files within the
-   docBase of a context such as web.xml. It should also be noted that
-   setting useBodyEncodingForURI="true" has the same effect as
-   setting URIEncoding="UTF-8" when processing requests with
-   bodies encoded with UTF-8.
-
-   This was fixed in
-   http://svn.apache.org/viewvc?rev=681065&view=rev";>
-   revision 681065.
-
-Affects: 4.1.0-4.1.37
-
   
 
 
@@ -1072,6 +1050,39 @@
page that called System.exit(1) rather than relying on a bug in an
internal Sun class.
 
+
+important: Directory traversal
+   http://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2008-2938";>
+   CVE-2008-2938
+
+
+Originally reported as a Tomcat vulnerability the root cause of this
+   issue is that the JVM does not correctly decode UTF-8 encoded URLs to
+   UTF-8. This exposes a directory traversal vulnerability when the
+   connector uses URIEncoding="UTF-8". This directory 
traversal
+   is limited to the docBase of the web application.
+   
+If a context is configured with allowLinking="true" then 
the
+   directory traversal vulnerability is extended to the entire file system
+   of the host server.
+   
+It should also be noted that setting
+   useBodyEncodingForURI="true" has the same effect as setting
+   URIEncoding="UTF-8" when processing requests with bodies
+   encoded with UTF-8.
+
+Although the root cause was quickly identified as a JVM issue and that 
it
+   affected multiple JVMs from multiple vendors, it was decided to report
+   this as a Tomcat vulnerability until such time as the JVM vendors
+   provided updates to resolve this issue. For further information on the
+   status of this issue for your JVM, contact your JVM vendor.
+   
+A workaround was implemented in
+   http://svn.apache.org/viewvc?rev=681065&view=rev";>
+   revision 681065 that protects against this and any similar character
+   encoding issues that may still exist in the JVM. This work around is
+   included in Tomcat 4.1.39 onwards.
+
   
 
 

Modified: tomcat/site/trunk/docs/security-5.html
URL: 
http://svn.apache.org/viewvc/tomcat/site/trunk/docs/security-5.html?rev=728026&r1=728025&r2=728026&view=diff
==
--- tomcat/site/trunk/docs/security-5.html (original)
+++ tomcat/site/trunk/docs/security-5.html Fri Dec 19 06:10:33 2008
@@ -288,28 +288,6 @@
 
 Affects: 5.5.0-5.5.26
 
-
-important: Directory traversal
-   http://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2008-2938";>
-   CVE-2008-2938
-
-
-If a context is configured with allowLinking="true" and the
-   connector is configured with URIEncoding="UTF-8" then a
-   malformed request may be used to access arbitrary files on the server.
-   If the connector is configured with URIEncoding="UTF-8" 
then
-   a malformed request may be used to access arbitrary files within the
-   docBase of a context such as web.xml. It should also be noted that
-   setting useBodyEncodingForURI="true" has the same effect as
-   setting URIEncoding="UTF-8" when processing requests with
-   bodies encoded with UTF-8.
-
-   This was fixed in
-   http://svn.apache.org/viewvc?rev=681029&view=rev";>
-   revision 681029.
-   
-Affects: 5.5.0-5.5.26
-
   
 
 
@@ -964,6 +942,39 @@
 The vulnerability described is in the web application deployed on Tomcat
rather than in Tomcat.
 
+
+important: Directory traversal
+   http://cve.mitre.org/cgi-bin/cvename.cgi?

svn commit: r728028 - /tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml

2008-12-19 Thread markt
Author: markt
Date: Fri Dec 19 06:20:40 2008
New Revision: 728028

URL: http://svn.apache.org/viewvc?rev=728028&view=rev
Log:
Update the change log

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

Modified: tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml?rev=728028&r1=728027&r2=728028&view=diff
==
--- tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Fri Dec 19 06:20:40 2008
@@ -363,6 +363,10 @@
 1.6 JDK but because it does this by skipping DBCP, release builds must
 be generated with a 1.5 JDK. (costin/markt)
   
+  
+46366: Correct information in RUNNING.txt regarding use of
+CATALINA_HOME and CATALINA_BASE. (markt) 
+  
 
   
 



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



svn commit: r728030 - /tomcat/connectors/trunk/jni/xdocs/index.xml

2008-12-19 Thread markt
Author: markt
Date: Fri Dec 19 06:23:49 2008
New Revision: 728030

URL: http://svn.apache.org/viewvc?rev=728030&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=46291
Add note that setenv.(sh|bat) should be created if not present

Modified:
tomcat/connectors/trunk/jni/xdocs/index.xml

Modified: tomcat/connectors/trunk/jni/xdocs/index.xml
URL: 
http://svn.apache.org/viewvc/tomcat/connectors/trunk/jni/xdocs/index.xml?rev=728030&r1=728029&r2=728030&view=diff
==
--- tomcat/connectors/trunk/jni/xdocs/index.xml (original)
+++ tomcat/connectors/trunk/jni/xdocs/index.xml Fri Dec 19 06:23:49 2008
@@ -170,8 +170,8 @@
 
 
   
-   Edit the $CATALINA_HOME/bin/setenv.sh and add the path to the tc-native 
libraries to LD_LIBRARY_PATH.
-   Something like:
+   Edit $CATALINA_HOME/bin/setenv.sh (creating the file if necessary) and add
+   the path to the tc-native libraries to LD_LIBRARY_PATH. Something like:

 LD_LIBRARY_PATH=$LD_LIBRARY_PATH:$CATALINA_HOME/lib
 export LD_LIBRARY_PATH
@@ -193,8 +193,8 @@
 
 
   
-Edit the $CATALINA_BASE\bin\setenv.bat and add the path to the tc-native 
libraries, apr and OpenSSL to PATH.
-For example:
+Edit $CATALINA_BASE\bin\setenv.bat (creating the file if necessary) and add
+the path to the tc-native libraries, apr and OpenSSL to PATH. For example:
   
 set 
PATH=%PATH;C:\cygwin\home\support\tomcat-native-current-win32-src\jni\native\Debug;C:\cygwin\home\support\tomcat-native-current-win32-src\jni\apr\Debug;C:\OpenSSL\lib\VC
   



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



DO NOT REPLY [Bug 46291] http://tomcat.apache.org/native-doc/ installation is wrong

2008-12-19 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=46291


Mark Thomas  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Comment #1 from Mark Thomas   2008-12-19 06:24:30 PST ---
By default, setenv does not exist. It should be created if required. I have
added a note to that effect to the docs.


-- 
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: r728032 - /tomcat/trunk/java/org/apache/catalina/security/SecurityUtil.java

2008-12-19 Thread markt
Author: markt
Date: Fri Dec 19 06:28:06 2008
New Revision: 728032

URL: http://svn.apache.org/viewvc?rev=728032&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=46408
Don't make assumptions about exception type

Modified:
tomcat/trunk/java/org/apache/catalina/security/SecurityUtil.java

Modified: tomcat/trunk/java/org/apache/catalina/security/SecurityUtil.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/security/SecurityUtil.java?rev=728032&r1=728031&r2=728032&view=diff
==
--- tomcat/trunk/java/org/apache/catalina/security/SecurityUtil.java (original)
+++ tomcat/trunk/java/org/apache/catalina/security/SecurityUtil.java Fri Dec 19 
06:28:06 2008
@@ -299,9 +299,14 @@
 }
 
 Subject.doAsPrivileged(subject, pea, null);   
-   } catch( PrivilegedActionException pe) {
-Throwable e = ((InvocationTargetException)pe.getException())
+} catch( PrivilegedActionException pe) {
+Throwable e;
+if (pe.getException() instanceof InvocationTargetException) {
+e = ((InvocationTargetException)pe.getException())
 .getTargetException();
+} else {
+e = pe;
+}
 
 if (log.isDebugEnabled()){
 log.debug(sm.getString("SecurityUtil.doAsPrivilege"), e); 



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



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

2008-12-19 Thread markt
Author: markt
Date: Fri Dec 19 06:29:48 2008
New Revision: 728033

URL: http://svn.apache.org/viewvc?rev=728033&view=rev
Log:
Prpose fix for 46408

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=728033&r1=728032&r2=728033&view=diff
==
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Fri Dec 19 06:29:48 2008
@@ -257,3 +257,9 @@
   +1: fhanik
   -1: 
 
+* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=46408
+  Correct potential invalid cast
+  http://svn.apache.org/viewvc?rev=728032&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 46408] Invalid cast in SecurityUtil

2008-12-19 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=46408





--- Comment #1 from Mark Thomas   2008-12-19 06:30:03 PST ---
Thanks for the report. 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



Re: Bayeux/Cometd API question

2008-12-19 Thread Filip Hanik - Dev Lists

fl...@gmx.de wrote:

Hi Filip,

ok, I'll take a look at the current implementation and if the effort seems 
reasonable for me, I'll try to allocate some days during the next (let's say 
3-6) weeks to contribute an implementation for the current API.

excellent.
 I would then really propose to use the API interface package org.cometd.* as used by Jetty (and others). Would that be ok for you? 
  
this is the Apache Software Foundation, so software here, unless we 
implement a standard, is always prefixed org.apache

If I have questions left, should I post them to the dev list here or should I 
contact you directly?
  

the mailing list is the way to go, anyone who wishes to participate can

Regards,
Florian

 Original-Nachricht 
  

Datum: Thu, 18 Dec 2008 15:01:26 -0700
Von: Filip Hanik - Dev Lists 
An: Tomcat Developers List 
Betreff: Re: Bayeux/Cometd API question



  

Feel free to contribute any API changes you feel need to be added

Filip

fl...@gmx.de wrote:


Hi together,

I've developed a Bayeux/Cometd application using the Jetty
  

implementation running in Tomcat 5. Development started several month back with 
no
Tomcat Bayeux implementation available.


With this setup the application works well for limited traffic but of
  

course not for high-volume traffic (since that way it can't really scale).
I'm following the Tomcat Bayeux development for several months now.
Currently, I'm somewhat concerned about the Bayeux-API differences between 
Jetty and
Tomcat, and I can't see any sign/any attempt of unifying the API (this
prevents us to switch to the Tomcat Bayeux implementation and we need to even
think about switching completely to Jetty). 


I know that org.cometd.Bayeux is not a real standardized thing and it's
  

being hosted at mortbay.org. But are there any plans for the Tomcat
implementation to catch up with the latest (at least half-"official") API? Or 
even
any vision of Tomcat/Jetty people pulling Bayeux/Cometd together towards a
common direction/standardized API (or even JSR)? The current situation is
somewhat unfortunate for a developer of a large Bayeux/Cometd application.


Best Regards,
Florian
  
  

-
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



svn commit: r728081 - in /tomcat/trunk/modules/jdbc-pool: ./ doc/ java/org/apache/tomcat/jdbc/pool/ java/org/apache/tomcat/jdbc/pool/interceptor/ test/org/apache/tomcat/jdbc/pool/ test/org/apache/tomc

2008-12-19 Thread fhanik
Author: fhanik
Date: Fri Dec 19 09:55:46 2008
New Revision: 728081

URL: http://svn.apache.org/viewvc?rev=728081&view=rev
Log:
Documented interceptors a bit better, implemented setters for interceptor 
properties

Added:
tomcat/trunk/modules/jdbc-pool/test/org/apache/tomcat/jdbc/pool/
tomcat/trunk/modules/jdbc-pool/test/org/apache/tomcat/jdbc/pool/interceptor/

tomcat/trunk/modules/jdbc-pool/test/org/apache/tomcat/jdbc/pool/interceptor/TestInterceptor.java

tomcat/trunk/modules/jdbc-pool/test/org/apache/tomcat/jdbc/test/TestInterceptorShortName.java
Modified:
tomcat/trunk/modules/jdbc-pool/build.xml
tomcat/trunk/modules/jdbc-pool/doc/jdbc-pool.xml

tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java

tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/JdbcInterceptor.java

tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/PoolProperties.java

tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/interceptor/SlowQueryReport.java
tomcat/trunk/modules/jdbc-pool/sign.sh

Modified: tomcat/trunk/modules/jdbc-pool/build.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/modules/jdbc-pool/build.xml?rev=728081&r1=728080&r2=728081&view=diff
==
--- tomcat/trunk/modules/jdbc-pool/build.xml (original)
+++ tomcat/trunk/modules/jdbc-pool/build.xml Fri Dec 19 09:55:46 2008
@@ -23,7 +23,7 @@
   
   
   
-  
+  
   
   
   

Modified: tomcat/trunk/modules/jdbc-pool/doc/jdbc-pool.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/modules/jdbc-pool/doc/jdbc-pool.xml?rev=728081&r1=728080&r2=728081&view=diff
==
--- tomcat/trunk/modules/jdbc-pool/doc/jdbc-pool.xml (original)
+++ tomcat/trunk/modules/jdbc-pool/doc/jdbc-pool.xml Fri Dec 19 09:55:46 2008
@@ -350,6 +350,75 @@
 of the results from a method invokation as well. You could build query 
performance analyzer that provides JMX notifications when a 
 query is running longer than the expected time.
   
+  
+Configuring JDBC interceptors is done using the jdbcInterceptors 
property.
+The property contains a list of semi colon separated class names. If the 
classname if not fully qualified it will be prefixed with the 
+org.apache.tomcat.jdbc.pool.interceptor. prefix.
+Example:
+
+  
jdbcInterceptors="org.apache.tomcat.jdbc.pool.interceptor.ConnectionState;org.apache.tomcat.jdbc.pool.interceptor.StatementFinalizer"
+  
+  is the same as 
+  
+  jdbcInterceptors="ConnectionState;StatementFinalizer"
+
+Interceptors can have properties as well. These would be configured within 
the paranthesis of the class names.
+Example:
+
+  
jdbcInterceptors="ConnectionState;StatementFinalizer(useWeakReferences=true,useEquals=true)"
+
+  
+  
+  
+Abstract base class for all interceptors, can not be instantiated.
+
+  
+(String as boolean) A custom query to be run when a connection is 
first created. 
+   The default value is false.
+
+  
+  
+  
+  
+Caches the connection for the following attributes 
autoCommit, readOnly,
+   transactionIsolation and catalog.
+   It is a performance enhancement to avoid roundtrip to the database when 
getters are called or setters are called with an already set value.
+   
+
+  
+  
+  
+Keeps track of all statements created using 
createStatement, prepareStatement or 
prepareCall
+   and closes these statements when the connection is returned to the pool.
+   
+
+  
+  
+  
+Keeps track of query performance and issues log entries when queries 
exceed a time threshold of fail.
+   The log level used is WARN
+   
+
+  
+(int as String) The number of milliseconds a query has to exceed 
before issuing a log alert.
+   The default value is 1000 milliseconds.
+
+  
+  
+(int as String) The maximum number of queries to keep track of in 
order to preserve memory space 
+   The default value is 1000.
+
+  
+  
+  
+  
+Extends the SlowQueryReport and in addition to log entries 
it issues JMX notification
+   for monitoring tools to react to. Inherits all the attributes from its 
parent class.
+   This class uses Tomcat's JMX engine so it wont work outside of the 
Tomcat container.
+   
+
+
+  
 
 
 
@@ -519,15 +588,41 @@
 
 The compare(String,Method) will use the 
useEquals flag on an interceptor and do either reference 
comparison or 
 a string value comparison when the useEquals=true flag is 
set.
-   
+
+Pool start/stop
+   When the connection pool is started or closed, you can be notifed. You 
will only be notified once per interceptor class
+   even though it is an instance 

svn commit: r728158 - /tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/PoolProperties.java

2008-12-19 Thread fhanik
Author: fhanik
Date: Fri Dec 19 13:45:04 2008
New Revision: 728158

URL: http://svn.apache.org/viewvc?rev=728158&view=rev
Log:
Fix property parsing

Modified:

tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/PoolProperties.java

Modified: 
tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/PoolProperties.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/PoolProperties.java?rev=728158&r1=728157&r2=728158&view=diff
==
--- 
tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/PoolProperties.java
 (original)
+++ 
tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/PoolProperties.java
 Fri Dec 19 13:45:04 2008
@@ -220,12 +220,13 @@
 InterceptorDefinition[] definitions = new 
InterceptorDefinition[interceptorValues.length];
 for (int i=0; i

svn commit: r728161 - in /tomcat/trunk/modules/jdbc-pool: build.xml sign.sh

2008-12-19 Thread fhanik
Author: fhanik
Date: Fri Dec 19 13:46:25 2008
New Revision: 728161

URL: http://svn.apache.org/viewvc?rev=728161&view=rev
Log:
inc version number

Modified:
tomcat/trunk/modules/jdbc-pool/build.xml
tomcat/trunk/modules/jdbc-pool/sign.sh

Modified: tomcat/trunk/modules/jdbc-pool/build.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/modules/jdbc-pool/build.xml?rev=728161&r1=728160&r2=728161&view=diff
==
--- tomcat/trunk/modules/jdbc-pool/build.xml (original)
+++ tomcat/trunk/modules/jdbc-pool/build.xml Fri Dec 19 13:46:25 2008
@@ -23,7 +23,7 @@
   
   
   
-  
+  
   
   
   

Modified: tomcat/trunk/modules/jdbc-pool/sign.sh
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/modules/jdbc-pool/sign.sh?rev=728161&r1=728160&r2=728161&view=diff
==
--- tomcat/trunk/modules/jdbc-pool/sign.sh (original)
+++ tomcat/trunk/modules/jdbc-pool/sign.sh Fri Dec 19 13:46:25 2008
@@ -1,4 +1,4 @@
-VERSION=v1.0.8-beta
+VERSION=v1.0.9-beta
 for i in $(find output/release/$VERSION -name "*.zip" -o -name "*.tar.gz"); do
   echo Signing $i
   echo $1|gpg --passphrase-fd 0 -a -b $i



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



svn commit: r728197 - in /tomcat/trunk/modules/jdbc-pool: build.xml java/org/apache/tomcat/jdbc/pool/ConnectionPool.java java/org/apache/tomcat/jdbc/pool/PoolProperties.java java/org/apache/tomcat/jdb

2008-12-19 Thread fhanik
Author: fhanik
Date: Fri Dec 19 16:45:46 2008
New Revision: 728197

URL: http://svn.apache.org/viewvc?rev=728197&view=rev
Log:
Removed calling to a JDK 1.6 method if its run under 1.5 and still take 
advantage of the JMX features

Modified:
tomcat/trunk/modules/jdbc-pool/build.xml

tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java

tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/PoolProperties.java

tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java
tomcat/trunk/modules/jdbc-pool/sign.sh

Modified: tomcat/trunk/modules/jdbc-pool/build.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/modules/jdbc-pool/build.xml?rev=728197&r1=728196&r2=728197&view=diff
==
--- tomcat/trunk/modules/jdbc-pool/build.xml (original)
+++ tomcat/trunk/modules/jdbc-pool/build.xml Fri Dec 19 16:45:46 2008
@@ -23,7 +23,7 @@
   
   
   
-  
+  
   
   
   

Modified: 
tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java?rev=728197&r1=728196&r2=728197&view=diff
==
--- 
tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
 (original)
+++ 
tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
 Fri Dec 19 16:45:46 2008
@@ -353,6 +353,9 @@
 PoolProperties.InterceptorDefinition[] proxies = 
getPoolProperties().getJdbcInterceptorsAsArray();
 for (int i=0; ihttp://svn.apache.org/viewvc/tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/PoolProperties.java?rev=728197&r1=728196&r2=728197&view=diff
==
--- 
tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/PoolProperties.java
 (original)
+++ 
tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/PoolProperties.java
 Fri Dec 19 16:45:46 2008
@@ -25,12 +25,16 @@
 import java.util.Properties;
 import java.util.concurrent.atomic.AtomicInteger;
 
+import org.apache.juli.logging.Log;
+import org.apache.juli.logging.LogFactory;
+
 /**
  * @author Filip Hanik
  *
  */
 public class PoolProperties {
 public static final String PKG_PREFIX = 
"org.apache.tomcat.jdbc.pool.interceptor.";
+protected static Log log = LogFactory.getLog(PoolProperties.class);
 
 protected static AtomicInteger poolCounter = new AtomicInteger(0);
 protected Properties dbProperties = new Properties();
@@ -460,8 +464,14 @@
 public Class getInterceptorClass() throws 
ClassNotFoundException {
 if (clazz==null) {
 if (getClassName().indexOf(".")<0) {
+if (log.isDebugEnabled()) {
+log.debug("Loading interceptor 
class:"+PoolProperties.PKG_PREFIX+getClassName());
+}
 clazz = 
Class.forName(PoolProperties.PKG_PREFIX+getClassName(), true, 
this.getClass().getClassLoader());
 } else {
+if (log.isDebugEnabled()) {
+log.debug("Loading interceptor class:"+getClassName());
+}
 clazz = Class.forName(getClassName(), true, 
this.getClass().getClassLoader());
 }
 }

Modified: 
tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java?rev=728197&r1=728196&r2=728197&view=diff
==
--- 
tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java
 (original)
+++ 
tomcat/trunk/modules/jdbc-pool/java/org/apache/tomcat/jdbc/pool/jmx/ConnectionPool.java
 Fri Dec 19 16:45:46 2008
@@ -37,6 +37,11 @@
 protected AtomicInteger sequence = new AtomicInteger(0);
 
 public ConnectionPool(org.apache.tomcat.jdbc.pool.ConnectionPool pool) {
+super();
+this.pool = pool;
+}
+
+public ConnectionPool(org.apache.tomcat.jdbc.pool.ConnectionPool pool, 
boolean for16) {
 super(getDefaultNotificationInfo());
 this.pool = pool;
 }

Modified: tomcat/trunk/modules/jdbc-pool/sign.sh
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/modules/jdbc-pool/sign.sh?rev=728197&r1=728196&r2=728197&view=diff
==
--- tomcat/trunk/modules/jdbc-pool/sign.sh (original)
+++ tomcat/trunk/modules/jdbc-pool/sign.sh Fri Dec 19 16:45:46 2008
@@ -1,4 +1,4 @@
-VERSION=v1.0.9-beta
+VERSION=v1.0.10-beta
 for i in $(find output/release/$VERSION -name "*.zip" -o -name "*.tar.gz"); do