Hibernate Offline Startup: drop CDI req ctx workaround

Author: michalvavrik

hibernate/hibernate-offline-startup/README.MD

@@ -23,5 +23,3 @@ Tests are motivated by the [QUARKUS-3363](https://issues.redhat.com/browse/QUARK
     * We create a schema using the Hibernate ORM SchemaManager, which is not supported for multitenancy (but works like a charm...). However, if we disable telemetry for the req_scope_credentials datasource and keep it for the app_scope_credentials datasource while both datasources are used by the same Hibernate persistence unit, traces for both schemes generation are present. I have tried to create a reproducer with supported scenarios and couldn’t. Therefore, we test telemetry for all tenants (datasources) resolved for the default persistence unit.
 * Detection of build-time configuration properties for forced dependencies
     * Current mechanism used by our framework to detect build-time properties relies on a classpath during the testing framework bootstrap. Thus, we either need to hardcode a list of build-time properties for these dependencies in our framework or put the build-time properties in our application.properties file. That’s the reason for having the `quarkus.hibernate-orm.dialect.mariadb.bytes-per-character` configuration property defined for all the test classes.
-* CDI Request Context
-    * Due to the https://github.com/quarkusio/quarkus/issues/50154, test that should verify credentials propagation using CDI Request Context is storing state in static variables. This would not work in a concurrent environment.

hibernate/hibernate-offline-startup/src/main/java/io/quarkus/ts/hibernate/startup/offline/pu/defaults/orm/CustomCredentialsProvider.java

@@ -4,7 +4,6 @@
 import java.util.concurrent.ConcurrentHashMap;
 
 import jakarta.enterprise.context.ApplicationScoped;
-import jakarta.enterprise.context.control.ActivateRequestContext;
 
 import io.quarkus.credentials.CredentialsProvider;
 import io.quarkus.logging.Log;
@@ -22,8 +21,6 @@ private record UsernameAndPassword(String username, String password) {
         this.requestScopedData = requestScopedData;
     }
 
-    // TODO: drop the activation when https://github.com/quarkusio/quarkus/issues/50154 is fixed
-    @ActivateRequestContext
     @Override
     public Map<String, String> getCredentials(String credentialsProviderName) {
         if (requestScopedData.foundCredentials()) {

hibernate/hibernate-offline-startup/src/main/java/io/quarkus/ts/hibernate/startup/offline/pu/defaults/orm/RequestScopedData.java

@@ -8,10 +8,8 @@
 @RequestScoped
 public final class RequestScopedData {
 
-    // TODO: make this member variables instead when https://github.com/quarkusio/quarkus/issues/50154 is fixed
-    //      or get the credentials directly from the RoutingContext instead
-    private static volatile String user;
-    private static volatile String password;
+    private volatile String user;
+    private volatile String password;
 
     @Inject
     RoutingContext routingContext;