[tomcat] branch 10.1.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 10.1.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git


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

commit fd8a2d098d64d50e56539142cc26b55114aa4fed
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
---
 .../main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java    | 6 ++++--
 webapps/docs/changelog.xml                                       | 9 ++++++++-
 2 files changed, 12 insertions(+), 3 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 30492c3971..c5714baf9c 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
@@ -373,8 +373,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 df8bf5f951..599c0abee1 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -105,8 +105,15 @@
   issues do not "pop up" wrt. others).
 -->
 <section name="Tomcat 10.1.15 (schultz)" 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 10.1.14 (schultz)" rtext="2023-10-10">
   <subsection name="Catalina">
     <changelog>


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