You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cxf.apache.org by Dan Diephouse <da...@envoisolutions.com> on 2007/04/23 15:31:29 UTC
Re: svn commit: r531341 - in /incubator/cxf/trunk: distribution/src/main/assembly/ distribution/src/main/release/lib/ distribution/src/main/release/samples/spring_http/ distribution/src/main/release/samples/spring_http/src/demo/ distribution/src/main
I'll admit I don't have a clue about the logic in updateDests, but it is
broken for certain cases (like the spring_http) demo. Also, the more
important thing to me here is to allow the default bus to be set so you can
access the bus outside of the servlet (i.e. when using
JaxWsServerFactoryBean). We'll figure out something on IRC though as I think
this involves some discussions about the TCK and that can't happen here :-\
- Dan
On 4/23/07, Daniel Kulp <dk...@apache.org> wrote:
>
>
> James,
>
> Yea, this commit broke a BUNCH of things for me too. When I get in in
> the morning, this will be my #1 priority to get straightened out. I'll
> test the servlet stuff at that point.
>
> Dan
>
>
>
> On Monday 23 April 2007 03:55, James Mao wrote:
> > Hi Dan,
> >
> > After this fix, i got:
> >
> > client-servlet:
> > [java] http://localhost:8080/helloworld/services/hello_world?wsdl
> > [java] Invoking sayHi...
> > [java] Exception in thread "main"
> > javax.xml.ws.WebServiceException: org.apa
> > che.cxf.interceptor.Fault: Could not send Message.
> > [java] at
> > org.apache.cxf.jaxws.JaxWsClientProxy.invoke(JaxWsClientProxy
> > .java:134)
> > [java] at $Proxy34.sayHi(Unknown Source)
> > [java] at demo.hw.client.Client.main(Client.java:60)
> > [java] Caused by: org.apache.cxf.interceptor.Fault: Could not
> > send Message.
> >
> > [java] at
> > org.apache.cxf.interceptor.MessageSenderInterceptor.handleMes
> > sage(MessageSenderInterceptor.java:47)
> > [java] at
> > org.apache.cxf.phase.PhaseInterceptorChain.doIntercept(PhaseI
> > nterceptorChain.java:148)
> > [java] at
> > org.apache.cxf.endpoint.ClientImpl.invoke(ClientImpl.java:215
> > )
> > [java] at
> > org.apache.cxf.frontend.ClientProxy.invokeSync(ClientProxy.ja
> > va:73)
> > [java] at
> > org.apache.cxf.jaxws.JaxWsClientProxy.invoke(JaxWsClientProxy
> > .java:122)
> > [java] ... 2 more
> > [java] Caused by: java.net.MalformedURLException: no protocol:
> > /hello_world
> >
> > [java] at java.net.URL.<init>(URL.java:567)
> > [java] at java.net.URL.<init>(URL.java:464)
> > [java] at java.net.URL.<init>(URL.java:413)
> > [java] at
> > org.apache.cxf.transport.http.HTTPConduit.setupURL(HTTPCondui
> > t.java:611)
> > [java] at
> > org.apache.cxf.transport.http.HTTPConduit.prepare(HTTPConduit
> > .java:440)
> > [java] at
> > org.apache.cxf.interceptor.MessageSenderInterceptor.handleMes
> > sage(MessageSenderInterceptor.java:45)
> > [java] ... 6 more
> > [java] Java Result: 1
> >
> > The previous problem is just the URL problems, to fix the problem,
> > what we need is modify the cxf-servlet.xml
> > change the jaxws:address from "/services/helloworld" to "/helloworld"
> >
> > But now seems the whole demo is broken.
> >
> > James.
> >
> > > Author: dandiep
> > > Date: Sun Apr 22 22:56:43 2007
> > > New Revision: 531341
> > >
> > > URL: http://svn.apache.org/viewvc?view=rev&rev=531341
> > > Log:
> > > o Change the servlet transport to look up services only by their
> > > path. This means we don't have to put the context name in the
> > > address now, which means the samples will work correctly.
> > > o Allow CXFServlet to set the defaultbus. If you don't do this, it
> > > breaks some Spring use cases. I also changed it so we ALWAYS
> > > create a Bus (unless the bus id was specified). This should prevent
> > > conflicts across servlet instances, which seems to be what the
> > > previous code was trying to prevent.
> > > o Add missing build.xml for spring_http sample and rework its
> > > directory structure to be like the other samples.
> > >
> > > Added:
> > >
> > > incubator/cxf/trunk/distribution/src/main/release/samples/spring_htt
> > >p/build.xml (with props)
> > > incubator/cxf/trunk/distribution/src/main/release/samples/spring_htt
> > >p/src/demo/ - copied from r531063,
> > > incubator/cxf/trunk/distribution/src/main/release/samples/spring_htt
> > >p/src/main/java/demo/
> > > incubator/cxf/trunk/distribution/src/main/release/samples/spring_htt
> > >p/webapp/ - copied from r531063,
> > > incubator/cxf/trunk/distribution/src/main/release/samples/spring_htt
> > >p/src/webapp/ Removed:
> > >
> > > incubator/cxf/trunk/distribution/src/main/release/samples/spring_htt
> > >p/src/main/java/demo/
> > > incubator/cxf/trunk/distribution/src/main/release/samples/spring_htt
> > >p/src/webapp/ Modified:
> > > incubator/cxf/trunk/distribution/src/main/assembly/bin.xml
> > > incubator/cxf/trunk/distribution/src/main/release/lib/WHICH_JARS
> > >
> > > incubator/cxf/trunk/distribution/src/main/release/samples/spring_htt
> > >p/src/demo/spring/servlet/Server.java
> > > incubator/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/
> > >transport/servlet/CXFServlet.java
> > > incubator/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/
> > >transport/servlet/ServletController.java
> > > incubator/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/
> > >transport/servlet/ServletDestination.java
> > > incubator/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/
> > >transport/servlet/ServletTransportFactory.java
> > > incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/se
> > >rvlet/AbstractServletTest.java
> > > incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/se
> > >rvlet/CXFServletTest.java
> > > incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/se
> > >rvlet/ExternalServicesServletTest.java
> > >
> > > Modified: incubator/cxf/trunk/distribution/src/main/assembly/bin.xml
> > > URL:
> > > http://svn.apache.org/viewvc/incubator/cxf/trunk/distribution/src/ma
> > >in/assembly/bin.xml?view=diff&rev=531341&r1=531340&r2=531341
> > > ====================================================================
> > >========== ---
> > > incubator/cxf/trunk/distribution/src/main/assembly/bin.xml
> > > (original) +++
> > > incubator/cxf/trunk/distribution/src/main/assembly/bin.xml Sun Apr
> > > 22 22:56:43 2007 @@ -35,6 +35,7 @@
> > > <include>LICENSE</include>
> > > <include>licenses/*.*</include>
> > > <include>modules/*</include>
> > > + <include>lib/*</include>
> > > <include>README</include>
> > > </includes>
> > > </fileSet>
> > >
> > > Modified:
> > > incubator/cxf/trunk/distribution/src/main/release/lib/WHICH_JARS
> > > URL:
> > > http://svn.apache.org/viewvc/incubator/cxf/trunk/distribution/src/ma
> > >in/release/lib/WHICH_JARS?view=diff&rev=531341&r1=531340&r2=531341
> > > ====================================================================
> > >========== ---
> > > incubator/cxf/trunk/distribution/src/main/release/lib/WHICH_JARS
> > > (original) +++
> > > incubator/cxf/trunk/distribution/src/main/release/lib/WHICH_JARS Sun
> > > Apr 22 22:56:43 2007 @@ -47,6 +47,11 @@
> > > - stax-utils.jar
> > >
> > > For WS-Security support:
> > > +- bcprov-jdk14.jar
> > > - wss4j.jar
> > > - xalan.jar
> > > -- xmlsec.jar
> > > \ No newline at end of file
> > > +- xmlsec.jar
> > > +
> > > +For HTTP Binding support:
> > > +- jra.jar
> > > +- jettison.jar (Needed for JSON services only)
> > >
> > > Added:
> > > incubator/cxf/trunk/distribution/src/main/release/samples/spring_htt
> > >p/build.xml URL:
> > > http://svn.apache.org/viewvc/incubator/cxf/trunk/distribution/src/ma
> > >in/release/samples/spring_http/build.xml?view=auto&rev=531341
> > > ====================================================================
> > >========== ---
> > > incubator/cxf/trunk/distribution/src/main/release/samples/spring_htt
> > >p/build.xml (added) +++
> > > incubator/cxf/trunk/distribution/src/main/release/samples/spring_htt
> > >p/build.xml Sun Apr 22 22:56:43 2007 @@ -0,0 +1,38 @@
> > > +<?xml version="1.0" encoding="UTF-8"?>
> > > +<!--
> > > + Licensed to the Apache Software Foundation (ASF) under one
> > > + or more contributor license agreements. See the NOTICE file
> > > + distributed with this work for additional information
> > > + regarding copyright ownership. The ASF licenses this file
> > > + to you under the Apache License, Version 2.0 (the
> > > + "License"); you may not use this file except in compliance
> > > + with the License. You may obtain a copy of the License at
> > > +
> > > + http://www.apache.org/licenses/LICENSE-2.0
> > > +
> > > + Unless required by applicable law or agreed to in writing,
> > > + software distributed under the License is distributed on an
> > > + "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
> > > + KIND, either express or implied. See the License for the
> > > + specific language governing permissions and limitations
> > > + under the License.
> > > +-->
> > > +<project name="Spring HTTP Servlet demo" default="build"
> > > basedir="."> +
> > > +
> > > + <import file="../common_build.xml"/>
> > > +
> > > + <target name="client" description="run demo client"
> > > depends="build"> + <property name="param" value=""/>
> > > + <cxfrun classname="demo.spring.client.Client" />
> > > + </target>
> > > +
> > > + <target name="server" description="run demo server"
> > > depends="build"> + <cxfrun
> > > classname="demo.spring.servlet.Server"/>
> > > + </target>
> > > +
> > > + <property name="cxf.war.file.name" value="spring_http"/>
> > > + <target name="war" depends="build">
> > > + <cxfwar filename="${cxf.war.file.name}.war"/>
> > > + </target>
> > > +</project>
> > >
> > > Propchange:
> > > incubator/cxf/trunk/distribution/src/main/release/samples/spring_htt
> > >p/build.xml
> > > --------------------------------------------------------------------
> > >---------- svn:eol-style = native
> > >
> > > Propchange:
> > > incubator/cxf/trunk/distribution/src/main/release/samples/spring_htt
> > >p/build.xml
> > > --------------------------------------------------------------------
> > >---------- svn:executable = *
> > >
> > > Propchange:
> > > incubator/cxf/trunk/distribution/src/main/release/samples/spring_htt
> > >p/build.xml
> > > --------------------------------------------------------------------
> > >---------- svn:keywords = Rev Date
> > >
> > > Propchange:
> > > incubator/cxf/trunk/distribution/src/main/release/samples/spring_htt
> > >p/build.xml
> > > --------------------------------------------------------------------
> > >---------- svn:mime-type = text/xml
> > >
> > > Modified:
> > > incubator/cxf/trunk/distribution/src/main/release/samples/spring_htt
> > >p/src/demo/spring/servlet/Server.java URL:
> > > http://svn.apache.org/viewvc/incubator/cxf/trunk/distribution/src/ma
> > >in/release/samples/spring_http/src/demo/spring/servlet/Server.java?vi
> > >ew=diff&rev=531341&r1=531063&r2=531341
> > > ====================================================================
> > >========== ---
> > > incubator/cxf/trunk/distribution/src/main/release/samples/spring_htt
> > >p/src/demo/spring/servlet/Server.java (original) +++
> > > incubator/cxf/trunk/distribution/src/main/release/samples/spring_htt
> > >p/src/demo/spring/servlet/Server.java Sun Apr 22 22:56:43 2007 @@
> > > -46,7 +46,7 @@
> > > WebAppContext webappcontext = new WebAppContext();
> > > webappcontext.setContextPath("/");
> > >
> > > - webappcontext.setWar("src/webapp");
> > > + webappcontext.setWar("webapp");
> > >
> > > HandlerCollection handlers = new HandlerCollection();
> > > handlers.setHandlers(new Handler[] {webappcontext, new
> > > DefaultHandler()});
> > >
> > > Modified:
> > > incubator/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/
> > >transport/servlet/CXFServlet.java URL:
> > > http://svn.apache.org/viewvc/incubator/cxf/trunk/rt/transports/http/
> > >src/main/java/org/apache/cxf/transport/servlet/CXFServlet.java?view=d
> > >iff&rev=531341&r1=531340&r2=531341
> > > ====================================================================
> > >========== ---
> > > incubator/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/
> > >transport/servlet/CXFServlet.java (original) +++
> > > incubator/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/
> > >transport/servlet/CXFServlet.java Sun Apr 22 22:56:43 2007 @@ -78,9
> > > +78,6 @@
> > > public void init(ServletConfig servletConfig) throws
> > > ServletException { super.init(servletConfig);
> > >
> > > - BusFactory.setDefaultBus(null);
> > > - BusFactory.setThreadDefaultBus(null);
> > > -
> > > String busid = servletConfig.getInitParameter("bus.id");
> > > if (null != busid) {
> > > WeakReference<Bus> ref = BUS_MAP.get(busid);
> > > @@ -101,13 +98,11 @@
> > > if (null != busid) {
> > > BUS_MAP.put(busid, new WeakReference<Bus>(bus));
> > > }
> > > - BusFactory.setDefaultBus(null);
> > > - BusFactory.setThreadDefaultBus(null);
> > > }
> > >
> > > private void loadBusNoConfig(ServletConfig servletConfig)
> > > throws ServletException { if (bus == null) {
> > > - bus = BusFactory.getDefaultBus();
> > > + bus = BusFactory.newInstance().createBus();
> > > }
> > > ResourceManager resourceManager =
> > > bus.getExtension(ResourceManager.class);
> > > resourceManager.addResourceResolver(new
> > > ServletContextResourceResolver(
> > >
> > > Modified:
> > > incubator/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/
> > >transport/servlet/ServletController.java URL:
> > > http://svn.apache.org/viewvc/incubator/cxf/trunk/rt/transports/http/
> > >src/main/java/org/apache/cxf/transport/servlet/ServletController.java
> > >?view=diff&rev=531341&r1=531340&r2=531341
> > > ====================================================================
> > >========== ---
> > > incubator/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/
> > >transport/servlet/ServletController.java (original) +++
> > > incubator/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/
> > >transport/servlet/ServletController.java Sun Apr 22 22:56:43 2007 @@
> > > -22,7 +22,6 @@
> > > import java.io.InputStream;
> > > import java.io.OutputStream;
> > > import java.util.Collection;
> > > -import java.util.Set;
> > > import java.util.logging.Level;
> > > import java.util.logging.Logger;
> > >
> > > @@ -46,32 +45,11 @@
> > >
> > > private ServletTransportFactory transport;
> > > private CXFServlet cxfServlet;
> > > - private String lastBase = "";
> > >
> > > public ServletController(ServletTransportFactory df, CXFServlet
> > > servlet) { this.transport = df;
> > > this.cxfServlet = servlet;
> > > }
> > > -
> > > - private synchronized void updateDests(HttpServletRequest
> > > request) { - String pathInfo = request.getPathInfo() == null
> > > ? "" : request.getPathInfo(); - String base =
> > > request.getRequestURL().toString();
> > > - base = base.substring(0, base.length() -
> > > pathInfo.length()); -
> > > - if (base.equals(lastBase)) {
> > > - return;
> > > - }
> > > - Set<String> paths = transport.getDestinationsPaths();
> > > - for (String path : paths) {
> > > - ServletDestination d2 =
> > > transport.getDestinationForPath(path); - String ad =
> > > d2.getEndpointInfo().getAddress(); - if (ad.equals(path)
> > > - || ad.equals(lastBase + path)) {
> > > - d2.getEndpointInfo().setAddress(base + path);
> > > - }
> > > - }
> > > - lastBase = base;
> > > - }
> > >
> > > public void invoke(HttpServletRequest request,
> > > HttpServletResponse res) throws ServletException { try {
> > > @@ -82,7 +60,6 @@
> > > ServletDestination d =
> > > (ServletDestination)transport.getDestination(ei);
> > >
> > > if (d.getMessageObserver() == null) {
> > > - updateDests(request);
> > > if (request.getRequestURI().endsWith("services")
> > >
> > > || request.getRequestURI().endsWith("services/"
> > > ||) StringUtils.isEmpty(request.getPathInfo())
> > >
> > > @@ -93,7 +70,6 @@
> > > generateNotFound(request, res);
> > > }
> > > } else {
> > > - updateDests(request);
> > > ei = d.getEndpointInfo();
> > > Bus bus = cxfServlet.getBus();
> > > if (null != request.getQueryString()
> > > @@ -133,14 +109,20 @@
> > > response.setContentType("text/html");
> > > response.getWriter().write("<html><body>");
> > >
> > > - String reqPerfix = request.getRequestURL().toString();
> > > - String pathInfo = request.getPathInfo() == null ? "" :
> > > request.getPathInfo(); - reqPerfix = reqPerfix.substring(0,
> > > reqPerfix.length() - pathInfo.length()); + String reqPerfix =
> > > getBaseURL(request);
> > >
> > > if (destinations.size() > 0) {
> > > for (ServletDestination sd : destinations) {
> > > if (null != sd.getEndpointInfo().getName()) {
> > > - String address =
> > > sd.getEndpointInfo().getAddress(); +
> > > StringBuilder sb = new StringBuilder(); +
> > > sb.append(reqPerfix);
> > > + String path = sd.getAddressPath();
> > > + if (reqPerfix.endsWith("/")) {
> > > + path = path.substring(1);
> > > + }
> > > + sb.append(path);
> > > +
> > > + String address = sb.toString();
> > > response.getWriter().write("<p> <a href=\"" +
> > > address + "?wsdl\">");
> > > response.getWriter().write(sd.getEndpointInfo().getName() + "</a>
> > > </p>"); }
> > > @@ -149,6 +131,13 @@
> > > response.getWriter().write("No service was found.");
> > > }
> > > response.getWriter().write("</body></html>");
> > > + }
> > > +
> > > + private String getBaseURL(HttpServletRequest request) {
> > > + String reqPerfix = request.getRequestURL().toString();
> > > + String pathInfo = request.getPathInfo() == null ? "" :
> > > request.getPathInfo(); + reqPerfix = reqPerfix.substring(0,
> > > reqPerfix.length() - pathInfo.length()); + return reqPerfix;
> > > }
> > >
> > > protected void generateNotFound(HttpServletRequest request,
> > > HttpServletResponse res) throws IOException {
> > >
> > > Modified:
> > > incubator/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/
> > >transport/servlet/ServletDestination.java URL:
> > > http://svn.apache.org/viewvc/incubator/cxf/trunk/rt/transports/http/
> > >src/main/java/org/apache/cxf/transport/servlet/ServletDestination.jav
> > >a?view=diff&rev=531341&r1=531340&r2=531341
> > > ====================================================================
> > >========== ---
> > > incubator/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/
> > >transport/servlet/ServletDestination.java (original) +++
> > > incubator/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/
> > >transport/servlet/ServletDestination.java Sun Apr 22 22:56:43 2007 @@
> > > -101,7 +101,20 @@
> > > }
> > > }
> > > }
> > > -
> > > +
> > > + public String getAddressPath() {
> > > + String path = endpointInfo.getAddress();
> > > + String lh = "http://localhost/";
> > > + String lhs = "https://localhost/";
> > > +
> > > + if (path.startsWith(lh)) {
> > > + path = "/" + path.substring(lh.length());
> > > + } else if (path.startsWith(lhs)) {
> > > + path = "/" + path.substring(lhs.length());
> > > + }
> > > + return path;
> > > + }
> > > +
> > > public MessageObserver getMessageObserver() {
> > > return this.incomingObserver;
> > > }
> > >
> > > Modified:
> > > incubator/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/
> > >transport/servlet/ServletTransportFactory.java URL:
> > > http://svn.apache.org/viewvc/incubator/cxf/trunk/rt/transports/http/
> > >src/main/java/org/apache/cxf/transport/servlet/ServletTransportFactor
> > >y.java?view=diff&rev=531341&r1=531340&r2=531341
> > > ====================================================================
> > >========== ---
> > > incubator/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/
> > >transport/servlet/ServletTransportFactory.java (original) +++
> > > incubator/cxf/trunk/rt/transports/http/src/main/java/org/apache/cxf/
> > >transport/servlet/ServletTransportFactory.java Sun Apr 22 22:56:43
> > > 2007 @@ -60,16 +60,28 @@
> > >
> > > public Destination getDestination(EndpointInfo endpointInfo)
> > > throws IOException {
> > > - ServletDestination d =
> > > destinations.get(endpointInfo.getAddress()); +
> > > ServletDestination d =
> > > getDestinationForPath(endpointInfo.getAddress()); if (d == null) {
> > > d = new ServletDestination(bus, null, endpointInfo);
> > > - destinations.put(endpointInfo.getAddress(), d);
> > > +
> > > destinations.put(getTrimmedPath(endpointInfo.getAddress()), d); }
> > > return d;
> > > }
> > >
> > > public ServletDestination getDestinationForPath(String path) {
> > > - return destinations.get(path);
> > > + return destinations.get(getTrimmedPath(path));
> > > + }
> > > +
> > > + private String getTrimmedPath(String path) {
> > > + String lh = "http://localhost/";
> > > + String lhs = "https://localhost/";
> > > +
> > > + if (path.startsWith(lh)) {
> > > + path = "/" + path.substring(lh.length());
> > > + } else if (path.startsWith(lhs)) {
> > > + path = "/" + path.substring(lhs.length());
> > > + }
> > > + return path;
> > > }
> > >
> > > public Collection<ServletDestination> getDestinations() {
> > >
> > > Modified:
> > > incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/se
> > >rvlet/AbstractServletTest.java URL:
> > > http://svn.apache.org/viewvc/incubator/cxf/trunk/systests/src/test/j
> > >ava/org/apache/cxf/systest/servlet/AbstractServletTest.java?view=diff
> > >&rev=531341&r1=531340&r2=531341
> > > ====================================================================
> > >========== ---
> > > incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/se
> > >rvlet/AbstractServletTest.java (original) +++
> > > incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/se
> > >rvlet/AbstractServletTest.java Sun Apr 22 22:56:43 2007 @@ -19,10
> > > +19,7 @@
> > > package org.apache.cxf.systest.servlet;
> > >
> > > import java.io.IOException;
> > > -import java.lang.ref.WeakReference;
> > > -import java.lang.reflect.Field;
> > > import java.net.MalformedURLException;
> > > -import java.util.Map;
> > >
> > > import org.xml.sax.SAXException;
> > >
> > > @@ -33,10 +30,7 @@
> > > import com.meterware.servletunit.ServletRunner;
> > > import com.meterware.servletunit.ServletUnitClient;
> > >
> > > -import org.apache.cxf.Bus;
> > > -import org.apache.cxf.helpers.CastUtils;
> > > import org.apache.cxf.test.AbstractCXFTest;
> > > -import org.apache.cxf.transport.servlet.CXFServlet;
> > > import org.junit.Before;
> > >
> > > public abstract class AbstractServletTest extends AbstractCXFTest {
> > > @@ -52,13 +46,6 @@
> > > } catch (HttpNotFoundException e) {
> > > // ignore, we just want to boot up the servlet
> > > }
> > > -
> > > - Field f = CXFServlet.class.getDeclaredField("BUS_MAP");
> > > - f.setAccessible(true);
> > > - Map<String, WeakReference<Bus>> obj =
> > > CastUtils.cast((Map<?, ?>)f.get(null)); - if
> > > (obj.containsKey("servlet.systest.bus.id")) {
> > > - bus = obj.get("servlet.systest.bus.id").get();
> > > - }
> > >
> > > HttpUnitOptions.setExceptionsThrownOnErrorStatus(true);
> > > }
> > >
> > > Modified:
> > > incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/se
> > >rvlet/CXFServletTest.java URL:
> > > http://svn.apache.org/viewvc/incubator/cxf/trunk/systests/src/test/j
> > >ava/org/apache/cxf/systest/servlet/CXFServletTest.java?view=diff&rev=
> > >531341&r1=531340&r2=531341
> > > ====================================================================
> > >========== ---
> > > incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/se
> > >rvlet/CXFServletTest.java (original) +++
> > > incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/se
> > >rvlet/CXFServletTest.java Sun Apr 22 22:56:43 2007 @@ -29,6 +29,8 @@
> > > import com.meterware.httpunit.WebResponse;
> > > import com.meterware.servletunit.ServletUnitClient;
> > >
> > > +import org.apache.cxf.Bus;
> > > +import org.apache.cxf.BusException;
> > > import org.apache.cxf.helpers.DOMUtils;
> > > import org.apache.cxf.jaxws.JaxWsServerFactoryBean;
> > > import org.apache.cxf.service.invoker.BeanInvoker;
> > > @@ -38,13 +40,18 @@
> > >
> > > public class CXFServletTest extends AbstractServletTest {
> > >
> > > + @Override
> > > + protected Bus createBus() throws BusException {
> > > + return null;
> > > + }
> > > +
> > > // Create the JaxWsService with the JaxWsServerFactoryBean
> > > protected void setupJaxwsService() {
> > > JaxWsServerFactoryBean svr = new JaxWsServerFactoryBean();
> > > URL resource =
> > > getClass().getResource("/wsdl/hello_world.wsdl");
> > > assertNotNull(resource);
> > > svr.getServiceFactory().setWsdlURL(resource.toString());
> > > - svr.setBus(getBus());
> > > +// svr.setBus(getBus());
> > > svr.setServiceClass(GreeterImpl.class);
> > > svr.setAddress("http://localhost/services/Greeter");
> > > GreeterImpl greeter = new GreeterImpl();
> > >
> > > Modified:
> > > incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/se
> > >rvlet/ExternalServicesServletTest.java URL:
> > > http://svn.apache.org/viewvc/incubator/cxf/trunk/systests/src/test/j
> > >ava/org/apache/cxf/systest/servlet/ExternalServicesServletTest.java?v
> > >iew=diff&rev=531341&r1=531340&r2=531341
> > > ====================================================================
> > >========== ---
> > > incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/se
> > >rvlet/ExternalServicesServletTest.java (original) +++
> > > incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/se
> > >rvlet/ExternalServicesServletTest.java Sun Apr 22 22:56:43 2007 @@
> > > -24,11 +24,18 @@
> > > import com.meterware.httpunit.WebRequest;
> > > import com.meterware.httpunit.WebResponse;
> > >
> > > +import org.apache.cxf.Bus;
> > > +import org.apache.cxf.BusException;
> > > import org.apache.cxf.helpers.DOMUtils;
> > > import org.junit.Test;
> > >
> > > public class ExternalServicesServletTest extends
> > > AbstractServletTest {
> > >
> > > + @Override
> > > + protected Bus createBus() throws BusException {
> > > + return null;
> > > + }
> > > +
> > > @Override
> > > protected String getConfiguration() {
> > > return "/org/apache/cxf/systest/servlet/web-external.xml";
>
> --
> J. Daniel Kulp
> Principal Engineer
> IONA
> P: 781-902-8727 C: 508-380-7194
> daniel.kulp@iona.com
> http://www.dankulp.com/blog
>
--
Dan Diephouse
Envoi Solutions
http://envoisolutions.com | http://netzooid.com/blog