axis-java-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Glen Daniels <gdani...@macromedia.com>
Subject RE: cvs commit: xml-axis/java/src/org/apache/axis/client Call.jav a
Date Sun, 04 Aug 2002 03:30:39 GMT

The point of the write-back (and the entire messing with "transport packages" in the first
place) is to make sure that custom URL schemes like "tcp:" are supported by the java.net.URL
class.  The way this works is via the system property.  See the javadoc for URL (the constructor
section).

Does the TCP transport test still work after this change?  If so, I'd be curious as to how.

--Glen

> -----Original Message-----
> From: rsitze@apache.org [mailto:rsitze@apache.org]
> Sent: Saturday, August 03, 2002 11:23 PM
> To: xml-axis-cvs@apache.org
> Subject: cvs commit: xml-axis/java/src/org/apache/axis/client 
> Call.java
> 
> 
> rsitze      2002/08/03 20:23:10
> 
>   Modified:    java/src/org/apache/axis/client Call.java
>   Log:
>   Removed logic to push tranport properties back out to a 
> system property.  This
>   is very unfriendly in a multi-app host.  After initial 
> setup TRANSPORT_PROPERTY
>   is never used again, so it the write-back is ignored by the 
> local Class anyway.
>   
>   Revision  Changes    Path
>   1.166     +12 -9     
> 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.165
>   retrieving revision 1.166
>   diff -u -r1.165 -r1.166
>   --- Call.java	2 Aug 2002 17:27:41 -0000	1.165
>   +++ Call.java	4 Aug 2002 03:23:10 -0000	1.166
>   @@ -1458,15 +1458,18 @@
>                return;
>    
>            transportPackages.add(packageName);
>   -
>   -        StringBuffer currentPackages = new StringBuffer();
>   -        for (Iterator i = transportPackages.iterator(); 
> i.hasNext();) {
>   -            String thisPackage = (String) i.next();
>   -            currentPackages.append(thisPackage);
>   -            currentPackages.append('|');
>   -        }
>   -
>   -        AxisProperties.setProperty(TRANSPORT_PROPERTY, 
> currentPackages.toString());
>   +        
>   +        // the following isn't safe/secure, and it's not 
> clear that it's necessary.
>   +        // particularly in light of the AxisProperties model.
>   +//
>   +//        StringBuffer currentPackages = new StringBuffer();
>   +//        for (Iterator i = transportPackages.iterator(); 
> i.hasNext();) {
>   +//            String thisPackage = (String) i.next();
>   +//            currentPackages.append(thisPackage);
>   +//            currentPackages.append('|');
>   +//        }
>   +//
>   +//        System.setProperty(TRANSPORT_PROPERTY, 
> currentPackages.toString());
>        }
>    
>        /**
>   
>   
>   
> 

Mime
View raw message