Avoid using the constructor

quarkusio · Jul 12, 2023 · b36f073 · b36f073
1 parent 0229092
commit b36f073
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 10 deletions.
diff --git a/docs/src/main/asciidoc/security-openid-connect-multitenancy.adoc b/docs/src/main/asciidoc/security-openid-connect-multitenancy.adoc
@@ -707,8 +707,9 @@ import io.quarkus.oidc.TenantResolverInterceptor;
 @HrTenant
 @Interceptor
 public class HrInterceptor extends TenantResolverInterceptor {
-    public HrInterceptor() {
-        super("hr");
+    @Override
+    protected String getTenantId() {
+        return "hr";
     }
 }
 

diff --git a/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/TenantResolverInterceptor.java b/extensions/oidc/runtime/src/main/java/io/quarkus/oidc/TenantResolverInterceptor.java
@@ -10,15 +10,12 @@
 public abstract class TenantResolverInterceptor {
     @Inject
     RoutingContext routingContext;
-    private final String tenantId;
-
-    public TenantResolverInterceptor(String tenantId) {
-        this.tenantId = tenantId;
-    }
 
     @AroundInvoke
     public Object setTenant(InvocationContext context) throws Exception {
-        routingContext.put(OidcUtils.TENANT_ID_ATTRIBUTE, tenantId);
+        routingContext.put(OidcUtils.TENANT_ID_ATTRIBUTE, getTenantId());
         return context.proceed();
     }
+
+    protected abstract String getTenantId();
 }
diff --git a/integration-tests/oidc-wiremock/src/main/java/io/quarkus/it/keycloak/HrInterceptor.java b/integration-tests/oidc-wiremock/src/main/java/io/quarkus/it/keycloak/HrInterceptor.java
@@ -7,7 +7,10 @@
 @HrTenant
 @Interceptor
 public class HrInterceptor extends TenantResolverInterceptor {
-    public HrInterceptor() {
-        super("hr");
+
+    @Override
+    protected String getTenantId() {
+        return "hr";
     }
+
 }