Skip to content

Commit

Permalink
If the http proxy is configured in Java system properties, don't use …
Browse files Browse the repository at this point in the history
…the proxy settings configuration
  • Loading branch information
josegar74 committed Sep 7, 2023
1 parent 79865d1 commit e12e114
Show file tree
Hide file tree
Showing 8 changed files with 219 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
108 changes: 108 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,108 @@
//=============================================================================
//=== Copyright (C) 2001-2022 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", "");
}

this.ignoreHostList = System.getProperty("http.nonProxyHosts", "");

} 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 e12e114

Please sign in to comment.