Skip to content

Commit

Permalink
Don't override proxy configuration when saving the settings, if the h…
Browse files Browse the repository at this point in the history
…ttp proxy is configured in Java system properties
  • Loading branch information
josegar74 committed Sep 7, 2023
1 parent 79865d1 commit 5175d11
Show file tree
Hide file tree
Showing 8 changed files with 222 additions and 42 deletions.
6 changes: 3 additions & 3 deletions core/src/main/java/org/fao/geonet/kernel/url/UrlChecker.java
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
//=============================================================================
//=== Copyright (C) 2001-2019 Food and Agriculture Organization of the
//=== Copyright (C) 2001-2023 Food and Agriculture Organization of the
//=== United Nations (FAO-UN), United Nations World Food Programme (WFP)
//=== and United Nations Environment Programme (UNEP)
//===
Expand Down Expand Up @@ -31,6 +31,7 @@
import org.fao.geonet.constants.Geonet;
import org.fao.geonet.domain.LinkStatus;
import org.fao.geonet.kernel.setting.SettingManager;
import org.fao.geonet.lib.Lib;
import org.fao.geonet.lib.NetLib;
import org.fao.geonet.utils.GeonetHttpRequestFactory;
import org.springframework.beans.factory.annotation.Autowired;
Expand Down Expand Up @@ -149,8 +150,7 @@ public Void apply(@Nullable HttpClientBuilder originalConfig) {
Log.info(Geonet.GEONETWORK,"UrlChecker: cannot determine hostname from url: "+url);
}
//now we have hostname, we can configure proxy
NetLib netLib = new NetLib();
netLib.setupProxy(settingManager, originalConfig, hostname);
Lib.net.setupProxy(settingManager, originalConfig, hostname);
return null;
}
};
Expand Down
73 changes: 47 additions & 26 deletions core/src/main/java/org/fao/geonet/lib/NetLib.java
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
//=============================================================================
//=== Copyright (C) 2001-2022 Food and Agriculture Organization of the
//=== Copyright (C) 2001-2023 Food and Agriculture Organization of the
//=== United Nations (FAO-UN), United Nations World Food Programme (WFP)
//=== and United Nations Environment Programme (UNEP)
//===
Expand Down Expand Up @@ -34,7 +34,6 @@
import org.fao.geonet.GeonetContext;
import org.fao.geonet.constants.Geonet;
import org.fao.geonet.kernel.setting.SettingManager;
import org.fao.geonet.kernel.setting.Settings;
import org.fao.geonet.utils.Log;
import org.fao.geonet.utils.XmlRequest;

Expand All @@ -51,6 +50,18 @@
import jeeves.server.context.ServiceContext;

public class NetLib {
private ProxyConfiguration proxyConfiguration;

public ProxyConfiguration getProxyConfiguration() {
return proxyConfiguration;
}

public NetLib() {
boolean isProxyConfiguredInSystemProperties = StringUtils.isNotBlank(System.getProperty("http.proxyHost")) ||
StringUtils.isNotBlank(System.getProperty("https.proxyHost"));

proxyConfiguration = new ProxyConfiguration(isProxyConfiguredInSystemProperties);
}

public void setupProxy(ServiceContext context, XmlRequest req) {
GeonetContext gc = (GeonetContext) context.getHandlerContext(Geonet.CONTEXT_NAME);
Expand All @@ -66,12 +77,14 @@ public void setupProxy(ServiceContext context, XmlRequest req) {
*/

public void setupProxy(SettingManager sm, XmlRequest req) {
boolean enabled = sm.getValueAsBool(Settings.SYSTEM_PROXY_USE, false);
String host = sm.getValue(Settings.SYSTEM_PROXY_HOST);
String port = sm.getValue(Settings.SYSTEM_PROXY_PORT);
String username = sm.getValue(Settings.SYSTEM_PROXY_USERNAME);
String password = sm.getValue(Settings.SYSTEM_PROXY_PASSWORD);
String ignoreHostList = sm.getValue(Settings.SYSTEM_PROXY_IGNOREHOSTLIST);
proxyConfiguration.refresh(sm);

boolean enabled = proxyConfiguration.isEnabled();
String host = proxyConfiguration.getHost();
String port = proxyConfiguration.getPort();
String username = proxyConfiguration.getUsername();
String password = proxyConfiguration.getPassword();
String ignoreHostList = proxyConfiguration.getIgnoreHostList();

if (!enabled) {
req.setUseProxy(false);
Expand Down Expand Up @@ -108,12 +121,14 @@ public CredentialsProvider setupProxy(ServiceContext context, HttpClientBuilder
* Setup proxy for http client
*/
public CredentialsProvider setupProxy(SettingManager sm, HttpClientBuilder client, String requestHost) {
boolean enabled = sm.getValueAsBool(Settings.SYSTEM_PROXY_USE, false);
String host = sm.getValue(Settings.SYSTEM_PROXY_HOST);
String port = sm.getValue(Settings.SYSTEM_PROXY_PORT);
String username = sm.getValue(Settings.SYSTEM_PROXY_USERNAME);
String password = sm.getValue(Settings.SYSTEM_PROXY_PASSWORD);
String ignoreHostList = sm.getValue(Settings.SYSTEM_PROXY_IGNOREHOSTLIST);
proxyConfiguration.refresh(sm);

boolean enabled = proxyConfiguration.isEnabled();
String host = proxyConfiguration.getHost();
String port = proxyConfiguration.getPort();
String username = proxyConfiguration.getUsername();
String password = proxyConfiguration.getPassword();
String ignoreHostList = proxyConfiguration.getIgnoreHostList();

CredentialsProvider provider = new BasicCredentialsProvider();
if (enabled) {
Expand Down Expand Up @@ -153,13 +168,17 @@ public void setupProxy(ServiceContext context) {
* Setup proxy for http client
*/
public void setupProxy(SettingManager sm) {
boolean useProxy = sm.getValueAsBool(Settings.SYSTEM_PROXY_USE, false);
proxyConfiguration.refresh(sm);

if (useProxy) {
String host = sm.getValue(Settings.SYSTEM_PROXY_HOST);
String port = sm.getValue(Settings.SYSTEM_PROXY_PORT);
String username = sm.getValue(Settings.SYSTEM_PROXY_USERNAME);
String ignoreHostList = sm.getValue(Settings.SYSTEM_PROXY_IGNOREHOSTLIST);
// If the proxy is configured in the system properties,
// ignore the proxy configuration in the system settings.
if (proxyConfiguration.isProxyConfiguredInSystemProperties()) return;

if (proxyConfiguration.isEnabled()) {
String host = proxyConfiguration.getHost();
String port = proxyConfiguration.getPort();
String username = proxyConfiguration.getUsername();
String ignoreHostList = proxyConfiguration.getIgnoreHostList();

Properties props = System.getProperties();
props.put("http.proxyHost", host);
Expand Down Expand Up @@ -189,12 +208,14 @@ public URLConnection setupProxy(ServiceContext context, URL url) throws IOExcept
GeonetContext gc = (GeonetContext) context.getHandlerContext(Geonet.CONTEXT_NAME);
SettingManager sm = gc.getBean(SettingManager.class);

boolean enabled = sm.getValueAsBool(Settings.SYSTEM_PROXY_USE, false);
String host = sm.getValue(Settings.SYSTEM_PROXY_HOST);
String port = sm.getValue(Settings.SYSTEM_PROXY_PORT);
String username = sm.getValue(Settings.SYSTEM_PROXY_USERNAME);
String password = sm.getValue(Settings.SYSTEM_PROXY_PASSWORD);
String ignoreHostList = sm.getValue(Settings.SYSTEM_PROXY_IGNOREHOSTLIST);
proxyConfiguration.refresh(sm);

boolean enabled = proxyConfiguration.isEnabled();
String host = proxyConfiguration.getHost();
String port = proxyConfiguration.getPort();
String username = proxyConfiguration.getUsername();
String password = proxyConfiguration.getPassword();
String ignoreHostList = proxyConfiguration.getIgnoreHostList();

URLConnection conn = null;
if (enabled) {
Expand Down
111 changes: 111 additions & 0 deletions core/src/main/java/org/fao/geonet/lib/ProxyConfiguration.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,111 @@
//=============================================================================
//=== Copyright (C) 2001-2023 Food and Agriculture Organization of the
//=== United Nations (FAO-UN), United Nations World Food Programme (WFP)
//=== and United Nations Environment Programme (UNEP)
//===
//=== This program is free software; you can redistribute it and/or modify
//=== it under the terms of the GNU General Public License as published by
//=== the Free Software Foundation; either version 2 of the License, or (at
//=== your option) any later version.
//===
//=== This program is distributed in the hope that it will be useful, but
//=== WITHOUT ANY WARRANTY; without even the implied warranty of
//=== MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
//=== General Public License for more details.
//===
//=== You should have received a copy of the GNU General Public License
//=== along with this program; if not, write to the Free Software
//=== Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA
//===
//=== Contact: Jeroen Ticheler - FAO - Viale delle Terme di Caracalla 2,
//=== Rome - Italy. email: [email protected]
//==============================================================================

package org.fao.geonet.lib;

import org.apache.commons.lang.StringUtils;
import org.fao.geonet.kernel.setting.SettingManager;
import org.fao.geonet.kernel.setting.Settings;

/**
* Class to abstract the http proxy configuration from Java system properties or GeoNetwork configuration.
*/
public class ProxyConfiguration {
private boolean enabled = false;

private boolean isProxyConfiguredInSystemProperties = false;
private String host;
private String port;
private String username;
private String password;
private String ignoreHostList;

public boolean isEnabled() {
return enabled;
}

public boolean isProxyConfiguredInSystemProperties() {
return isProxyConfiguredInSystemProperties;
}

public String getHost() {
return host;
}

public String getPort() {
return port;
}

public String getUsername() {
return username;
}

public String getPassword() {
return password;
}

public String getIgnoreHostList() {
return ignoreHostList;
}

public ProxyConfiguration(boolean isProxyConfiguredInSystemProperties) {
this.isProxyConfiguredInSystemProperties = isProxyConfiguredInSystemProperties;
if (this.isProxyConfiguredInSystemProperties) {
this.enabled = true;
}
}

public void refresh(SettingManager settingManager) {
this.enabled = this.isProxyConfiguredInSystemProperties ||
settingManager.getValueAsBool(Settings.SYSTEM_PROXY_USE, false);

if (this.enabled) {
if (this.isProxyConfiguredInSystemProperties) {
if (StringUtils.isNotBlank(System.getProperty("https.proxyHost"))) {
this.host = System.getProperty("https.proxyHost");
this.port = System.getProperty("https.proxyPort");
this.username = System.getProperty("https.proxyUser", "");
this.password = System.getProperty("https.proxyPassword", "");
} else {
this.host = System.getProperty("http.proxyHost");
this.port = System.getProperty("http.proxyPort");
this.username = System.getProperty("http.proxyUser", "");
this.password = System.getProperty("http.proxyPassword", "");
}

// Escape characters for regular expression matching
this.ignoreHostList = System.getProperty("http.nonProxyHosts", "")
.replaceAll("\\.", "\\\\.")
.replaceAll("\\*", "\\.\\*");

} else {
this.host = settingManager.getValue(Settings.SYSTEM_PROXY_HOST);
this.port = settingManager.getValue(Settings.SYSTEM_PROXY_PORT);
this.username = settingManager.getValue(Settings.SYSTEM_PROXY_USERNAME);
this.password = settingManager.getValue(Settings.SYSTEM_PROXY_PASSWORD);
this.ignoreHostList = settingManager.getValue(Settings.SYSTEM_PROXY_IGNOREHOSTLIST);

}
}
}
}
29 changes: 24 additions & 5 deletions services/src/main/java/org/fao/geonet/api/site/SiteApi.java
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@
import org.fao.geonet.kernel.setting.SettingManager;
import org.fao.geonet.kernel.setting.Settings;
import org.fao.geonet.lib.Lib;
import org.fao.geonet.lib.ProxyConfiguration;
import org.fao.geonet.repository.*;
import org.fao.geonet.repository.specification.MetadataSpecs;
import org.fao.geonet.resources.Resources;
Expand Down Expand Up @@ -161,12 +162,12 @@ public static void reloadServices(ServiceContext context) throws Exception {
try {
// Load proxy information into Jeeves
ProxyInfo pi = JeevesProxyInfo.getInstance();
boolean useProxy = settingMan.getValueAsBool(Settings.SYSTEM_PROXY_USE, false);
boolean useProxy = Lib.net.getProxyConfiguration().isEnabled();
if (useProxy) {
String proxyHost = settingMan.getValue(Settings.SYSTEM_PROXY_HOST);
String proxyPort = settingMan.getValue(Settings.SYSTEM_PROXY_PORT);
String username = settingMan.getValue(Settings.SYSTEM_PROXY_USERNAME);
String password = settingMan.getValue(Settings.SYSTEM_PROXY_PASSWORD);
String proxyHost = Lib.net.getProxyConfiguration().getHost();
String proxyPort = Lib.net.getProxyConfiguration().getPort();
String username = Lib.net.getProxyConfiguration().getUsername();
String password = Lib.net.getProxyConfiguration().getPassword();
pi.setProxyInfo(proxyHost, Integer.valueOf(proxyPort), username, password);
} else {
pi.setProxyInfo(null, -1, null, null);
Expand Down Expand Up @@ -735,6 +736,24 @@ public StatusValueNotificationLevel[] getNotificationLevel() {
return StatusValueNotificationLevel.values();
}

@io.swagger.v3.oas.annotations.Operation(
summary = "Get proxy configuration details",
description = "Get the proxy configuration.")
@RequestMapping(
path = "/info/proxy",
produces = MediaType.APPLICATION_JSON_VALUE,
method = RequestMethod.GET)
@ResponseStatus(HttpStatus.OK)
@ApiResponses(value = {
@ApiResponse(responseCode = "200", description = "Proxy configuration.")
})
@PreAuthorize("hasAuthority('Administrator')")
@ResponseBody
public ProxyConfiguration getProxyConfiguration(
) {
return Lib.net.getProxyConfiguration();
}

@io.swagger.v3.oas.annotations.Operation(
summary = "Set catalog logo",
description = "Logos are stored in the data directory " +
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -192,6 +192,11 @@
* element name in XML Jeeves request element).
*/
function loadSettings() {
$http.get("../api/site/info/proxy").then(function (response) {
$scope.isProxyConfiguredInSystemProperties =
response.data.proxyConfiguredInSystemProperties;
});

$http.get("../api/site/info/build").then(function (response) {
$scope.systemInfo = response.data;
});
Expand Down Expand Up @@ -269,10 +274,23 @@
var level2name = level1name + "/" + tokens[1];
if (sectionsLevel2.indexOf(level2name) === -1) {
sectionsLevel2.push(level2name);

var sectionChildren;

// Remove the system proxy information if using Java system properties
if (
level2name === "system/proxy" &&
$scope.isProxyConfiguredInSystemProperties
) {
sectionChildren = [];
} else {
sectionChildren = filterBySection($scope.settings, level2name);
}

$scope.sectionsLevel1[level1name].children.push({
name: level2name,
position: $scope.settings[i].position,
children: filterBySection($scope.settings, level2name)
children: sectionChildren
});
}
}
Expand Down
3 changes: 2 additions & 1 deletion web-ui/src/main/resources/catalog/locales/en-admin.json
Original file line number Diff line number Diff line change
Expand Up @@ -1452,6 +1452,7 @@
"ui-footerCustomMenu-help": "List of static page IDs associated with the footer section to display: <ul><li>When a list is provided, the links are displayed in the order provided and only for the pages listed.</li><li>When a list is not provided, all static pages configured for the footer section are displayed, with no guaranteed order.</li>",
"es.url": "ElasticSearch server",
"es.version": "ElasticSearch version",
"es.index": "Index name"
"es.index": "Index name",
"systemPropertiesProxyConfiguration": "Using http proxy settings in system properties."
}

Original file line number Diff line number Diff line change
Expand Up @@ -709,6 +709,12 @@ <h4 class="modal-title" data-translate="">
</div>
</div>
</div>

<div
data-ng-if="section2.name == 'system/proxy' && isProxyConfiguredInSystemProperties"
>
<h5 data-translate="">systemPropertiesProxyConfiguration</h5>
</div>
</fieldset>
<button
type="submit"
Expand Down
Loading

0 comments on commit 5175d11

Please sign in to comment.