Details

    • Type: Bug Bug
    • Status: Resolved Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.0-M5
    • Component/s: None
    • Labels:
      None
    • Number of attachments :
      0

      Description

      the example at http://xfire.codehaus.org/Spring throws an exception when generating the wsdl.

      org.codehaus.yom.IllegalDataException: Attribute data cannot start with '/'.
      org.codehaus.yom.util.XMLUtils.assertValidAttributeData(XMLUtils.java:72)
      org.codehaus.yom.Attribute.setValue(Attribute.java:140)
      org.codehaus.yom.Attribute.<init>(Attribute.java:49)
      org.codehaus.yom.Attribute.<init>(Attribute.java:42)
      org.codehaus.yom.converters.DOMConverter.convert(DOMConverter.java:19)
      org.codehaus.yom.converters.DOMConverter.convert(DOMConverter.java:78)
      org.codehaus.yom.converters.DOMConverter.convert(DOMConverter.java:92)
      org.codehaus.yom.converters.DOMConverter.convert(DOMConverter.java:31)
      org.codehaus.xfire.wsdl11.builder.AbstractWSDL.writeDocument(AbstractWSDL.java:97)
      org.codehaus.xfire.wsdl11.builder.AbstractJavaWSDL.<init>(AbstractJavaWSDL.java:55)
      org.codehaus.xfire.wsdl11.builder.DocumentWSDL.<init>(DocumentWSDL.java:30)
      org.codehaus.xfire.wsdl11.builder.WrappedWSDL.<init>(WrappedWSDL.java:30)
      org.codehaus.xfire.wsdl11.builder.WSDLBuilder.createWSDLWriter(WSDLBuilder.java:44)
      org.codehaus.xfire.service.object.DefaultObjectService.getWSDLWriter(DefaultObjectService.java:169)
      org.codehaus.xfire.DefaultXFire.getWSDL(DefaultXFire.java:165)
      org.codehaus.xfire.DefaultXFire.generateWSDL(DefaultXFire.java:146)
      org.codehaus.xfire.transport.http.XFireServletController.generateWSDL(XFireServletController.java:247)
      org.codehaus.xfire.transport.http.XFireServletController.doService(XFireServletController.java:104)
      org.codehaus.xfire.spring.XFireExporter.handleRequest(XFireExporter.java:78)
      org.springframework.web.servlet.mvc.SimpleControllerHandlerAdapter.handle(SimpleControllerHandlerAdapter.java:44)
      org.springframework.web.servlet.DispatcherServlet.doDispatch(DispatcherServlet.java:675)
      org.springframework.web.servlet.DispatcherServlet.doService(DispatcherServlet.java:623)
      org.springframework.web.servlet.FrameworkServlet.serviceWrapper(FrameworkServlet.java:384)
      org.springframework.web.servlet.FrameworkServlet.doGet(FrameworkServlet.java:344)
      javax.servlet.http.HttpServlet.service(HttpServlet.java:697)
      javax.servlet.http.HttpServlet.service(HttpServlet.java:810)
      org.jboss.web.tomcat.filters.ReplyHeaderFilter.doFilter(ReplyHeaderFilter.java:75)

        Activity

        Hide
        Dan Diephouse added a comment -

        You're right, this is a YOM issue. I'm just not on the ball today... I will post a new YOM snapshot tomorrow with a bug fix. Thanks Chris.

        Show
        Dan Diephouse added a comment - You're right, this is a YOM issue. I'm just not on the ball today... I will post a new YOM snapshot tomorrow with a bug fix. Thanks Chris.
        Arjen Poutsma made changes -
        Field Original Value New Value
        Assignee Dan Diephouse [ dandiep ] Arjen Poutsma [ poutsma ]
        Arjen Poutsma made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        Arjen Poutsma added a comment -

        Is it really a YOM issue? I can imagine that the service gets exported with a serviceName /Echo, because by default the DispatcherServlet is using the Beannamemappinghandler. This handlerMapping exposes beans by their names (hence /Echo).

        Note that the spring example in the distribution doesn't use the BeanNameMappingHandler, and therefore works fine.

        Show
        Arjen Poutsma added a comment - Is it really a YOM issue? I can imagine that the service gets exported with a serviceName /Echo, because by default the DispatcherServlet is using the Beannamemappinghandler. This handlerMapping exposes beans by their names (hence /Echo). Note that the spring example in the distribution doesn't use the BeanNameMappingHandler, and therefore works fine.
        Hide
        Dan Diephouse added a comment -

        I deployed a new jar with this bug fixed:

        http://dist.codehaus.org/yom/jars/yom-1.0-SNAPSHOT.jar

        We still however should sync the spring example in the distribution and the user's guide example.

        Show
        Dan Diephouse added a comment - I deployed a new jar with this bug fixed: http://dist.codehaus.org/yom/jars/yom-1.0-SNAPSHOT.jar We still however should sync the spring example in the distribution and the user's guide example.
        Hide
        Christoph Sturm added a comment -

        actually its still not fixed

        you changed ValidityUtils.assertValidData(), but there's still XMLUtils.assertValidAttributeData, which is called in this case.

        -chris

        Show
        Christoph Sturm added a comment - actually its still not fixed you changed ValidityUtils.assertValidData(), but there's still XMLUtils.assertValidAttributeData, which is called in this case. -chris
        Hide
        Christoph Sturm added a comment -

        this works now

        Show
        Christoph Sturm added a comment - this works now
        Christoph Sturm made changes -
        Resolution Fixed [ 1 ]
        Fix Version/s 1.0-M5 [ 11600 ]
        Status In Progress [ 3 ] Resolved [ 5 ]
        Hide
        Arjen Poutsma added a comment -

        It seems that this is rekated to the Spring exporter. The XFireExporter exposes services by their bean name, and this can (and often will) contain a slash, especially if using a BeanNameUrlHandlerMapping (which is the default in the DispatcherServlet).

        Show
        Arjen Poutsma added a comment - It seems that this is rekated to the Spring exporter. The XFireExporter exposes services by their bean name, and this can (and often will) contain a slash, especially if using a BeanNameUrlHandlerMapping (which is the default in the DispatcherServlet).
        Arjen Poutsma made changes -
        Status Resolved [ 5 ] Reopened [ 4 ]
        Resolution Fixed [ 1 ]
        Hide
        Arjen Poutsma added a comment -

        Should be fixed in cvs.

        Show
        Arjen Poutsma added a comment - Should be fixed in cvs.
        Arjen Poutsma made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]

          People

          • Assignee:
            Arjen Poutsma
            Reporter:
            Christoph Sturm
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: