From c8bf3f573d82597d835ba51e20cbc59f3d6dec95 Mon Sep 17 00:00:00 2001
From: Titouan Compiegne <titouancompiegne@grvtfr1033.home>
Date: Tue, 30 May 2023 09:17:02 +0200
Subject: [PATCH] feat(idp): support Oracle database for JDBC IdP

closes AM-586
---
 .../gravitee-am-identityprovider-jdbc/pom.xml | 40 ++++++++++++++++---
 .../JdbcIdentityProviderConfiguration.java    |  7 +++-
 .../jdbc/user/JdbcUserProvider.java           | 11 +++--
 .../jdbc/utils/ColumnMapRowMapper.java        |  7 ++--
 .../jdbc/utils/ParametersUtils.java           |  4 +-
 .../src/main/resources/database/oracle.schema |  7 ++++
 .../main/resources/schemas/schema-form.json   | 10 ++++-
 ...JdbcAuthenticationProvider_OracleTest.java | 28 +++++++++++++
 ...ationProviderConfigurationTest_Oracle.java | 35 ++++++++++++++++
 .../user/JdbcUserProvider_OracleTest.java     | 28 +++++++++++++
 .../jdbc/JdbcReporterConfiguration.java       |  4 +-
 .../gravitee-am-repository-jdbc-api/pom.xml   |  2 +-
 .../impl/ConnectionFactoryProvider.java       | 20 +++++++---
 .../gravitee-am-repository-jdbc/pom.xml       | 28 ++++++++++++-
 .../AbstractTestRepositoryConfiguration.java  |  4 +-
 15 files changed, 204 insertions(+), 31 deletions(-)
 create mode 100644 gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/resources/database/oracle.schema
 create mode 100644 gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/test/java/io/gravitee/am/identityprovider/jdbc/authentication/JdbcAuthenticationProvider_OracleTest.java
 create mode 100644 gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/test/java/io/gravitee/am/identityprovider/jdbc/configuration/JdbcAuthenticationProviderConfigurationTest_Oracle.java
 create mode 100644 gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/test/java/io/gravitee/am/identityprovider/jdbc/user/JdbcUserProvider_OracleTest.java

diff --git a/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/pom.xml b/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/pom.xml
index 74eead03a5a..32e04b5dcf3 100644
--- a/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/pom.xml
+++ b/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/pom.xml
@@ -33,12 +33,13 @@
     <name>Gravitee IO - Access Management - Identity Provider - JDBC</name>
 
     <properties>
-        <r2dbc-pool.version>0.8.6.RELEASE</r2dbc-pool.version>
-        <r2dbc-postgresql.version>0.8.11.RELEASE</r2dbc-postgresql.version>
-        <r2dbc-mssql.version>0.8.7.RELEASE</r2dbc-mssql.version>
+        <r2dbc-pool.version>0.9.2.RELEASE</r2dbc-pool.version>
+        <r2dbc-postgresql.version>0.9.2.RELEASE</r2dbc-postgresql.version>
+        <r2dbc-mssql.version>0.9.0.RELEASE</r2dbc-mssql.version>
         <r2dbc-mysql.version>0.8.2.RELEASE</r2dbc-mysql.version>
-        <r2dbc-mariadb.version>1.0.0</r2dbc-mariadb.version>
-        <r2dbc-spi.version>0.8.4.RELEASE</r2dbc-spi.version>
+        <r2dbc-mariadb.version>1.1.2</r2dbc-mariadb.version>
+        <r2dbc-oracle.version>0.4.0</r2dbc-oracle.version>
+        <r2dbc-spi.version>0.9.1.RELEASE</r2dbc-spi.version>
         <spring-data-r2dbc.version>1.4.1</spring-data-r2dbc.version>
     </properties>
 
@@ -86,7 +87,7 @@
 
         <!-- R2DBC -->
         <dependency>
-            <groupId>io.r2dbc</groupId>
+            <groupId>org.postgresql</groupId>
             <artifactId>r2dbc-postgresql</artifactId>
             <version>${r2dbc-postgresql.version}</version>
             <exclusions>
@@ -145,6 +146,21 @@
                 </exclusion>
             </exclusions>
         </dependency>
+        <dependency>
+            <groupId>com.oracle.database.r2dbc</groupId>
+            <artifactId>oracle-r2dbc</artifactId>
+            <version>${r2dbc-oracle.version}</version>
+            <exclusions>
+                <exclusion>
+                    <artifactId>r2dbc-spi</artifactId>
+                    <groupId>io.r2dbc</groupId>
+                </exclusion>
+                <exclusion>
+                    <artifactId>reactor-core</artifactId>
+                    <groupId>io.projectreactor</groupId>
+                </exclusion>
+            </exclusions>
+        </dependency>
 
         <!-- Spring -->
         <dependency>
@@ -214,6 +230,18 @@
             <version>${test-container.version}</version>
             <scope>test</scope>
         </dependency>
+        <dependency>
+            <groupId>org.testcontainers</groupId>
+            <artifactId>oracle-xe</artifactId>
+            <version>1.18.1</version>
+            <scope>test</scope>
+        </dependency>
+        <dependency>
+            <groupId>com.oracle.database.jdbc</groupId>
+            <artifactId>ojdbc11</artifactId>
+            <version>21.7.0.0</version>
+            <scope>test</scope>
+        </dependency>
 
         <dependency>
             <!--
diff --git a/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/java/io/gravitee/am/identityprovider/jdbc/configuration/JdbcIdentityProviderConfiguration.java b/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/java/io/gravitee/am/identityprovider/jdbc/configuration/JdbcIdentityProviderConfiguration.java
index d530926012a..de80c93f9de 100644
--- a/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/java/io/gravitee/am/identityprovider/jdbc/configuration/JdbcIdentityProviderConfiguration.java
+++ b/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/java/io/gravitee/am/identityprovider/jdbc/configuration/JdbcIdentityProviderConfiguration.java
@@ -45,6 +45,7 @@ public class JdbcIdentityProviderConfiguration implements IdentityProviderConfig
     private String selectUserByEmailQuery;
     private String identifierAttribute = FIELD_ID;
     private String emailAttribute = FIELD_EMAIL;
+    private String metadataAttribute = FIELD_METADATA;
     private String usernameAttribute = FIELD_USERNAME;
     private String passwordAttribute = FIELD_PASSWORD;
     private String passwordEncoder = PasswordEncoder.BCRYPT;
@@ -227,7 +228,11 @@ public void setEmailAttribute(String emailAttribute) {
     }
 
     public String getMetadataAttribute() {
-        return FIELD_METADATA;
+        return metadataAttribute;
+    }
+
+    public void setMetadataAttribute(String metadataAttribute) {
+        this.metadataAttribute = metadataAttribute;
     }
 
     public boolean getAutoProvisioning() {
diff --git a/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/java/io/gravitee/am/identityprovider/jdbc/user/JdbcUserProvider.java b/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/java/io/gravitee/am/identityprovider/jdbc/user/JdbcUserProvider.java
index 047e12d530a..6c0f7c45d91 100644
--- a/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/java/io/gravitee/am/identityprovider/jdbc/user/JdbcUserProvider.java
+++ b/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/java/io/gravitee/am/identityprovider/jdbc/user/JdbcUserProvider.java
@@ -169,10 +169,13 @@ public Single<UserProvider> asyncStart() {
     }
 
     private String tableExists(String protocol, String table) {
-        if ("sqlserver".equalsIgnoreCase(protocol)) {
-            return "SELECT 1 FROM sysobjects WHERE name = '" + table + "' AND xtype = 'U'";
-        } else {
-            return "SELECT 1 FROM information_schema.tables WHERE table_name = '" + table + "'";
+        switch (protocol) {
+            case "sqlserver" :
+                return "SELECT 1 FROM sysobjects WHERE name = '" + table + "' AND xtype = 'U'";
+            case "oracle":
+                return "SELECT 1 from user_tables where table_name = '" + table + "'";
+            default:
+                return "SELECT 1 FROM information_schema.tables WHERE table_name = '" + table + "'";
         }
     }
 
diff --git a/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/java/io/gravitee/am/identityprovider/jdbc/utils/ColumnMapRowMapper.java b/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/java/io/gravitee/am/identityprovider/jdbc/utils/ColumnMapRowMapper.java
index b39a729a430..e9f011832ab 100644
--- a/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/java/io/gravitee/am/identityprovider/jdbc/utils/ColumnMapRowMapper.java
+++ b/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/java/io/gravitee/am/identityprovider/jdbc/utils/ColumnMapRowMapper.java
@@ -29,10 +29,9 @@
 public final class ColumnMapRowMapper {
 
     public static Map<String, Object> mapRow(Row row, RowMetadata rowMetadata) {
-        Map<String, Object> claims = new HashMap<>(rowMetadata.getColumnNames()
+        return new HashMap<>(rowMetadata.getColumnMetadatas()
                 .stream()
-                .filter(c -> row.get(c) != null)
-                .collect(Collectors.toMap(c -> c, c -> row.get(c))));
-        return claims;
+                .filter(c -> row.get(c.getName()) != null)
+                .collect(Collectors.toMap(c -> c.getName(), c -> row.get(c.getName()))));
     }
 }
diff --git a/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/java/io/gravitee/am/identityprovider/jdbc/utils/ParametersUtils.java b/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/java/io/gravitee/am/identityprovider/jdbc/utils/ParametersUtils.java
index b941f73b51d..e8cfa6752b0 100644
--- a/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/java/io/gravitee/am/identityprovider/jdbc/utils/ParametersUtils.java
+++ b/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/java/io/gravitee/am/identityprovider/jdbc/utils/ParametersUtils.java
@@ -15,8 +15,6 @@
  */
 package io.gravitee.am.identityprovider.jdbc.utils;
 
-import io.r2dbc.spi.Statement;
-
 /**
  * @author Titouan COMPIEGNE (titouan.compiegne at graviteesource.com)
  * @author GraviteeSource Team
@@ -32,6 +30,8 @@ public static String getIndexParameter(String database, int index, String field)
                 return "$" + index;
             case "sqlserver":
                 return "@" + field;
+            case "oracle":
+                return ":" + field;
             default:
                 return "" + index;
         }
diff --git a/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/resources/database/oracle.schema b/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/resources/database/oracle.schema
new file mode 100644
index 00000000000..6b852841fa5
--- /dev/null
+++ b/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/resources/database/oracle.schema
@@ -0,0 +1,7 @@
+--
+-- Script to create Oracle schema
+-- Table & index names are suffixed by 3 underscores to define a
+-- pattern to replace with the domain name
+--
+CREATE TABLE idp_users___ (id NVARCHAR2(64) NOT NULL, username NVARCHAR2(320) NOT NULL, password NVARCHAR2(255) NULL, email NVARCHAR2(320) NULL, metadata CLOB NULL, PRIMARY KEY (id))
+CREATE INDEX idp_users____username_idx ON users(username)
\ No newline at end of file
diff --git a/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/resources/schemas/schema-form.json b/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/resources/schemas/schema-form.json
index 20c6fd715b6..389315ceb11 100644
--- a/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/resources/schemas/schema-form.json
+++ b/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/main/resources/schemas/schema-form.json
@@ -14,9 +14,9 @@
     },
     "protocol" : {
       "type": "string",
-      "enum": ["postgresql", "mysql", "sqlserver", "mariadb"],
+      "enum": ["postgresql", "mysql", "sqlserver", "mariadb", "oracle"],
       "title": "Database driver identifier",
-      "description": "Type of your relational database (RDBMS). PostgreSQL, MySQL, SQL Server, MariaDB, ..."
+      "description": "Type of your relational database (RDBMS). PostgreSQL, MySQL, SQL Server, MariaDB, Oracle"
     },
     "database" : {
       "type" : "string",
@@ -79,6 +79,12 @@
       "title": "User email attribute",
       "description": "Email field of your users"
     },
+    "metadataAttribute" : {
+      "type" : "string",
+      "default": "metadata",
+      "title": "User metadata attribute",
+      "description": "Additional information field of your users"
+    },
     "passwordAttribute" : {
       "type" : "string",
       "default": "password",
diff --git a/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/test/java/io/gravitee/am/identityprovider/jdbc/authentication/JdbcAuthenticationProvider_OracleTest.java b/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/test/java/io/gravitee/am/identityprovider/jdbc/authentication/JdbcAuthenticationProvider_OracleTest.java
new file mode 100644
index 00000000000..6963d987a64
--- /dev/null
+++ b/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/test/java/io/gravitee/am/identityprovider/jdbc/authentication/JdbcAuthenticationProvider_OracleTest.java
@@ -0,0 +1,28 @@
+/**
+ * Copyright (C) 2015 The Gravitee team (http://gravitee.io)
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *         http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package io.gravitee.am.identityprovider.jdbc.authentication;
+
+import io.gravitee.am.identityprovider.jdbc.authentication.spring.JdbcAuthenticationProviderConfiguration;
+import io.gravitee.am.identityprovider.jdbc.configuration.JdbcAuthenticationProviderConfigurationTest_Oracle;
+import org.springframework.test.context.ContextConfiguration;
+import org.springframework.test.context.support.AnnotationConfigContextLoader;
+
+/**
+ * @author Titouan COMPIEGNE (titouan.compiegne at graviteesource.com)
+ * @author GraviteeSource Team
+ */
+@ContextConfiguration(classes = { JdbcAuthenticationProviderConfigurationTest_Oracle.class, JdbcAuthenticationProviderConfiguration.class }, loader = AnnotationConfigContextLoader.class)
+public class JdbcAuthenticationProvider_OracleTest extends JdbcAuthenticationProviderTest { }
diff --git a/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/test/java/io/gravitee/am/identityprovider/jdbc/configuration/JdbcAuthenticationProviderConfigurationTest_Oracle.java b/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/test/java/io/gravitee/am/identityprovider/jdbc/configuration/JdbcAuthenticationProviderConfigurationTest_Oracle.java
new file mode 100644
index 00000000000..edbbdc4cb8f
--- /dev/null
+++ b/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/test/java/io/gravitee/am/identityprovider/jdbc/configuration/JdbcAuthenticationProviderConfigurationTest_Oracle.java
@@ -0,0 +1,35 @@
+/**
+ * Copyright (C) 2015 The Gravitee team (http://gravitee.io)
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *         http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package io.gravitee.am.identityprovider.jdbc.configuration;
+
+import org.springframework.context.annotation.Configuration;
+
+/**
+ * @author Titouan COMPIEGNE (titouan.compiegne at graviteesource.com)
+ * @author GraviteeSource Team
+ */
+@Configuration
+public class JdbcAuthenticationProviderConfigurationTest_Oracle extends JdbcAuthenticationProviderConfigurationTest {
+
+    public String url() {
+        return "r2dbc:tc:oracle:///db?TC_IMAGE_TAG=21-slim-faststart";
+    }
+
+    @Override
+    public String protocol() {
+        return "oracle";
+    }
+}
diff --git a/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/test/java/io/gravitee/am/identityprovider/jdbc/user/JdbcUserProvider_OracleTest.java b/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/test/java/io/gravitee/am/identityprovider/jdbc/user/JdbcUserProvider_OracleTest.java
new file mode 100644
index 00000000000..4cb43bccd3c
--- /dev/null
+++ b/gravitee-am-identityprovider/gravitee-am-identityprovider-jdbc/src/test/java/io/gravitee/am/identityprovider/jdbc/user/JdbcUserProvider_OracleTest.java
@@ -0,0 +1,28 @@
+/**
+ * Copyright (C) 2015 The Gravitee team (http://gravitee.io)
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *         http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package io.gravitee.am.identityprovider.jdbc.user;
+
+import io.gravitee.am.identityprovider.jdbc.configuration.JdbcAuthenticationProviderConfigurationTest_Oracle;
+import io.gravitee.am.identityprovider.jdbc.user.spring.JdbcUserProviderConfiguration;
+import org.springframework.test.context.ContextConfiguration;
+import org.springframework.test.context.support.AnnotationConfigContextLoader;
+
+/**
+ * @author Titouan COMPIEGNE (titouan.compiegne at graviteesource.com)
+ * @author GraviteeSource Team
+ */
+@ContextConfiguration(classes = { JdbcAuthenticationProviderConfigurationTest_Oracle.class, JdbcUserProviderConfiguration.class }, loader = AnnotationConfigContextLoader.class)
+public class JdbcUserProvider_OracleTest extends JdbcUserProvider_Test { }
diff --git a/gravitee-am-reporter/gravitee-am-reporter-jdbc/src/main/java/io/gravitee/am/reporter/jdbc/JdbcReporterConfiguration.java b/gravitee-am-reporter/gravitee-am-reporter-jdbc/src/main/java/io/gravitee/am/reporter/jdbc/JdbcReporterConfiguration.java
index 0e61ea5d21e..79e516f42a5 100644
--- a/gravitee-am-reporter/gravitee-am-reporter-jdbc/src/main/java/io/gravitee/am/reporter/jdbc/JdbcReporterConfiguration.java
+++ b/gravitee-am-reporter/gravitee-am-reporter-jdbc/src/main/java/io/gravitee/am/reporter/jdbc/JdbcReporterConfiguration.java
@@ -35,8 +35,8 @@ public class JdbcReporterConfiguration implements ReporterConfiguration {
     private Integer maxSize = 10;
     private Integer maxIdleTime = 30000;
     private Integer maxLifeTime = 30000;
-    private Integer maxAcquireTime = 0 ;
-    private Integer maxCreateConnectionTime = 0;
+    private Integer maxAcquireTime = -1;
+    private Integer maxCreateConnectionTime = -1;
     private String validationQuery = "SELECT 1";
     private String tableSuffix;
     private Integer bulkActions = 1000;
diff --git a/gravitee-am-repository/gravitee-am-repository-jdbc-api/pom.xml b/gravitee-am-repository/gravitee-am-repository-jdbc-api/pom.xml
index eb922d70b5c..a903d1b0313 100644
--- a/gravitee-am-repository/gravitee-am-repository-jdbc-api/pom.xml
+++ b/gravitee-am-repository/gravitee-am-repository-jdbc-api/pom.xml
@@ -30,7 +30,7 @@
     <name>Gravitee IO - Access Management - Repository JDBC API</name>
 
     <properties>
-        <r2dbc-releasetrain.version>Arabba-SR13</r2dbc-releasetrain.version>
+        <r2dbc-releasetrain.version>Borca-SR2</r2dbc-releasetrain.version>
         <reactor-adapter.version>3.4.9</reactor-adapter.version>
         <reactor-core.version>3.4.13</reactor-core.version>
         <spring-data-r2dbc.version>1.4.1</spring-data-r2dbc.version>
diff --git a/gravitee-am-repository/gravitee-am-repository-jdbc-api/src/main/java/io/gravitee/am/repository/jdbc/provider/impl/ConnectionFactoryProvider.java b/gravitee-am-repository/gravitee-am-repository-jdbc-api/src/main/java/io/gravitee/am/repository/jdbc/provider/impl/ConnectionFactoryProvider.java
index 7838e6cf500..9408642a6a3 100644
--- a/gravitee-am-repository/gravitee-am-repository-jdbc-api/src/main/java/io/gravitee/am/repository/jdbc/provider/impl/ConnectionFactoryProvider.java
+++ b/gravitee-am-repository/gravitee-am-repository-jdbc-api/src/main/java/io/gravitee/am/repository/jdbc/provider/impl/ConnectionFactoryProvider.java
@@ -24,6 +24,7 @@
 import io.micrometer.core.instrument.Tag;
 import io.micrometer.core.instrument.Tags;
 import io.r2dbc.pool.ConnectionPool;
+import io.r2dbc.pool.ConnectionPoolConfiguration;
 import io.r2dbc.pool.PoolingConnectionFactoryProvider;
 import io.r2dbc.spi.ConnectionFactories;
 import io.r2dbc.spi.ConnectionFactory;
@@ -91,8 +92,8 @@ public static ConnectionFactory createClient(R2DBCConnectionConfiguration config
                 .option(PoolingConnectionFactoryProvider.MAX_SIZE, 10)
                 .option(PoolingConnectionFactoryProvider.MAX_IDLE_TIME, Duration.of(30000l, ChronoUnit.MILLIS))
                 .option(PoolingConnectionFactoryProvider.MAX_LIFE_TIME, Duration.of(30000l, ChronoUnit.MILLIS))
-                .option(PoolingConnectionFactoryProvider.MAX_ACQUIRE_TIME, Duration.of(0, ChronoUnit.MILLIS))
-                .option(PoolingConnectionFactoryProvider.MAX_CREATE_CONNECTION_TIME, Duration.of(0, ChronoUnit.MILLIS))
+                .option(PoolingConnectionFactoryProvider.MAX_ACQUIRE_TIME, ConnectionPoolConfiguration.NO_TIMEOUT)
+                .option(PoolingConnectionFactoryProvider.MAX_CREATE_CONNECTION_TIME, ConnectionPoolConfiguration.NO_TIMEOUT)
                 .option(PoolingConnectionFactoryProvider.VALIDATION_DEPTH, ValidationDepth.LOCAL);
 
         List<Map<String, String>> options = configuration.getOptions();
@@ -104,7 +105,16 @@ public static ConnectionFactory createClient(R2DBCConnectionConfiguration config
             });
         }
 
-        ConnectionPool connectionPool = (ConnectionPool) ConnectionFactories.get(builder.build());
+        ConnectionPool connectionPool;
+        // TCCL has been introduced to support Oracle driver loading
+        ClassLoader origLoader = Thread.currentThread().getContextClassLoader();
+        try {
+            ClassLoader goldenLoader = ConnectionFactories.class.getClassLoader();
+            Thread.currentThread().setContextClassLoader(goldenLoader);
+            connectionPool = (ConnectionPool) ConnectionFactories.get(builder.build());
+        } finally {
+            Thread.currentThread().setContextClassLoader(origLoader);
+        }
         LOGGER.info("Connection pool created for database server {} on host {}", configuration.getProtocol(), configuration.getHost());
 
         final Tags tags = Tags.of(
@@ -149,8 +159,8 @@ public ConnectionFactory factory() {
                     .option(PoolingConnectionFactoryProvider.MAX_SIZE, Integer.parseInt(environment.getProperty(prefix+"maxSize", "10")))
                     .option(PoolingConnectionFactoryProvider.MAX_IDLE_TIME, Duration.of(Long.parseLong(environment.getProperty(prefix+"maxIdleTime", "30000")), ChronoUnit.MILLIS))
                     .option(PoolingConnectionFactoryProvider.MAX_LIFE_TIME, Duration.of(Long.parseLong(environment.getProperty(prefix+"maxLifeTime", "30000")), ChronoUnit.MILLIS))
-                    .option(PoolingConnectionFactoryProvider.MAX_ACQUIRE_TIME, Duration.of(Long.parseLong(environment.getProperty(prefix+"maxAcquireTime", "0")), ChronoUnit.MILLIS))
-                    .option(PoolingConnectionFactoryProvider.MAX_CREATE_CONNECTION_TIME, Duration.of(Long.parseLong(environment.getProperty(prefix+"maxCreateConnectionTime", "0")), ChronoUnit.MILLIS));
+                    .option(PoolingConnectionFactoryProvider.MAX_ACQUIRE_TIME, Duration.of(Long.parseLong(environment.getProperty(prefix+"maxAcquireTime", "-1")), ChronoUnit.MILLIS))
+                    .option(PoolingConnectionFactoryProvider.MAX_CREATE_CONNECTION_TIME, Duration.of(Long.parseLong(environment.getProperty(prefix+"maxCreateConnectionTime", "-1")), ChronoUnit.MILLIS));
 
             if (port != null) {
                 builder.option(PORT, Integer.parseInt(port));
diff --git a/gravitee-am-repository/gravitee-am-repository-jdbc/pom.xml b/gravitee-am-repository/gravitee-am-repository-jdbc/pom.xml
index 28f24e98b8c..1042c625cfa 100644
--- a/gravitee-am-repository/gravitee-am-repository-jdbc/pom.xml
+++ b/gravitee-am-repository/gravitee-am-repository-jdbc/pom.xml
@@ -31,12 +31,13 @@
     <name>Gravitee IO - Access Management - Repository JDBC</name>
 
     <properties>
-        <r2dbc-releasetrain.version>Arabba-SR13</r2dbc-releasetrain.version>
+        <r2dbc-releasetrain.version>Borca-SR2</r2dbc-releasetrain.version>
 
         <jdbc-postgresql.version>42.2.25</jdbc-postgresql.version>
         <jdbc-mssql.version>7.4.1.jre8</jdbc-mssql.version>
         <jdbc-mysql.version>8.0.29</jdbc-mysql.version>
         <jdbc-mariadb.version>2.4.0</jdbc-mariadb.version>
+        <jdbc-oracle.version>21.7.0.0</jdbc-oracle.version>
 
         <dozer.version>6.5.2</dozer.version>
 
@@ -130,7 +131,7 @@
 
         <!-- R2DBC -->
         <dependency>
-            <groupId>io.r2dbc</groupId>
+            <groupId>org.postgresql</groupId>
             <artifactId>r2dbc-postgresql</artifactId>
             <exclusions>
                 <exclusion>
@@ -163,6 +164,7 @@
         <dependency>
             <groupId>dev.miku</groupId>
             <artifactId>r2dbc-mysql</artifactId>
+            <version>0.8.2.RELEASE</version>
             <exclusions>
                 <exclusion>
                     <artifactId>reactor-core</artifactId>
@@ -190,6 +192,21 @@
             </exclusions>
             <scope>provided</scope>
         </dependency>
+        <dependency>
+            <groupId>com.oracle.database.r2dbc</groupId>
+            <artifactId>oracle-r2dbc</artifactId>
+            <exclusions>
+                <exclusion>
+                    <artifactId>r2dbc-spi</artifactId>
+                    <groupId>io.r2dbc</groupId>
+                </exclusion>
+                <exclusion>
+                    <artifactId>reactor-core</artifactId>
+                    <groupId>io.projectreactor</groupId>
+                </exclusion>
+            </exclusions>
+            <scope>provided</scope>
+        </dependency>
 
         <dependency>
             <!--
@@ -224,6 +241,13 @@
             <scope>provided</scope>
         </dependency>
 
+        <dependency>
+            <groupId>com.oracle.database.jdbc</groupId>
+            <artifactId>ojdbc11</artifactId>
+            <version>${jdbc-oracle.version}</version>
+            <scope>provided</scope>
+        </dependency>
+
 
         <!-- bean mapper -->
         <dependency>
diff --git a/gravitee-am-repository/gravitee-am-repository-jdbc/src/test/java/io/gravitee/am/repository/jdbc/common/AbstractTestRepositoryConfiguration.java b/gravitee-am-repository/gravitee-am-repository-jdbc/src/test/java/io/gravitee/am/repository/jdbc/common/AbstractTestRepositoryConfiguration.java
index 1b95acd5324..1d8e77644a5 100644
--- a/gravitee-am-repository/gravitee-am-repository-jdbc/src/test/java/io/gravitee/am/repository/jdbc/common/AbstractTestRepositoryConfiguration.java
+++ b/gravitee-am-repository/gravitee-am-repository-jdbc/src/test/java/io/gravitee/am/repository/jdbc/common/AbstractTestRepositoryConfiguration.java
@@ -64,7 +64,7 @@ protected void initializeDatabaseSchema(ConnectionFactoryOptions options, Enviro
             final String jdbcUrl = container.getJdbcUrl();
 
             try (Connection connection = DriverManager.getConnection(jdbcUrl,
-                    options.getValue(USER), options.getValue(PASSWORD).toString())) {
+                    options.getValue(USER).toString(), options.getValue(PASSWORD).toString())) {
                 LOGGER.debug("Running Liquibase on {}", jdbcUrl);
                 runLiquibase(connection);
                 runLiquibase_addSpecificTestIndexes(connection);
@@ -124,7 +124,7 @@ public ConnectionFactory connectionFactory() {
         options = ConnectionFactoryOptions.builder()
                 .from(options)
                 .option(DRIVER, "pool")
-                .option(PROTOCOL, options.getValue(DRIVER))
+                .option(PROTOCOL, options.getValue(DRIVER).toString())
                 .build();
         return ConnectionFactories.get(options);
     }