rsitze 2002/06/04 15:58:32
Modified: java/src/org/apache/axis/transport/http AxisServlet.java
HTTPSender.java
java/src/org/apache/axis/configuration FileProvider.java
DefaultEngineConfigurationFactory.java
java/src/org/apache/axis/client Call.java
java/src/org/apache/axis MessageContext.java AxisEngine.java
java/src/org/apache/axis/server
DefaultAxisServerFactory.java AxisServer.java
Log:
New guides lines for using 'system properties'. Briefly, use
AxisEngine.getGlobalProperties() to get these, so we can
establish a central point of access. In the near future I
expect to update this method to introduce other ways of
setting these properties (as alternatives to the system properties).
The only exception to this (for now) is for the command line utilities,
which are left using System.getProperty().
Revision Changes Path
1.107 +1 -1
xml-axis/java/src/org/apache/axis/transport/http/AxisServlet.java
Index: AxisServlet.java
===================================================================
RCS file:
/home/cvs/xml-axis/java/src/org/apache/axis/transport/http/AxisServlet.java,v
retrieving revision 1.106
retrieving revision 1.107
diff -u -r1.106 -r1.107
--- AxisServlet.java 4 Jun 2002 20:35:42 -0000 1.106
+++ AxisServlet.java 4 Jun 2002 22:58:32 -0000 1.107
@@ -747,7 +747,7 @@
String param,
String dephault)
{
- String value = System.getProperty(param);
+ String value = AxisEngine.getGlobalProperty(param);
if (value == null)
value = getInitParameter(param);
1.65 +15 -14 xml-axis/java/src/org/apache/axis/transport/http/HTTPSender.java
Index: HTTPSender.java
===================================================================
RCS file:
/home/cvs/xml-axis/java/src/org/apache/axis/transport/http/HTTPSender.java,v
retrieving revision 1.64
retrieving revision 1.65
diff -u -r1.64 -r1.65
--- HTTPSender.java 3 Jun 2002 12:24:55 -0000 1.64
+++ HTTPSender.java 4 Jun 2002 22:58:32 -0000 1.65
@@ -54,6 +54,7 @@
*/
package org.apache.axis.transport.http;
+import org.apache.axis.AxisEngine;
import org.apache.axis.AxisFault;
import org.apache.axis.Message;
import org.apache.axis.MessageContext;
@@ -187,15 +188,15 @@
}
// Get https.proxyXXX settings
- String tunnelHost = System.getProperty("https.proxyHost");
- String tunnelPortStr = System.getProperty("https.proxyPort");
+ String tunnelHost = AxisEngine.getGlobalProperty("https.proxyHost");
+ String tunnelPortStr = AxisEngine.getGlobalProperty("https.proxyPort");
// Use http.proxyXXX settings if https.proxyXXX is not set
if (tunnelHost == null) {
- tunnelHost = System.getProperty("http.proxyHost");
+ tunnelHost = AxisEngine.getGlobalProperty("http.proxyHost");
}
if (tunnelPortStr == null) {
- tunnelPortStr = System.getProperty("http.proxyPort");
+ tunnelPortStr = AxisEngine.getGlobalProperty("http.proxyPort");
}
try {
@@ -214,7 +215,7 @@
Object factory = null;
// Hook in a different SSL socket factory
- String socketFactoryClass = System.getProperty("axis.socketFactory");
+ String socketFactoryClass =
AxisEngine.getGlobalProperty("axis.socketFactory");
if (socketFactoryClass != null) {
try {
Class c1 = Class.forName(socketFactoryClass);
@@ -262,14 +263,14 @@
PrintWriter out = new PrintWriter(
new BufferedWriter(
new OutputStreamWriter(tunnelOutputStream)));
- String tunnelUser = System.getProperty("https.proxyUser");
- String tunnelPassword = System.getProperty("https.proxyPassword");
+ String tunnelUser = AxisEngine.getGlobalProperty("https.proxyUser");
+ String tunnelPassword =
AxisEngine.getGlobalProperty("https.proxyPassword");
if (tunnelUser == null) {
- tunnelUser = System.getProperty("http.proxyUser");
+ tunnelUser = AxisEngine.getGlobalProperty("http.proxyUser");
}
if (tunnelPassword == null) {
- tunnelPassword = System.getProperty("http.proxyPassword");
+ tunnelPassword =
AxisEngine.getGlobalProperty("http.proxyPassword");
}
// More secure version... engage later?
@@ -392,12 +393,12 @@
String host, int port, StringBuffer otherHeaders, BooleanHolder
useFullURL)
throws IOException {
Socket sock = null;
- String proxyHost = System.getProperty("http.proxyHost");
- String proxyPort = System.getProperty("http.proxyPort");
- String nonProxyHosts = System.getProperty("http.nonProxyHosts");
+ String proxyHost = AxisEngine.getGlobalProperty("http.proxyHost");
+ String proxyPort = AxisEngine.getGlobalProperty("http.proxyPort");
+ String nonProxyHosts = AxisEngine.getGlobalProperty("http.nonProxyHosts");
boolean hostInNonProxyList = isHostInNonProxyList(host, nonProxyHosts);
- String proxyUsername = System.getProperty("http.proxyUser");
- String proxyPassword = System.getProperty("http.proxyPassword");
+ String proxyUsername = AxisEngine.getGlobalProperty("http.proxyUser");
+ String proxyPassword = AxisEngine.getGlobalProperty("http.proxyPassword");
if (proxyUsername != null) {
StringBuffer tmpBuf = new StringBuffer();
1.28 +0 -2
xml-axis/java/src/org/apache/axis/configuration/FileProvider.java
Index: FileProvider.java
===================================================================
RCS file:
/home/cvs/xml-axis/java/src/org/apache/axis/configuration/FileProvider.java,v
retrieving revision 1.27
retrieving revision 1.28
diff -u -r1.27 -r1.28
--- FileProvider.java 21 May 2002 14:50:50 -0000 1.27
+++ FileProvider.java 4 Jun 2002 22:58:32 -0000 1.28
@@ -95,8 +95,6 @@
protected static Log log =
LogFactory.getLog(FileProvider.class.getName());
- protected String sep = System.getProperty("file.separator");
-
protected WSDDDeployment deployment = null;
private static final String CURRENT_DIR = ".";
1.11 +4 -3
xml-axis/java/src/org/apache/axis/configuration/DefaultEngineConfigurationFactory.java
Index: DefaultEngineConfigurationFactory.java
===================================================================
RCS file:
/home/cvs/xml-axis/java/src/org/apache/axis/configuration/DefaultEngineConfigurationFactory.java,v
retrieving revision 1.10
retrieving revision 1.11
diff -u -r1.10 -r1.11
--- DefaultEngineConfigurationFactory.java 30 May 2002 23:46:00 -0000 1.10
+++ DefaultEngineConfigurationFactory.java 4 Jun 2002 22:58:32 -0000 1.11
@@ -55,6 +55,7 @@
package org.apache.axis.configuration;
+import org.apache.axis.AxisEngine;
import org.apache.axis.EngineConfigurationFactory;
import org.apache.axis.EngineConfiguration;
import org.apache.axis.utils.JavaUtils;
@@ -95,7 +96,7 @@
*/
public DefaultEngineConfigurationFactory() {
String fClassName =
- System.getProperty(EngineConfigurationFactory.SYSTEM_PROPERTY_NAME);
+
AxisEngine.getGlobalProperty(EngineConfigurationFactory.SYSTEM_PROPERTY_NAME);
if (fClassName != null) {
try {
@@ -107,12 +108,12 @@
}
}
- clientConfigFile = System.getProperty("axis.ClientConfigFile");
+ clientConfigFile = AxisEngine.getGlobalProperty("axis.ClientConfigFile");
if (clientConfigFile == null) {
clientConfigFile = CLIENT_CONFIG_FILE;
}
- serverConfigFile = System.getProperty("axis.ServerConfigFile");
+ serverConfigFile = AxisEngine.getGlobalProperty("axis.ServerConfigFile");
if (serverConfigFile == null) {
serverConfigFile = SERVER_CONFIG_FILE;
}
1.129 +2 -1 xml-axis/java/src/org/apache/axis/client/Call.java
Index: Call.java
===================================================================
RCS file: /home/cvs/xml-axis/java/src/org/apache/axis/client/Call.java,v
retrieving revision 1.128
retrieving revision 1.129
diff -u -r1.128 -r1.129
--- Call.java 4 Jun 2002 20:34:02 -0000 1.128
+++ Call.java 4 Jun 2002 22:58:32 -0000 1.129
@@ -55,6 +55,7 @@
package org.apache.axis.client ;
+import org.apache.axis.AxisEngine;
import org.apache.axis.AxisFault;
import org.apache.axis.Constants;
import org.apache.axis.Handler;
@@ -1293,7 +1294,7 @@
public static synchronized void addTransportPackage(String packageName) {
if (transportPackages == null) {
transportPackages = new ArrayList();
- String currentPackages = System.getProperty(TRANSPORT_PROPERTY);
+ String currentPackages =
AxisEngine.getGlobalProperty(TRANSPORT_PROPERTY);
if (currentPackages != null) {
StringTokenizer tok = new StringTokenizer(currentPackages,
"|");
1.97 +1 -1 xml-axis/java/src/org/apache/axis/MessageContext.java
Index: MessageContext.java
===================================================================
RCS file: /home/cvs/xml-axis/java/src/org/apache/axis/MessageContext.java,v
retrieving revision 1.96
retrieving revision 1.97
diff -u -r1.96 -r1.97
--- MessageContext.java 4 Jun 2002 20:34:02 -0000 1.96
+++ MessageContext.java 4 Jun 2002 22:58:32 -0000 1.97
@@ -259,7 +259,7 @@
protected static String systemTempDir= null;
static {
try {
- systemTempDir=System.getProperty(AxisEngine.ENV_ATTACHMENT_DIR);
+
systemTempDir=AxisEngine.getGlobalProperty(AxisEngine.ENV_ATTACHMENT_DIR);
} catch(Throwable t) {
systemTempDir= null;
}
1.78 +9 -1 xml-axis/java/src/org/apache/axis/AxisEngine.java
Index: AxisEngine.java
===================================================================
RCS file: /home/cvs/xml-axis/java/src/org/apache/axis/AxisEngine.java,v
retrieving revision 1.77
retrieving revision 1.78
diff -u -r1.77 -r1.78
--- AxisEngine.java 9 May 2002 18:13:01 -0000 1.77
+++ AxisEngine.java 4 Jun 2002 22:58:32 -0000 1.78
@@ -184,7 +184,7 @@
/*Set the default attachment implementation */
setOptionDefault(PROP_ATTACHMENT_IMPLEMENTATION,
- System.getProperty("axis." +
PROP_ATTACHMENT_IMPLEMENTATION ));
+ AxisEngine.getGlobalProperty("axis." +
PROP_ATTACHMENT_IMPLEMENTATION ));
setOptionDefault(PROP_ATTACHMENT_IMPLEMENTATION, DEFAULT_ATTACHMENT_IMPL);
@@ -397,5 +397,13 @@
public ClassCache getClassCache() {
return classCache;
+ }
+
+ /**
+ * Central access point for AXIS to obtain "global" configuration properties.
+ * To be extended in the future... or replaced with non-global properties.
+ */
+ public static String getGlobalProperty(String property) {
+ return System.getProperty(property);
}
};
1.8 +1 -1
xml-axis/java/src/org/apache/axis/server/DefaultAxisServerFactory.java
Index: DefaultAxisServerFactory.java
===================================================================
RCS file:
/home/cvs/xml-axis/java/src/org/apache/axis/server/DefaultAxisServerFactory.java,v
retrieving revision 1.7
retrieving revision 1.8
diff -u -r1.7 -r1.8
--- DefaultAxisServerFactory.java 9 May 2002 18:35:11 -0000 1.7
+++ DefaultAxisServerFactory.java 4 Jun 2002 22:58:32 -0000 1.8
@@ -175,7 +175,7 @@
if (config == null) {
// A default engine configuration class may be set in a system
// property. If so, try creating an engine configuration.
- String configClass = System.getProperty(AxisEngine.PROP_CONFIG_CLASS);
+ String configClass =
AxisEngine.getGlobalProperty(AxisEngine.PROP_CONFIG_CLASS);
if (configClass != null) {
try {
// Got one - so try to make it (which means it had better have
1.67 +1 -1 xml-axis/java/src/org/apache/axis/server/AxisServer.java
Index: AxisServer.java
===================================================================
RCS file: /home/cvs/xml-axis/java/src/org/apache/axis/server/AxisServer.java,v
retrieving revision 1.66
retrieving revision 1.67
diff -u -r1.66 -r1.67
--- AxisServer.java 9 May 2002 18:33:15 -0000 1.66
+++ AxisServer.java 4 Jun 2002 22:58:32 -0000 1.67
@@ -84,7 +84,7 @@
public static AxisServer getServer(Map environment) throws AxisFault
{
if (factory == null) {
- String factoryClassName = System.getProperty("axis.ServerFactory");
+ String factoryClassName = getGlobalProperty("axis.ServerFactory");
if (factoryClassName != null) {
try {
Class factoryClass = Class.forName(factoryClassName);