diff --git a/src/main/java/de/ozgcloud/formcycle/OzgCloudFormDataHttpClient.java b/src/main/java/de/ozgcloud/formcycle/OzgCloudFormDataHttpClient.java index 7c4df091468df7d72f1cac96cc6ac4b36980811e..6e0d7145f83c5edb76eb91c987c7054a7194fa1d 100644 --- a/src/main/java/de/ozgcloud/formcycle/OzgCloudFormDataHttpClient.java +++ b/src/main/java/de/ozgcloud/formcycle/OzgCloudFormDataHttpClient.java @@ -92,7 +92,7 @@ public class OzgCloudFormDataHttpClient { ProxyConfig getProxyConfig() { var proxyConfig = config.getProxyConfig(); - if (nonNull(proxyConfig.getHost())) { + if (isNoneBlank(proxyConfig.getHost())) { return proxyConfig; } if (isNull(systemProxyConfig)) { @@ -188,8 +188,10 @@ public class OzgCloudFormDataHttpClient { HttpClientContext createContext() { var clientContext = HttpClientContext.create(); - clientContext.setCredentialsProvider(getCredentialsProvider()); - clientContext.setAuthCache(createAuthCache()); + if (nonNull(getProxyConfig())) { + clientContext.setCredentialsProvider(getCredentialsProvider()); + clientContext.setAuthCache(createAuthCache()); + } return clientContext; } diff --git a/src/main/java/de/ozgcloud/formcycle/PluginBundleProperties.java b/src/main/java/de/ozgcloud/formcycle/PluginBundleProperties.java index 3d1719372c5340ea483ac9d28db30b7e15c04b77..bb3f81373257dfee3f913df329deb7a37276a8ff 100644 --- a/src/main/java/de/ozgcloud/formcycle/PluginBundleProperties.java +++ b/src/main/java/de/ozgcloud/formcycle/PluginBundleProperties.java @@ -45,12 +45,12 @@ public class PluginBundleProperties implements IBundleProperties { @Override public Map<String, IBundleConfigParam> getConfigProperties(IPluginResourceHelper resHelper, Locale currentLocale) { var configMap = new LinkedHashMap<String, IBundleConfigParam>(); - configMap.put(KEY_EINGANGSADAPTER_URL, new BundleConfigParam("Eingangsadapter URL", "", MANDATORY)); + configMap.put(KEY_EINGANGSADAPTER_URL, new BundleConfigParam(KEY_EINGANGSADAPTER_URL, "", MANDATORY)); configMap.put("proxy", new BundleConfigGroupItem("Konfiguration für den Proxy", "")); - configMap.put(KEY_PROXY_HOST, new BundleConfigParam("proxyHost", "", NOT_MANDATORY)); - configMap.put(KEY_PROXY_PORT, new BundleConfigParam("proxyPort", "", NOT_MANDATORY)); - configMap.put(KEY_PROXY_USER, new BundleConfigParam("proxyUser", "", NOT_MANDATORY)); - configMap.put(KEY_PROXY_PASSWORD, new BundleConfigParam("proxyPassword", "", NOT_MANDATORY, CRYPTIC_VALUE)); + configMap.put(KEY_PROXY_HOST, new BundleConfigParam(KEY_PROXY_HOST, "", NOT_MANDATORY)); + configMap.put(KEY_PROXY_PORT, new BundleConfigParam(KEY_PROXY_PORT, "", NOT_MANDATORY)); + configMap.put(KEY_PROXY_USER, new BundleConfigParam(KEY_PROXY_USER, "", NOT_MANDATORY)); + configMap.put(KEY_PROXY_PASSWORD, new BundleConfigParam(KEY_PROXY_PASSWORD, "", NOT_MANDATORY, CRYPTIC_VALUE)); return Collections.unmodifiableMap(configMap); } } diff --git a/src/test/java/de/ozgcloud/formcycle/OzgCloudFormDataHttpClientTest.java b/src/test/java/de/ozgcloud/formcycle/OzgCloudFormDataHttpClientTest.java index ee4bcecb557951174e8a710c71ef6e9c80522e40..bf92bb588bd208ad2578dcab1a754666b5e02548 100644 --- a/src/test/java/de/ozgcloud/formcycle/OzgCloudFormDataHttpClientTest.java +++ b/src/test/java/de/ozgcloud/formcycle/OzgCloudFormDataHttpClientTest.java @@ -439,6 +439,16 @@ class OzgCloudFormDataHttpClientTest { assertThat(result.getAuthCache()).isEqualTo(authCache); } + + @Test + void shouldReturnEmptyContext() { + doReturn(null).when(client).getProxyConfig(); + + client.createContext(); + + verify(client, never()).getCredentialsProvider(); + verify(client, never()).createAuthCache(); + } } } diff --git a/src/test/java/de/ozgcloud/formcycle/ProxyConfigTestFactory.java b/src/test/java/de/ozgcloud/formcycle/ProxyConfigTestFactory.java index 8247b0d9267ea774a3d12f3294a28ca7ba375ec3..ff4f389516a475205f31565174eecc96a5cc4cf5 100644 --- a/src/test/java/de/ozgcloud/formcycle/ProxyConfigTestFactory.java +++ b/src/test/java/de/ozgcloud/formcycle/ProxyConfigTestFactory.java @@ -27,7 +27,7 @@ public class ProxyConfigTestFactory { public static final Integer PROXY_PORT = Integer.parseInt(PropertiesTestFactory.PROXY_PORT); public static ProxyConfig createEmpty() { - return ProxyConfig.builder().build(); + return createBuilder().host("").port(null).user("").password("").build(); } public static ProxyConfig create() {