Revert "4.4 to 5.2 upmerge 2024-08-29" (joomla#43998)

Author: Hackwar

tests/System/plugins/db.mjs

@@ -4,10 +4,6 @@ import postgres from 'postgres';
 // Items cache which are added by an insert statement
 let insertedItems = [];
 
-// Use of the PostgreSQL connection pool to limit the number of sessions, see
-// https://github.com/porsager/postgres?tab=readme-ov-file#connection-details
-let postgresConnectionPool = null;
-
 /**
  * Does run the given query against the database from the configuration. It caches all inserted items.
  *
@@ -33,20 +29,17 @@ function queryTestDB(joomlaQuery, config) {
     insertedItems.push(insertItem);
   }
 
-  // Do we use PostgreSQL?
+  // Check if the DB is from postgres
   if (config.env.db_type === 'pgsql' || config.env.db_type === 'PostgreSQL (PDO)') {
-
-    if (postgresConnectionPool === null) {
-      // Initialisation on the first call
-      postgresConnectionPool = postgres({
-        host: config.env.db_host,
-        port: config.env.db_port,
-        database: config.env.db_name,
-        username: config.env.db_user,
-        password: config.env.db_password,
-        max: 10, // Use only this (unchanged default) maximum number of connections in the pool
-      });
-    }
+    const connection = postgres({
+      host: config.env.db_host,
+      port: config.env.db_port,
+      database: config.env.db_name,
+      username: config.env.db_user,
+      password: config.env.db_password,
+      idle_timeout: 1,
+      max_lifetime: 1,
+    });
 
     // Postgres delivers the data direct as result of the insert query
     if (insertItem) {
@@ -56,7 +49,7 @@ function queryTestDB(joomlaQuery, config) {
     // Postgres needs double quotes
     query = query.replaceAll('`', '"');
 
-    return postgresConnectionPool.unsafe(query).then((result) => {
+    return connection.unsafe(query).then((result) => {
       // Select query should always return an array
       if (query.indexOf('SELECT') === 0 && !Array.isArray(result)) {
         return [result];
@@ -71,12 +64,12 @@ function queryTestDB(joomlaQuery, config) {
         insertItem.rows.push(result[0].id);
       }
 
-      // Normalize the object and return from PostgreSQL
+      // Normalize the object
       return { insertId: result[0].id };
     });
   }
 
-  // Return a promise which runs the query for MariaDB / MySQL
+  // Return a promise which runs the query
   return new Promise((resolve, reject) => {
     // Create the connection and connect
     const connection = mysql.createConnection({
@@ -101,7 +94,7 @@ function queryTestDB(joomlaQuery, config) {
         insertItem.rows.push(results.insertId);
       }
 
-      // Resolve the result from MariaDB / MySQL
+      // Resolve the result
       return resolve(results);
     });
   });