[tomcat] branch 9.0.x updated: Correct regression caused by clean-up of unnecessary FQCN.

[markt]

This is an automated email from the ASF dual-hosted git repository.

markt pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/9.0.x by this push:
     new e2aef02b8f Correct regression caused by clean-up of unnecessary FQCN.
e2aef02b8f is described below

commit e2aef02b8f6371f79aff16a7657465bce83074a8
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Tue Oct 10 11:13:11 2023 -0300

    Correct regression caused by clean-up of unnecessary FQCN.
    
    This usage is necessary to avoid confusion between
    javax.sql.PooledConnection and
    org.apache.tomcat.jdbc.pool.PooledConnection
---
 .../src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java | 6 ++++--
 webapps/docs/changelog.xml                                        | 8 ++++++++
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git 
a/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
 
b/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
index 479650c89c..3fb8d89fd1 100644
--- 
a/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
+++ 
b/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java
@@ -366,8 +366,10 @@ public class ConnectionPool {
         //cache the constructor
         if (proxyClassConstructor == null ) {
             Class<?> proxyClass = xa ?
-                Proxy.getProxyClass(ConnectionPool.class.getClassLoader(), new 
Class[] {Connection.class, PooledConnection.class, XAConnection.class}) :
-                Proxy.getProxyClass(ConnectionPool.class.getClassLoader(), new 
Class[] {Connection.class, PooledConnection.class});
+                    Proxy.getProxyClass(ConnectionPool.class.getClassLoader(),
+                            new Class[] {Connection.class, 
javax.sql.PooledConnection.class, XAConnection.class}) :
+                    Proxy.getProxyClass(ConnectionPool.class.getClassLoader(),
+                            new Class[] {Connection.class, 
javax.sql.PooledConnection.class});
             proxyClassConstructor = proxyClass.getConstructor(new Class[] { 
InvocationHandler.class });
         }
         return proxyClassConstructor;
diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index 0cb16afab5..988b1e6e91 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -105,6 +105,14 @@
   issues do not "pop up" wrt. others).
 -->
 <section name="Tomcat 9.0.82 (remm)" rtext="in development">
+  <subsection name="jdbc-pool">
+    <changelog>
+      <fix>
+        Correct a regression in the clean-up of unencessary use of fully
+        qualified class names in 10.1.14 that broke the jdbc-pool. (markt)
+      </fix>
+    </changelog>
+  </subsection>
 </section>
 <section name="Tomcat 9.0.81 (remm)" rtext="release in progress">
   <subsection name="Catalina">


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