Fix the 'Test Connection' Button (#244)

jamesrobson-secondmind · web-flow · commit ed18810ab828 · 2022-07-27T16:50:27.000+02:00
diff --git a/src/main/java/com/datapipe/jenkins/vault/credentials/common/VaultHelper.java b/src/main/java/com/datapipe/jenkins/vault/credentials/common/VaultHelper.java
@@ -60,11 +60,11 @@ static Map<String, String> getVaultSecret(@NonNull String secretPath,
         try {
             VaultConfig vaultConfig = configuration.getVaultConfig();
 
-            if (prefixPath != null) {
+            if (prefixPath != null && ! prefixPath.isEmpty()) {
                 vaultConfig.prefixPath(prefixPath);
             }
 
-            if (namespace != null) {
+            if (namespace != null && ! namespace.isEmpty()) {
                 vaultConfig.nameSpace(namespace);
             }
 

Original file line number	Diff line number	Diff line change
`@@ -60,11 +60,11 @@ static Map<String, String> getVaultSecret(@NonNull String secretPath,`
`60`	`60`	`try {`
`61`	`61`	`VaultConfig vaultConfig = configuration.getVaultConfig();`
`62`	`62`
`63`		`- if (prefixPath != null) {`
	`63`	`+ if (prefixPath != null && ! prefixPath.isEmpty()) {`
`64`	`64`	`vaultConfig.prefixPath(prefixPath);`
`65`	`65`	`}`
`66`	`66`
`67`		`- if (namespace != null) {`
	`67`	`+ if (namespace != null && ! namespace.isEmpty()) {`
`68`	`68`	`vaultConfig.nameSpace(namespace);`
`69`	`69`	`}`
`70`	`70`