Andrew, Ryan, is this patch going to conflict w/ your changes? I'm +1 on adding it, and will eventually do so unless you guys speak up. :)
- Dan "Danny Angus" <[EMAIL PROTECTED]> writes: > Hi, > Heres a small patch to allow the constructor to setBasicAuthentication from the user >and password information in a URL of this style: > > protocol://user:[EMAIL PROTECTED]:port/path > > Patch assumes that if only one parameter is present that it is the username, and >uses the empty string for the password, this may not be correct. > > (patch shouldn't wrap, but if it does I'll send an attachment) > d. > > Index: XmlRpcClient.java > =================================================================== > RCS file: /home/cvspublic/xml-rpc/src/java/org/apache/xmlrpc/XmlRpcClient.java,v > retrieving revision 1.14 > diff -u -r1.14 XmlRpcClient.java > --- XmlRpcClient.java 21 Nov 2002 01:28:16 -0000 1.14 > +++ XmlRpcClient.java 26 Nov 2002 12:18:20 -0000 > @@ -97,6 +97,15 @@ > public XmlRpcClient(URL url) > { > this.url = url; > + if(url.getUserInfo()!=null){ > + String user = url.getUserInfo(); > + int split = user.indexOf(':'); > + if (split > -1){ > + this.setBasicAuthentication(user.substring(0,split), >user.substring(split+1)); > + }else{ > + this.setBasicAuthentication(user,""); > + } > + } > if (XmlRpc.debug) > { > System.out.println("Created client to url space " + url); >