Why was this marked as invalid?  Looks like you fixed exactly the problem.

Jeff

David Jencks (JIRA) wrote:
     [ http://issues.apache.org/jira/browse/GERONIMO-666?page=all ]
David Jencks closed GERONIMO-666:
---------------------------------

    Resolution: Invalid

I've added a web-service-virtual-host element of unbounded multiplicity to 
support jetty.  Tomcat only looks at the first element if present.   The 
geronimo commit also includes partial support for web-app virtual hosts, 
GERONIMO-845.

openejb:
Checking in modules/core/src/java/org/openejb/server/axis/WSContainer.java;
new revision: 1.14; previous revision: 1.13
Checking in modules/core/src/java/org/openejb/server/axis/WSContainerGBean.java;
new revision: 1.8; previous revision: 1.7
Checking in modules/core/src/java/org/openejb/server/soap/SoapHttpListener.java;
new revision: 1.12; previous revision: 1.11
Checking in 
modules/openejb-builder/src/java/org/openejb/deployment/SessionBuilder.java;
new revision: 1.34; previous revision: 1.33
Checking in modules/openejb-builder/src/schema/openejb-jar.xsd;
new revision: 1.21; previous revision: 1.20

geronimo:
Sending        
trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyContainerImpl.java
Sending        
trunk/modules/jetty/src/java/org/apache/geronimo/jetty/JettyWebAppContext.java
Sending        
trunk/modules/jetty/src/test/org/apache/geronimo/jetty/ContainerTest.java
Sending        
trunk/modules/tomcat/src/java/org/apache/geronimo/tomcat/TomcatContainer.java
Sending        
trunk/modules/tomcat/src/test/org/apache/geronimo/tomcat/ContainerTest.java
Sending        
trunk/modules/webservices/src/java/org/apache/geronimo/webservices/SoapHandler.java
Transmitting file data ......
Committed revision 234156.


The addWebservice() call in SoapHandler, JettyContainerImpl needs to support a 
Virtual Host
-------------------------------------------------------------------------------------------

        Key: GERONIMO-666
        URL: http://issues.apache.org/jira/browse/GERONIMO-666
    Project: Geronimo
       Type: Bug
 Components: webservices
   Versions: 1.0-M4
   Reporter: Jeff Genender
   Assignee: David Jencks
   Priority: Blocker
    Fix For: 1.0-M5


The addWebService() call in SoapHandler, JettyContainerImpl needs to support a 
Virtual Host.  This may require the OpenEJBModuleBuilder support this concept 
as well (not sure).  The addWebService()  should have a String virtualHost 
parameter, so that the created context can be applied to a particular virtual 
host.
As it stands the Tomcat integration requires the ability to set the web service 
to a particular host.  The Tomcat webservice integration is blocked until this 
is fixed.


Reply via email to