You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cxf.apache.org by Daniel Kulp <dk...@apache.org> on 2012/03/23 12:38:09 UTC
Re: svn commit: r1304181 - /cxf/trunk/api/src/main/java/org/apache/cxf/transport/TransportURIResolver.java
Willem,
Not objecting to this, but wondering why it's necessary? Such conduits
really should be configured based on URL, not some strange internal name or
similar. Seriously, a conduit thing like:
<http:conduit name="http.*"> ...
would likely accomplish the exact same thing.
Dan
On Friday, March 23, 2012 04:01:19 AM ningjiang@apache.org wrote:
> Author: ningjiang
> Date: Fri Mar 23 04:01:19 2012
> New Revision: 1304181
>
> URL: http://svn.apache.org/viewvc?rev=1304181&view=rev
> Log:
> CXF-4195 fixed the configuration issue of http conduit for WsdlUrl
>
> Modified:
>
> cxf/trunk/api/src/main/java/org/apache/cxf/transport/TransportURIResolver
> .java
>
> Modified:
> cxf/trunk/api/src/main/java/org/apache/cxf/transport/TransportURIResolver
> .java URL:
> http://svn.apache.org/viewvc/cxf/trunk/api/src/main/java/org/apache/cxf/t
> ransport/TransportURIResolver.java?rev=1304181&r1=1304180&r2=1304181&view=
> diff
> =========================================================================
> ===== ---
> cxf/trunk/api/src/main/java/org/apache/cxf/transport/TransportURIResolver
> .java (original) +++
> cxf/trunk/api/src/main/java/org/apache/cxf/transport/TransportURIResolver
> .java Fri Mar 23 04:01:19 2012 @@ -26,6 +26,8 @@ import
> java.net.URISyntaxException;
> import java.util.HashSet;
> import java.util.Set;
>
> +import javax.xml.namespace.QName;
> +
> import org.xml.sax.InputSource;
>
> import org.apache.cxf.Bus;
> @@ -101,6 +103,8 @@ public class TransportURIResolver extend
> }
> if (ci != null) {
> EndpointInfo info = new EndpointInfo();
> + // set the endpointInfo name which could be used for
> configuration + info.setName(new
> QName("http://cxf.apache.org", "TransportURIResolver"));
> info.setAddress(base.toString());
> final Conduit c = ci.getConduit(info);
> Message message = new MessageImpl();
--
Daniel Kulp
dkulp@apache.org - http://dankulp.com/blog
Talend Community Coder - http://coders.talend.com
Re: svn commit: r1304181 - /cxf/trunk/api/src/main/java/org/apache/cxf/transport/TransportURIResolver.java
Posted by Willem Jiang <wi...@gmail.com>.
Hi Dan,
I did some test by using the <http:conduit name="*.http-conduit">.
It doesn't work and the created HttpConduit 's getBeanName return null.
I didn't see the <http:conduit name="http.*"> can do the trick.
On Fri Mar 23 19:38:09 2012, Daniel Kulp wrote:
>
> Willem,
>
> Not objecting to this, but wondering why it's necessary? Such conduits
> really should be configured based on URL, not some strange internal name or
> similar. Seriously, a conduit thing like:
>
> <http:conduit name="http.*"> ...
>
> would likely accomplish the exact same thing.
>
> Dan
>
>
> On Friday, March 23, 2012 04:01:19 AM ningjiang@apache.org wrote:
>> Author: ningjiang
>> Date: Fri Mar 23 04:01:19 2012
>> New Revision: 1304181
>>
>> URL: http://svn.apache.org/viewvc?rev=1304181&view=rev
>> Log:
>> CXF-4195 fixed the configuration issue of http conduit for WsdlUrl
>>
>> Modified:
>>
>> cxf/trunk/api/src/main/java/org/apache/cxf/transport/TransportURIResolver
>> .java
>>
>> Modified:
>> cxf/trunk/api/src/main/java/org/apache/cxf/transport/TransportURIResolver
>> .java URL:
>> http://svn.apache.org/viewvc/cxf/trunk/api/src/main/java/org/apache/cxf/t
>> ransport/TransportURIResolver.java?rev=1304181&r1=1304180&r2=1304181&view=
>> diff
>> =========================================================================
>> ===== ---
>> cxf/trunk/api/src/main/java/org/apache/cxf/transport/TransportURIResolver
>> .java (original) +++
>> cxf/trunk/api/src/main/java/org/apache/cxf/transport/TransportURIResolver
>> .java Fri Mar 23 04:01:19 2012 @@ -26,6 +26,8 @@ import
>> java.net.URISyntaxException;
>> import java.util.HashSet;
>> import java.util.Set;
>>
>> +import javax.xml.namespace.QName;
>> +
>> import org.xml.sax.InputSource;
>>
>> import org.apache.cxf.Bus;
>> @@ -101,6 +103,8 @@ public class TransportURIResolver extend
>> }
>> if (ci != null) {
>> EndpointInfo info = new EndpointInfo();
>> + // set the endpointInfo name which could be used for
>> configuration + info.setName(new
>> QName("http://cxf.apache.org", "TransportURIResolver"));
>> info.setAddress(base.toString());
>> final Conduit c = ci.getConduit(info);
>> Message message = new MessageImpl();
--
Willem
----------------------------------
FuseSource
Web: http://www.fusesource.com
Blog: http://willemjiang.blogspot.com (English)
http://jnn.javaeye.com (Chinese)
Twitter: willemjiang
Weibo: willemjiang