Merge pull request #5007 from poikilotherm/5006-enhance-secretdir

FISH-788 Support sub-directories for MPCONFIG SecretDirConfigSource. #5006
payara · Feb 1, 2021 · 0f5e522 · 0f5e522
1 parent fa786d5
commit 0f5e522
Show file tree

Hide file tree

Showing 7 changed files with 948 additions and 353 deletions.
diff --git a/...service/src/main/java/fish/payara/nucleus/microprofile/config/source/DirConfigSource.java b/...service/src/main/java/fish/payara/nucleus/microprofile/config/source/DirConfigSource.java
diff --git a/...vice/src/main/java/fish/payara/nucleus/microprofile/config/source/PayaraConfigSource.java b/...vice/src/main/java/fish/payara/nucleus/microprofile/config/source/PayaraConfigSource.java
@@ -67,4 +67,12 @@ public PayaraConfigSource() {
         configService = null;
     }
 
+    /**
+     * Should only be used for test purposes
+     * @param configService Usually a mocked implementation
+     */
+    PayaraConfigSource(ConfigProviderResolverImpl configService) {
+        this.domainConfiguration = null;
+        this.configService = configService;
+    }
 }
diff --git a/.../src/main/java/fish/payara/nucleus/microprofile/config/source/SecretsDirConfigSource.java b/.../src/main/java/fish/payara/nucleus/microprofile/config/source/SecretsDirConfigSource.java
diff --git a/...src/main/java/fish/payara/nucleus/microprofile/config/spi/ConfigProviderResolverImpl.java b/...src/main/java/fish/payara/nucleus/microprofile/config/spi/ConfigProviderResolverImpl.java
@@ -65,6 +65,7 @@
 import fish.payara.nucleus.microprofile.config.converters.CharacterConverter;
 import fish.payara.nucleus.microprofile.config.converters.ShortConverter;
 import fish.payara.nucleus.microprofile.config.source.PayaraExpressionConfigSource;
+import fish.payara.nucleus.executorservice.PayaraExecutorService;
 import org.eclipse.microprofile.config.Config;
 import org.eclipse.microprofile.config.spi.ConfigBuilder;
 import org.eclipse.microprofile.config.spi.ConfigProviderResolver;
@@ -102,7 +103,7 @@
 import fish.payara.nucleus.microprofile.config.source.PasswordAliasConfigSource;
 import fish.payara.nucleus.microprofile.config.source.PayaraServerProperties;
 import fish.payara.nucleus.microprofile.config.source.PropertiesConfigSource;
-import fish.payara.nucleus.microprofile.config.source.SecretsDirConfigSource;
+import fish.payara.nucleus.microprofile.config.source.DirConfigSource;
 import fish.payara.nucleus.microprofile.config.source.ServerConfigSource;
 import fish.payara.nucleus.microprofile.config.source.SystemPropertyConfigSource;
 
@@ -131,6 +132,10 @@ public class ConfigProviderResolverImpl extends ConfigProviderResolver {
 
     @Inject
     private ServerContext context;
+
+    // Some sources might want to execute background tasks in a controlled fashion
+    @Inject
+    private PayaraExecutorService executorService;
 
     // Gives access to deployed applications
     @Inject
@@ -284,6 +289,10 @@ public ConfigBuilder getBuilder() {
         return new PayaraConfigBuilder(this);
     }
 
+    public PayaraExecutorService getExecutor() {
+        return this.executorService;
+    }
+
     Config getNamedConfig(String applicationName) {
         Config result = null;
         ApplicationInfo info = applicationRegistry.get(applicationName);
@@ -309,7 +318,7 @@ private List<ConfigSource> getDefaultSources(String appName, String moduleName)
         sources.add(new SystemPropertyConfigSource());
         sources.add(new JNDIConfigSource());
         sources.add(new PayaraServerProperties());
-        sources.add(new SecretsDirConfigSource());
+        sources.add(new DirConfigSource());
         sources.add(new PasswordAliasConfigSource());
         if (appName != null) {
             sources.add(new ApplicationConfigSource(appName));