From eab80688cce76f66583e870abb1b8f6131c78720 Mon Sep 17 00:00:00 2001 From: Michael Jumper Date: Mon, 21 Jun 2021 14:44:09 -0700 Subject: [PATCH] GUACAMOLE-641: Ensure MyBatis "expectedConnectionTypeCode" is generated and matches valid database connections. --- .../auth/jdbc/DynamicallyAuthenticatedDataSource.java | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc/DynamicallyAuthenticatedDataSource.java b/extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc/DynamicallyAuthenticatedDataSource.java index 1936d1e96..4c8e3c0f8 100644 --- a/extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc/DynamicallyAuthenticatedDataSource.java +++ b/extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/apache/guacamole/auth/jdbc/DynamicallyAuthenticatedDataSource.java @@ -80,6 +80,13 @@ public class DynamicallyAuthenticatedDataSource extends PooledDataSource { }); + // Force recalculation of expectedConnectionTypeCode. The + // PooledDataSource constructor accepting a single UnpooledDataSource + // will otherwise leave this value uninitialized, resulting in all + // connections failing to pass sanity checks and never being returned + // to the pool. + super.forceCloseAll(); + } @Override