[ https://jira.jboss.org/browse/JBWS-3131?page=com.atlassian.jira.plugin.sy... ] Fernando Rubbo updated JBWS-3131: --------------------------------- Fix Version/s: (was: jbossws-native-3.0.4) Affects Version/s: jbossws-native-3.4.0 Environment: AS 4.x, AS 5.x (was: AS 5.0.0.CR1) Description: I've cloned this issue because it was closed and I was not able to reopen it. The bug is on class org.jboss.ws.metadata.builder.MetaDataBuilder in initEndpointBinding(..) method. Below the explanation of the bug and the patch to fix the problem. protected void initEndpointBinding(WSDLEndpoint wsdlEndpoint, EndpointMetaData epMetaData) { WSDLDefinitions wsdlDefinitions = wsdlEndpoint.getWsdlService().getWsdlDefinitions(); //WSDLInterface wsdlInterface = wsdlEndpoint.getInterface(); //WSDLBinding wsdlBinding = wsdlDefinitions.getBindingByInterfaceName(wsdlInterface.getName()); /* BEGIN PATCH */ /* * Both commented lines above were replaced by the piece of code below. * * CAUSE: Once an interface may have more than one binding and once, at this point of the code, * we already know which binding to use (it is inside of the wsdlEndpoint), we can simple get the binding * of wsdlEndpoint INSTEAD of trying to guess which binding the user would like to use. * * NOTE: This bug is used to be intermittent because it depends on the name given for the binding. The bindings property * inside of WSDLBinding is a LinkedHashMap in which the key is the binding Name. Once the method wsdlDefinitions.getBindingByInterfaceName(..) * is called it iterates over bindings proprety and gets the last binding associated to the given interface. Note that we * MUST NOT get the last one. We know what binding we would like to use. * * KNOWN ISSUE: This issue was already submitted in the past (https://jira.jboss.org/browse/JBWS-2278?page=com.atlassian.jira.plugin.sy...) * but it seems that it was closed because the developer did not understood exactly what the problem was about. Moreover * it seems that up until now it was not fixed yet. Even in the newest version of the jbossws-core. */ WSDLBinding wsdlBinding; final QName binding = wsdlEndpoint.getBinding(); if(binding!=null){ wsdlBinding = wsdlDefinitions.getBinding(binding); }else{ // These lines are here only for backward compatibility. Supose a case which wsdlEndpoint does // not contain a binding associated. In this case, it will keep doing as it did before. WSDLInterface wsdlInterface = wsdlEndpoint.getInterface(); wsdlBinding = wsdlDefinitions.getBindingByInterfaceName(wsdlInterface.getName()); } /* END PATCH */ String bindingType = wsdlBinding.getType(); if (Constants.NS_SOAP11.equals(bindingType)) { epMetaData.setBindingId(Constants.SOAP11HTTP_BINDING); } else if (Constants.NS_SOAP12.equals(bindingType)) { epMetaData.setBindingId(Constants.SOAP12HTTP_BINDING); } } The WSDL file: <?xml version="1.0" encoding="utf-8"?> <wsdl:definitions xmlns:soap="http://schemas.xmlsoap.org/wsdl/soap/" xmlns:tm="http://microsoft.com/wsdl/mime/textMatching/" xmlns:soapenc="http://schemas.xmlsoap.org/soap/encoding/" xmlns:mime="http://schemas.xmlsoap.org/wsdl/mime/" xmlns:tns="http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2" xmlns:s="http://www.w3.org/2001/XMLSchema" xmlns:soap12="http://schemas.xmlsoap.org/wsdl/soap12/" xmlns:http="http://schemas.xmlsoap.org/wsdl/http/" targetNamespace="http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2" xmlns:wsdl="http://schemas.xmlsoap.org/wsdl/"> <wsdl:types> <s:schema elementFormDefault="qualified" targetNamespace="http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2"> <s:element name="nfeDadosMsg"> <s:complexType mixed="true"> <s:sequence> <s:any /> </s:sequence> </s:complexType> </s:element> <s:element name="nfeStatusServicoNF2Result"> <s:complexType mixed="true"> <s:sequence> <s:any /> </s:sequence> </s:complexType> </s:element> <s:element name="nfeCabecMsg" type="tns:nfeCabecMsg" /> <s:complexType name="nfeCabecMsg"> <s:sequence> <s:element minOccurs="0" maxOccurs="1" name="versaoDados" type="s:string" /> <s:element minOccurs="0" maxOccurs="1" name="cUF" type="s:string" /> </s:sequence> <s:anyAttribute /> </s:complexType> </s:schema> </wsdl:types> <wsdl:message name="nfeStatusServicoNF2SoapIn"> <wsdl:part name="nfeDadosMsg" element="tns:nfeDadosMsg" /> </wsdl:message> <wsdl:message name="nfeStatusServicoNF2SoapOut"> <wsdl:part name="nfeStatusServicoNF2Result" element="tns:nfeStatusServicoNF2Result" /> </wsdl:message> <wsdl:message name="nfeStatusServicoNF2nfeCabecMsg"> <wsdl:part name="nfeCabecMsg" element="tns:nfeCabecMsg" /> </wsdl:message> <wsdl:portType name="NfeStatusServico2Soap"> <wsdl:operation name="nfeStatusServicoNF2"> <wsdl:documentation xmlns:wsdl="http://schemas.xmlsoap.org/wsdl/">Serviço destinado à consulta do status do serviço prestado pela Sefaz Virtual do Ambiente Nacional</wsdl:documentation> <wsdl:input message="tns:nfeStatusServicoNF2SoapIn" /> <wsdl:output message="tns:nfeStatusServicoNF2SoapOut" /> </wsdl:operation> </wsdl:portType> <wsdl:binding name="NfeStatusServico2Soap" type="tns:NfeStatusServico2Soap"> <soap:binding transport="http://schemas.xmlsoap.org/soap/http" /> <wsdl:operation name="nfeStatusServicoNF2"> <soap:operation soapAction="http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2/nfeStatusServicoNF2" style="document" /> <wsdl:input> <soap:body use="literal" /> <soap:header message="tns:nfeStatusServicoNF2nfeCabecMsg" part="nfeCabecMsg" use="literal" /> </wsdl:input> <wsdl:output> <soap:body use="literal" /> </wsdl:output> </wsdl:operation> </wsdl:binding> <wsdl:binding name="NfeStatusServico2Soap12" type="tns:NfeStatusServico2Soap"> <soap12:binding transport="http://schemas.xmlsoap.org/soap/http" /> <wsdl:operation name="nfeStatusServicoNF2"> <soap12:operation soapAction="http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2/nfeStatusServicoNF2" style="document" /> <wsdl:input> <soap12:body use="literal" /> <soap12:header message="tns:nfeStatusServicoNF2nfeCabecMsg" part="nfeCabecMsg" use="literal" /> </wsdl:input> <wsdl:output> <soap12:body use="literal" /> </wsdl:output> </wsdl:operation> </wsdl:binding> <wsdl:service name="NfeStatusServico2"> <wsdl:port name="NfeStatusServico2Soap" binding="tns:NfeStatusServico2Soap"> <soap:address location="https://hom.nfe.fazenda.gov.br/SCAN/NfeStatusServico2/NfeStatusServico2.asmx" /> </wsdl:port> <wsdl:port name="NfeStatusServico2Soap12" binding="tns:NfeStatusServico2Soap12"> <soap12:address location="https://hom.nfe.fazenda.gov.br/SCAN/NfeStatusServico2/NfeStatusServico2.asmx" /> </wsdl:port> </wsdl:service> </wsdl:definitions> Also, I've done some unit testes to show the problem. P.S.: Remember to set in the "VM Arguments" the following property. -Djava.endorsed.dirs="<JBOSS_HOME>\\lib\\endorsed" / ** * @author Fernando Rubbo */ public class TestWSDLWithTwoBindingsForOneInterface { private final String PATH_WSDL_FILE = "<PATH>\\NfeStatusServico2.wsdl"; private final String PATH_CHANGED_WSDL_FILE = "<PATH>\\NfeStatusServico21.wsdl"; private Service service; private Service serviceChanged; @Before public void setUp() throws IOException { URL wsdlLoc = new URL(PATH_WSDL_FILE); QName serviceName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2"); service = Service.create(wsdlLoc, serviceName); /* * After changing the line 72 of the WSDL. We have no more problems. * Original line * <wsdl:port name="NfeStatusServico2Soap" binding="tns:NfeStatusServico2Soap"> * Modified line * <wsdl:port name="NfeStatusServico2Soap1" binding="tns:NfeStatusServico2Soap"> */ wsdlLoc = new URL(PATH_CHANGED_WSDL_FILE); serviceName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2"); serviceChanged = Service.create(wsdlLoc, serviceName); } @Test public void testSOAP11OnOriginalWSDL() throws IOException { // asking for using the SOAP 1.1 binding. no problem... QName portName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2Soap"); Dispatch<Source> dispatch = service.createDispatch(portName, Source.class, Mode.MESSAGE); String bindingID = dispatch.getBinding().getBindingID(); Assert.assertEquals("http://schemas.xmlsoap.org/wsdl/soap/http";, bindingID); } @Test public void testSOAP12OnOriginalWSDL() throws IOException { // As you can see, the below code is asking for using the SOAP 1.2 binding. It should return a dispath for // the SOAP 1.2 binding, however it does not behaviour as expected... QName portName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2Soap12"); Dispatch<Source> dispatch = service.createDispatch(portName, Source.class, Mode.MESSAGE); String bindingID = dispatch.getBinding().getBindingID(); Assert.assertEquals("http://www.w3.org/2003/05/soap/bindings/HTTP/";, bindingID); } @Test public void testSOAP11OnChangedWSDL() throws IOException { // in this test we are asking for using the SOAP 1.1 binding but using the changed WSDL. Note that the order of the // the bindings property of WSDLBinding has changed. Therefore, it gets the wrong binding for the SOAP 1.1. QName portName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2Soap1"); Dispatch<Source> dispatch = serviceChanged.createDispatch(portName, Source.class, Mode.MESSAGE); String bindingID = dispatch.getBinding().getBindingID(); Assert.assertEquals("http://schemas.xmlsoap.org/wsdl/soap/http";, bindingID); } @Test public void testSOAP12OnChangedWSDL() throws IOException { // Since the order the bindings property of WSDLBinding has changed we have no problem asking for using the SOAP 1.2 binding. QName portName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2Soap12"); Dispatch<Source> dispatch = serviceChanged.createDispatch(portName, Source.class, Mode.MESSAGE); String bindingID = dispatch.getBinding().getBindingID(); Assert.assertEquals("http://www.w3.org/2003/05/soap/bindings/HTTP/";, bindingID); } } Hope this helps to fix the problem. Thanks in advance ---------------------------------------------------------------------------------------------------------------------------------------------------- I am implementing an interop test using WSDL organized as follows: <?xml version="1.0" encoding="UTF-8"?> <wsdl:definitions targetNamespace="http://www.wstf.org/docs/scenarios/sc002" xmlns:xs="http://www.w3.org/2001/XMLSchema" xmlns:soap="http://schemas.xmlsoap.org/wsdl/soap/" xmlns:soap12="http://schemas.xmlsoap.org/wsdl/soap12/" xmlns:wsdl="http://schemas.xmlsoap.org/wsdl/" xmlns:tns="http://www.wstf.org/docs/scenarios/sc002"> <wsdl:types> . . . </wsdl:types> <wsdl:message name="Begin"> <wsdl:part name="Begin" element="tns:Begin"/> . . . </wsdl:message> <wsdl:portType name="sc002Port"> <wsdl:operation name="Begin"> . . . </wsdl:portType> <wsdl:binding name="sc002SOAP11Binding" type="tns:sc002Port"> <soap:binding style="document" transport="http://schemas.xmlsoap.org/soap/http"/> <wsdl:operation name="Begin"> . . . </wsdl:binding> <wsdl:binding name="sc002SOAP12Binding" type="tns:sc002Port"> <soap12:binding style="document" transport="http://schemas.xmlsoap.org/soap/http"/> <wsdl:operation name="Begin"> . . . </wsdl:binding> <wsdl:service name="sc002Service"> <wsdl:port name="soap11port" binding="tns:sc002SOAP11Binding"> <soap:address location="http://www.wstf.org/sc002/sc002SOAP11"/> </wsdl:port> <wsdl:port name="soap12port" binding="tns:sc002SOAP12Binding"> <soap12:address location="http://www.wstf.org/sc002/sc002SOAP12"/> </wsdl:port> </wsdl:service> </wsdl:definitions> In other words, there is one port, two bindings and one service exporting the two bindings. I create a client proxy for the port named "soap11port" by invoking new Sc002Service().getPort(new QName("http://www.wstf.org/docs/scenarios/sc002";, "soap11port"), Sc002Port.class) The returned port uses a SOAP12 binding even though the WSDL labels it clearly as a 1.1 port. The problem arises because of an error in the construction of the ClientEndPointMetaData used as the endpointMetaData attached to the ClientProxy. This is constructed inside the ServiceDelegateImpl constructor by invoking JAXWSClientMetaDataBuilder.buildMetaData(). The error occurs in inherited method MetaDataBuilder.initEndpointBinding() when it tries to identify the binding associated with QName sc002:soap11port. The following 4 lines are where it goes wrong: WSDLDefinitions wsdlDefinitions = wsdlEndpoint.getWsdlService().getWsdlDefinitions(); WSDLInterface wsdlInterface = wsdlEndpoint.getInterface(); WSDLBinding wsdlBinding = wsdlDefinitions.getBindingByInterfaceName(wsdlInterface.getName()); String bindingType = wsdlBinding.getType(); The first two lines locate the port sc002 associated with the binding identified by sc002:soap11port. The 3rd line then reverse searches from the port to find a binding which binds it. Of course there are two of these and, mirabile dictu, the code even prints a warning about this. It then ignores the correct and returns the soap12 binding. Ironically before reaching this point the JAXWSClientMetaDataBuilder does this search correctly in method buildMetaDataInternal at lines 154/5 it executes this code WSDLBinding wsdlBinding = wsdlEndpoint.getWsdlService().getWsdlDefinitions().getBinding(bindingName); String bindingType = wsdlBinding.getType(); I assume the convoluted code in the inherited method is there for a reason but, then again, that's what overriding was invented for. was: I am implementing an interop test using WSDL organized as follows: <?xml version="1.0" encoding="UTF-8"?> <wsdl:definitions targetNamespace="http://www.wstf.org/docs/scenarios/sc002" xmlns:xs="http://www.w3.org/2001/XMLSchema" xmlns:soap="http://schemas.xmlsoap.org/wsdl/soap/" xmlns:soap12="http://schemas.xmlsoap.org/wsdl/soap12/" xmlns:wsdl="http://schemas.xmlsoap.org/wsdl/" xmlns:tns="http://www.wstf.org/docs/scenarios/sc002"> <wsdl:types> . . . </wsdl:types> <wsdl:message name="Begin"> <wsdl:part name="Begin" element="tns:Begin"/> . . . </wsdl:message> <wsdl:portType name="sc002Port"> <wsdl:operation name="Begin"> . . . </wsdl:portType> <wsdl:binding name="sc002SOAP11Binding" type="tns:sc002Port"> <soap:binding style="document" transport="http://schemas.xmlsoap.org/soap/http"/> <wsdl:operation name="Begin"> . . . </wsdl:binding> <wsdl:binding name="sc002SOAP12Binding" type="tns:sc002Port"> <soap12:binding style="document" transport="http://schemas.xmlsoap.org/soap/http"/> <wsdl:operation name="Begin"> . . . </wsdl:binding> <wsdl:service name="sc002Service"> <wsdl:port name="soap11port" binding="tns:sc002SOAP11Binding"> <soap:address location="http://www.wstf.org/sc002/sc002SOAP11"/> </wsdl:port> <wsdl:port name="soap12port" binding="tns:sc002SOAP12Binding"> <soap12:address location="http://www.wstf.org/sc002/sc002SOAP12"/> </wsdl:port> </wsdl:service> </wsdl:definitions> In other words, there is one port, two bindings and one service exporting the two bindings. I create a client proxy for the port named "soap11port" by invoking new Sc002Service().getPort(new QName("http://www.wstf.org/docs/scenarios/sc002";, "soap11port"), Sc002Port.class) The returned port uses a SOAP12 binding even though the WSDL labels it clearly as a 1.1 port. The problem arises because of an error in the construction of the ClientEndPointMetaData used as the endpointMetaData attached to the ClientProxy. This is constructed inside the ServiceDelegateImpl constructor by invoking JAXWSClientMetaDataBuilder.buildMetaData(). The error occurs in inherited method MetaDataBuilder.initEndpointBinding() when it tries to identify the binding associated with QName sc002:soap11port. The following 4 lines are where it goes wrong: WSDLDefinitions wsdlDefinitions = wsdlEndpoint.getWsdlService().getWsdlDefinitions(); WSDLInterface wsdlInterface = wsdlEndpoint.getInterface(); WSDLBinding wsdlBinding = wsdlDefinitions.getBindingByInterfaceName(wsdlInterface.getName()); String bindingType = wsdlBinding.getType(); The first two lines locate the port sc002 associated with the binding identified by sc002:soap11port. The 3rd line then reverse searches from the port to find a binding which binds it. Of course there are two of these and, mirabile dictu, the code even prints a warning about this. It then ignores the correct and returns the soap12 binding. Ironically before reaching this point the JAXWSClientMetaDataBuilder does this search correctly in method buildMetaDataInternal at lines 154/5 it executes this code WSDLBinding wsdlBinding = wsdlEndpoint.getWsdlService().getWsdlDefinitions().getBinding(bindingName); String bindingType = wsdlBinding.getType(); I assume the convoluted code in the inherited method is there for a reason but, then again, that's what overriding was invented for. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://jira.jboss.org/browse/JBWS-3131?page=com.atlassian.jira.plugin.sy... ] Fernando Rubbo updated JBWS-3131: --------------------------------- Description: I've cloned this issue because it was closed and I was not able to reopen it. The bug is on class org.jboss.ws.metadata.builder.MetaDataBuilder in initEndpointBinding(..) method. Below the explanation of the bug and the patch to fix the problem. protected void initEndpointBinding(WSDLEndpoint wsdlEndpoint, EndpointMetaData epMetaData) { WSDLDefinitions wsdlDefinitions = wsdlEndpoint.getWsdlService().getWsdlDefinitions(); //WSDLInterface wsdlInterface = wsdlEndpoint.getInterface(); //WSDLBinding wsdlBinding = wsdlDefinitions.getBindingByInterfaceName(wsdlInterface.getName()); /* BEGIN PATCH */ /* * Both commented lines above were replaced by the piece of code below. * * CAUSE: Once an interface may have more than one binding and once, at this point of the code, * we already know which binding to use (it is inside of the wsdlEndpoint), we can simple get the binding * of wsdlEndpoint INSTEAD of trying to guess which binding the user would like to use. * * NOTE: This bug is used to be intermittent because it depends on the name given for the binding. The bindings property * inside of WSDLBinding is a LinkedHashMap in which the key is the binding Name. Once the method wsdlDefinitions.getBindingByInterfaceName(..) * is called it iterates over bindings proprety and gets the last binding associated to the given interface. Note that we * MUST NOT get the last one. We know what binding we would like to use. * * KNOWN ISSUE: This issue was already submitted in the past (https://jira.jboss.org/browse/JBWS-2278?page=com.atlassian.jira.plugin.sy...) * but it seems that it was closed because the developer did not understood exactly what the problem was about. Moreover * it seems that up until now it was not fixed yet. Even in the newest version of the jbossws-core. */ WSDLBinding wsdlBinding; final QName binding = wsdlEndpoint.getBinding(); if(binding!=null){ wsdlBinding = wsdlDefinitions.getBinding(binding); }else{ // These lines are here only for backward compatibility. Supose a case which wsdlEndpoint does // not contain a binding associated. In this case, it will keep doing as it did before. WSDLInterface wsdlInterface = wsdlEndpoint.getInterface(); wsdlBinding = wsdlDefinitions.getBindingByInterfaceName(wsdlInterface.getName()); } /* END PATCH */ String bindingType = wsdlBinding.getType(); if (Constants.NS_SOAP11.equals(bindingType)) { epMetaData.setBindingId(Constants.SOAP11HTTP_BINDING); } else if (Constants.NS_SOAP12.equals(bindingType)) { epMetaData.setBindingId(Constants.SOAP12HTTP_BINDING); } } The WSDL file which can be used to reproduce the this issue: <?xml version="1.0" encoding="utf-8"?> <wsdl:definitions xmlns:soap="http://schemas.xmlsoap.org/wsdl/soap/" xmlns:tm="http://microsoft.com/wsdl/mime/textMatching/" xmlns:soapenc="http://schemas.xmlsoap.org/soap/encoding/" xmlns:mime="http://schemas.xmlsoap.org/wsdl/mime/" xmlns:tns="http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2" xmlns:s="http://www.w3.org/2001/XMLSchema" xmlns:soap12="http://schemas.xmlsoap.org/wsdl/soap12/" xmlns:http="http://schemas.xmlsoap.org/wsdl/http/" targetNamespace="http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2" xmlns:wsdl="http://schemas.xmlsoap.org/wsdl/"> <wsdl:types> <s:schema elementFormDefault="qualified" targetNamespace="http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2"> <s:element name="nfeDadosMsg"> <s:complexType mixed="true"> <s:sequence> <s:any /> </s:sequence> </s:complexType> </s:element> <s:element name="nfeStatusServicoNF2Result"> <s:complexType mixed="true"> <s:sequence> <s:any /> </s:sequence> </s:complexType> </s:element> <s:element name="nfeCabecMsg" type="tns:nfeCabecMsg" /> <s:complexType name="nfeCabecMsg"> <s:sequence> <s:element minOccurs="0" maxOccurs="1" name="versaoDados" type="s:string" /> <s:element minOccurs="0" maxOccurs="1" name="cUF" type="s:string" /> </s:sequence> <s:anyAttribute /> </s:complexType> </s:schema> </wsdl:types> <wsdl:message name="nfeStatusServicoNF2SoapIn"> <wsdl:part name="nfeDadosMsg" element="tns:nfeDadosMsg" /> </wsdl:message> <wsdl:message name="nfeStatusServicoNF2SoapOut"> <wsdl:part name="nfeStatusServicoNF2Result" element="tns:nfeStatusServicoNF2Result" /> </wsdl:message> <wsdl:message name="nfeStatusServicoNF2nfeCabecMsg"> <wsdl:part name="nfeCabecMsg" element="tns:nfeCabecMsg" /> </wsdl:message> <wsdl:portType name="NfeStatusServico2Soap"> <wsdl:operation name="nfeStatusServicoNF2"> <wsdl:documentation xmlns:wsdl="http://schemas.xmlsoap.org/wsdl/">Serviço destinado à consulta do status do serviço prestado pela Sefaz Virtual do Ambiente Nacional</wsdl:documentation> <wsdl:input message="tns:nfeStatusServicoNF2SoapIn" /> <wsdl:output message="tns:nfeStatusServicoNF2SoapOut" /> </wsdl:operation> </wsdl:portType> <wsdl:binding name="NfeStatusServico2Soap" type="tns:NfeStatusServico2Soap"> <soap:binding transport="http://schemas.xmlsoap.org/soap/http" /> <wsdl:operation name="nfeStatusServicoNF2"> <soap:operation soapAction="http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2/nfeStatusServicoNF2" style="document" /> <wsdl:input> <soap:body use="literal" /> <soap:header message="tns:nfeStatusServicoNF2nfeCabecMsg" part="nfeCabecMsg" use="literal" /> </wsdl:input> <wsdl:output> <soap:body use="literal" /> </wsdl:output> </wsdl:operation> </wsdl:binding> <wsdl:binding name="NfeStatusServico2Soap12" type="tns:NfeStatusServico2Soap"> <soap12:binding transport="http://schemas.xmlsoap.org/soap/http" /> <wsdl:operation name="nfeStatusServicoNF2"> <soap12:operation soapAction="http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2/nfeStatusServicoNF2" style="document" /> <wsdl:input> <soap12:body use="literal" /> <soap12:header message="tns:nfeStatusServicoNF2nfeCabecMsg" part="nfeCabecMsg" use="literal" /> </wsdl:input> <wsdl:output> <soap12:body use="literal" /> </wsdl:output> </wsdl:operation> </wsdl:binding> <wsdl:service name="NfeStatusServico2"> <wsdl:port name="NfeStatusServico2Soap" binding="tns:NfeStatusServico2Soap"> <soap:address location="https://hom.nfe.fazenda.gov.br/SCAN/NfeStatusServico2/NfeStatusServico2.asmx" /> </wsdl:port> <wsdl:port name="NfeStatusServico2Soap12" binding="tns:NfeStatusServico2Soap12"> <soap12:address location="https://hom.nfe.fazenda.gov.br/SCAN/NfeStatusServico2/NfeStatusServico2.asmx" /> </wsdl:port> </wsdl:service> </wsdl:definitions> Also, I've done some unit testes to show the problem. P.S.: Remember to set in the "VM Arguments" the following property. -Djava.endorsed.dirs="<JBOSS_HOME>\\lib\\endorsed" / ** * @author Fernando Rubbo */ public class TestWSDLWithTwoBindingsForOneInterface { private final String PATH_WSDL_FILE = "<PATH>\\NfeStatusServico2.wsdl"; private final String PATH_CHANGED_WSDL_FILE = "<PATH>\\NfeStatusServico21.wsdl"; private Service service; private Service serviceChanged; @Before public void setUp() throws IOException { URL wsdlLoc = new URL(PATH_WSDL_FILE); QName serviceName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2"); service = Service.create(wsdlLoc, serviceName); /* * After changing the line 72 of the WSDL as shown below. We have the order of the wsdlDefinitions.bindings property changed. * Original line * <wsdl:port name="NfeStatusServico2Soap" binding="tns:NfeStatusServico2Soap"> * Modified line * <wsdl:port name="NfeStatusServico2Soap1" binding="tns:NfeStatusServico2Soap"> */ wsdlLoc = new URL(PATH_CHANGED_WSDL_FILE); serviceName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2"); serviceChanged = Service.create(wsdlLoc, serviceName); } @Test public void testSOAP11OnOriginalWSDL() throws IOException { // asking for using the SOAP 1.1 binding. no problem... QName portName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2Soap"); Dispatch<Source> dispatch = service.createDispatch(portName, Source.class, Mode.MESSAGE); String bindingID = dispatch.getBinding().getBindingID(); Assert.assertEquals("http://schemas.xmlsoap.org/wsdl/soap/http";, bindingID); } @Test public void testSOAP12OnOriginalWSDL() throws IOException { // As you can see, the below code is asking for using the SOAP 1.2 binding. It should return a dispath for // the SOAP 1.2 binding, however it does not behaviour as expected... QName portName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2Soap12"); Dispatch<Source> dispatch = service.createDispatch(portName, Source.class, Mode.MESSAGE); String bindingID = dispatch.getBinding().getBindingID(); Assert.assertEquals("http://www.w3.org/2003/05/soap/bindings/HTTP/";, bindingID); } @Test public void testSOAP11OnChangedWSDL() throws IOException { // in this test we are asking for using the SOAP 1.1 binding but using the changed WSDL. Note that the order of the // the bindings property of WSDLBinding has changed. Therefore, it gets the wrong binding for the SOAP 1.1. QName portName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2Soap1"); Dispatch<Source> dispatch = serviceChanged.createDispatch(portName, Source.class, Mode.MESSAGE); String bindingID = dispatch.getBinding().getBindingID(); Assert.assertEquals("http://schemas.xmlsoap.org/wsdl/soap/http";, bindingID); } @Test public void testSOAP12OnChangedWSDL() throws IOException { // Since the order the bindings property of WSDLBinding has changed we have no problem asking for using the SOAP 1.2 binding. QName portName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2Soap12"); Dispatch<Source> dispatch = serviceChanged.createDispatch(portName, Source.class, Mode.MESSAGE); String bindingID = dispatch.getBinding().getBindingID(); Assert.assertEquals("http://www.w3.org/2003/05/soap/bindings/HTTP/";, bindingID); } } Hope this helps to fix the problem. Thanks in advance ---------------------------------------------------------------------------------------------------------------------------------------------------- I am implementing an interop test using WSDL organized as follows: <?xml version="1.0" encoding="UTF-8"?> <wsdl:definitions targetNamespace="http://www.wstf.org/docs/scenarios/sc002" xmlns:xs="http://www.w3.org/2001/XMLSchema" xmlns:soap="http://schemas.xmlsoap.org/wsdl/soap/" xmlns:soap12="http://schemas.xmlsoap.org/wsdl/soap12/" xmlns:wsdl="http://schemas.xmlsoap.org/wsdl/" xmlns:tns="http://www.wstf.org/docs/scenarios/sc002"> <wsdl:types> . . . </wsdl:types> <wsdl:message name="Begin"> <wsdl:part name="Begin" element="tns:Begin"/> . . . </wsdl:message> <wsdl:portType name="sc002Port"> <wsdl:operation name="Begin"> . . . </wsdl:portType> <wsdl:binding name="sc002SOAP11Binding" type="tns:sc002Port"> <soap:binding style="document" transport="http://schemas.xmlsoap.org/soap/http"/> <wsdl:operation name="Begin"> . . . </wsdl:binding> <wsdl:binding name="sc002SOAP12Binding" type="tns:sc002Port"> <soap12:binding style="document" transport="http://schemas.xmlsoap.org/soap/http"/> <wsdl:operation name="Begin"> . . . </wsdl:binding> <wsdl:service name="sc002Service"> <wsdl:port name="soap11port" binding="tns:sc002SOAP11Binding"> <soap:address location="http://www.wstf.org/sc002/sc002SOAP11"/> </wsdl:port> <wsdl:port name="soap12port" binding="tns:sc002SOAP12Binding"> <soap12:address location="http://www.wstf.org/sc002/sc002SOAP12"/> </wsdl:port> </wsdl:service> </wsdl:definitions> In other words, there is one port, two bindings and one service exporting the two bindings. I create a client proxy for the port named "soap11port" by invoking new Sc002Service().getPort(new QName("http://www.wstf.org/docs/scenarios/sc002";, "soap11port"), Sc002Port.class) The returned port uses a SOAP12 binding even though the WSDL labels it clearly as a 1.1 port. The problem arises because of an error in the construction of the ClientEndPointMetaData used as the endpointMetaData attached to the ClientProxy. This is constructed inside the ServiceDelegateImpl constructor by invoking JAXWSClientMetaDataBuilder.buildMetaData(). The error occurs in inherited method MetaDataBuilder.initEndpointBinding() when it tries to identify the binding associated with QName sc002:soap11port. The following 4 lines are where it goes wrong: WSDLDefinitions wsdlDefinitions = wsdlEndpoint.getWsdlService().getWsdlDefinitions(); WSDLInterface wsdlInterface = wsdlEndpoint.getInterface(); WSDLBinding wsdlBinding = wsdlDefinitions.getBindingByInterfaceName(wsdlInterface.getName()); String bindingType = wsdlBinding.getType(); The first two lines locate the port sc002 associated with the binding identified by sc002:soap11port. The 3rd line then reverse searches from the port to find a binding which binds it. Of course there are two of these and, mirabile dictu, the code even prints a warning about this. It then ignores the correct and returns the soap12 binding. Ironically before reaching this point the JAXWSClientMetaDataBuilder does this search correctly in method buildMetaDataInternal at lines 154/5 it executes this code WSDLBinding wsdlBinding = wsdlEndpoint.getWsdlService().getWsdlDefinitions().getBinding(bindingName); String bindingType = wsdlBinding.getType(); I assume the convoluted code in the inherited method is there for a reason but, then again, that's what overriding was invented for. was: I've cloned this issue because it was closed and I was not able to reopen it. The bug is on class org.jboss.ws.metadata.builder.MetaDataBuilder in initEndpointBinding(..) method. Below the explanation of the bug and the patch to fix the problem. protected void initEndpointBinding(WSDLEndpoint wsdlEndpoint, EndpointMetaData epMetaData) { WSDLDefinitions wsdlDefinitions = wsdlEndpoint.getWsdlService().getWsdlDefinitions(); //WSDLInterface wsdlInterface = wsdlEndpoint.getInterface(); //WSDLBinding wsdlBinding = wsdlDefinitions.getBindingByInterfaceName(wsdlInterface.getName()); /* BEGIN PATCH */ /* * Both commented lines above were replaced by the piece of code below. * * CAUSE: Once an interface may have more than one binding and once, at this point of the code, * we already know which binding to use (it is inside of the wsdlEndpoint), we can simple get the binding * of wsdlEndpoint INSTEAD of trying to guess which binding the user would like to use. * * NOTE: This bug is used to be intermittent because it depends on the name given for the binding. The bindings property * inside of WSDLBinding is a LinkedHashMap in which the key is the binding Name. Once the method wsdlDefinitions.getBindingByInterfaceName(..) * is called it iterates over bindings proprety and gets the last binding associated to the given interface. Note that we * MUST NOT get the last one. We know what binding we would like to use. * * KNOWN ISSUE: This issue was already submitted in the past (https://jira.jboss.org/browse/JBWS-2278?page=com.atlassian.jira.plugin.sy...) * but it seems that it was closed because the developer did not understood exactly what the problem was about. Moreover * it seems that up until now it was not fixed yet. Even in the newest version of the jbossws-core. */ WSDLBinding wsdlBinding; final QName binding = wsdlEndpoint.getBinding(); if(binding!=null){ wsdlBinding = wsdlDefinitions.getBinding(binding); }else{ // These lines are here only for backward compatibility. Supose a case which wsdlEndpoint does // not contain a binding associated. In this case, it will keep doing as it did before. WSDLInterface wsdlInterface = wsdlEndpoint.getInterface(); wsdlBinding = wsdlDefinitions.getBindingByInterfaceName(wsdlInterface.getName()); } /* END PATCH */ String bindingType = wsdlBinding.getType(); if (Constants.NS_SOAP11.equals(bindingType)) { epMetaData.setBindingId(Constants.SOAP11HTTP_BINDING); } else if (Constants.NS_SOAP12.equals(bindingType)) { epMetaData.setBindingId(Constants.SOAP12HTTP_BINDING); } } The WSDL file which can be used to reproduce the this issue: <?xml version="1.0" encoding="utf-8"?> <wsdl:definitions xmlns:soap="http://schemas.xmlsoap.org/wsdl/soap/" xmlns:tm="http://microsoft.com/wsdl/mime/textMatching/" xmlns:soapenc="http://schemas.xmlsoap.org/soap/encoding/" xmlns:mime="http://schemas.xmlsoap.org/wsdl/mime/" xmlns:tns="http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2" xmlns:s="http://www.w3.org/2001/XMLSchema" xmlns:soap12="http://schemas.xmlsoap.org/wsdl/soap12/" xmlns:http="http://schemas.xmlsoap.org/wsdl/http/" targetNamespace="http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2" xmlns:wsdl="http://schemas.xmlsoap.org/wsdl/"> <wsdl:types> <s:schema elementFormDefault="qualified" targetNamespace="http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2"> <s:element name="nfeDadosMsg"> <s:complexType mixed="true"> <s:sequence> <s:any /> </s:sequence> </s:complexType> </s:element> <s:element name="nfeStatusServicoNF2Result"> <s:complexType mixed="true"> <s:sequence> <s:any /> </s:sequence> </s:complexType> </s:element> <s:element name="nfeCabecMsg" type="tns:nfeCabecMsg" /> <s:complexType name="nfeCabecMsg"> <s:sequence> <s:element minOccurs="0" maxOccurs="1" name="versaoDados" type="s:string" /> <s:element minOccurs="0" maxOccurs="1" name="cUF" type="s:string" /> </s:sequence> <s:anyAttribute /> </s:complexType> </s:schema> </wsdl:types> <wsdl:message name="nfeStatusServicoNF2SoapIn"> <wsdl:part name="nfeDadosMsg" element="tns:nfeDadosMsg" /> </wsdl:message> <wsdl:message name="nfeStatusServicoNF2SoapOut"> <wsdl:part name="nfeStatusServicoNF2Result" element="tns:nfeStatusServicoNF2Result" /> </wsdl:message> <wsdl:message name="nfeStatusServicoNF2nfeCabecMsg"> <wsdl:part name="nfeCabecMsg" element="tns:nfeCabecMsg" /> </wsdl:message> <wsdl:portType name="NfeStatusServico2Soap"> <wsdl:operation name="nfeStatusServicoNF2"> <wsdl:documentation xmlns:wsdl="http://schemas.xmlsoap.org/wsdl/">Serviço destinado à consulta do status do serviço prestado pela Sefaz Virtual do Ambiente Nacional</wsdl:documentation> <wsdl:input message="tns:nfeStatusServicoNF2SoapIn" /> <wsdl:output message="tns:nfeStatusServicoNF2SoapOut" /> </wsdl:operation> </wsdl:portType> <wsdl:binding name="NfeStatusServico2Soap" type="tns:NfeStatusServico2Soap"> <soap:binding transport="http://schemas.xmlsoap.org/soap/http" /> <wsdl:operation name="nfeStatusServicoNF2"> <soap:operation soapAction="http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2/nfeStatusServicoNF2" style="document" /> <wsdl:input> <soap:body use="literal" /> <soap:header message="tns:nfeStatusServicoNF2nfeCabecMsg" part="nfeCabecMsg" use="literal" /> </wsdl:input> <wsdl:output> <soap:body use="literal" /> </wsdl:output> </wsdl:operation> </wsdl:binding> <wsdl:binding name="NfeStatusServico2Soap12" type="tns:NfeStatusServico2Soap"> <soap12:binding transport="http://schemas.xmlsoap.org/soap/http" /> <wsdl:operation name="nfeStatusServicoNF2"> <soap12:operation soapAction="http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2/nfeStatusServicoNF2" style="document" /> <wsdl:input> <soap12:body use="literal" /> <soap12:header message="tns:nfeStatusServicoNF2nfeCabecMsg" part="nfeCabecMsg" use="literal" /> </wsdl:input> <wsdl:output> <soap12:body use="literal" /> </wsdl:output> </wsdl:operation> </wsdl:binding> <wsdl:service name="NfeStatusServico2"> <wsdl:port name="NfeStatusServico2Soap" binding="tns:NfeStatusServico2Soap"> <soap:address location="https://hom.nfe.fazenda.gov.br/SCAN/NfeStatusServico2/NfeStatusServico2.asmx" /> </wsdl:port> <wsdl:port name="NfeStatusServico2Soap12" binding="tns:NfeStatusServico2Soap12"> <soap12:address location="https://hom.nfe.fazenda.gov.br/SCAN/NfeStatusServico2/NfeStatusServico2.asmx" /> </wsdl:port> </wsdl:service> </wsdl:definitions> Also, I've done some unit testes to show the problem. P.S.: Remember to set in the "VM Arguments" the following property. -Djava.endorsed.dirs="<JBOSS_HOME>\\lib\\endorsed" / ** * @author Fernando Rubbo */ public class TestWSDLWithTwoBindingsForOneInterface { private final String PATH_WSDL_FILE = "<PATH>\\NfeStatusServico2.wsdl"; private final String PATH_CHANGED_WSDL_FILE = "<PATH>\\NfeStatusServico21.wsdl"; private Service service; private Service serviceChanged; @Before public void setUp() throws IOException { URL wsdlLoc = new URL(PATH_WSDL_FILE); QName serviceName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2"); service = Service.create(wsdlLoc, serviceName); /* * After changing the line 72 of the WSDL. We have no more problems. * Original line * <wsdl:port name="NfeStatusServico2Soap" binding="tns:NfeStatusServico2Soap"> * Modified line * <wsdl:port name="NfeStatusServico2Soap1" binding="tns:NfeStatusServico2Soap"> */ wsdlLoc = new URL(PATH_CHANGED_WSDL_FILE); serviceName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2"); serviceChanged = Service.create(wsdlLoc, serviceName); } @Test public void testSOAP11OnOriginalWSDL() throws IOException { // asking for using the SOAP 1.1 binding. no problem... QName portName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2Soap"); Dispatch<Source> dispatch = service.createDispatch(portName, Source.class, Mode.MESSAGE); String bindingID = dispatch.getBinding().getBindingID(); Assert.assertEquals("http://schemas.xmlsoap.org/wsdl/soap/http";, bindingID); } @Test public void testSOAP12OnOriginalWSDL() throws IOException { // As you can see, the below code is asking for using the SOAP 1.2 binding. It should return a dispath for // the SOAP 1.2 binding, however it does not behaviour as expected... QName portName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2Soap12"); Dispatch<Source> dispatch = service.createDispatch(portName, Source.class, Mode.MESSAGE); String bindingID = dispatch.getBinding().getBindingID(); Assert.assertEquals("http://www.w3.org/2003/05/soap/bindings/HTTP/";, bindingID); } @Test public void testSOAP11OnChangedWSDL() throws IOException { // in this test we are asking for using the SOAP 1.1 binding but using the changed WSDL. Note that the order of the // the bindings property of WSDLBinding has changed. Therefore, it gets the wrong binding for the SOAP 1.1. QName portName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2Soap1"); Dispatch<Source> dispatch = serviceChanged.createDispatch(portName, Source.class, Mode.MESSAGE); String bindingID = dispatch.getBinding().getBindingID(); Assert.assertEquals("http://schemas.xmlsoap.org/wsdl/soap/http";, bindingID); } @Test public void testSOAP12OnChangedWSDL() throws IOException { // Since the order the bindings property of WSDLBinding has changed we have no problem asking for using the SOAP 1.2 binding. QName portName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2Soap12"); Dispatch<Source> dispatch = serviceChanged.createDispatch(portName, Source.class, Mode.MESSAGE); String bindingID = dispatch.getBinding().getBindingID(); Assert.assertEquals("http://www.w3.org/2003/05/soap/bindings/HTTP/";, bindingID); } } Hope this helps to fix the problem. Thanks in advance ---------------------------------------------------------------------------------------------------------------------------------------------------- I am implementing an interop test using WSDL organized as follows: <?xml version="1.0" encoding="UTF-8"?> <wsdl:definitions targetNamespace="http://www.wstf.org/docs/scenarios/sc002" xmlns:xs="http://www.w3.org/2001/XMLSchema" xmlns:soap="http://schemas.xmlsoap.org/wsdl/soap/" xmlns:soap12="http://schemas.xmlsoap.org/wsdl/soap12/" xmlns:wsdl="http://schemas.xmlsoap.org/wsdl/" xmlns:tns="http://www.wstf.org/docs/scenarios/sc002"> <wsdl:types> . . . </wsdl:types> <wsdl:message name="Begin"> <wsdl:part name="Begin" element="tns:Begin"/> . . . </wsdl:message> <wsdl:portType name="sc002Port"> <wsdl:operation name="Begin"> . . . </wsdl:portType> <wsdl:binding name="sc002SOAP11Binding" type="tns:sc002Port"> <soap:binding style="document" transport="http://schemas.xmlsoap.org/soap/http"/> <wsdl:operation name="Begin"> . . . </wsdl:binding> <wsdl:binding name="sc002SOAP12Binding" type="tns:sc002Port"> <soap12:binding style="document" transport="http://schemas.xmlsoap.org/soap/http"/> <wsdl:operation name="Begin"> . . . </wsdl:binding> <wsdl:service name="sc002Service"> <wsdl:port name="soap11port" binding="tns:sc002SOAP11Binding"> <soap:address location="http://www.wstf.org/sc002/sc002SOAP11"/> </wsdl:port> <wsdl:port name="soap12port" binding="tns:sc002SOAP12Binding"> <soap12:address location="http://www.wstf.org/sc002/sc002SOAP12"/> </wsdl:port> </wsdl:service> </wsdl:definitions> In other words, there is one port, two bindings and one service exporting the two bindings. I create a client proxy for the port named "soap11port" by invoking new Sc002Service().getPort(new QName("http://www.wstf.org/docs/scenarios/sc002";, "soap11port"), Sc002Port.class) The returned port uses a SOAP12 binding even though the WSDL labels it clearly as a 1.1 port. The problem arises because of an error in the construction of the ClientEndPointMetaData used as the endpointMetaData attached to the ClientProxy. This is constructed inside the ServiceDelegateImpl constructor by invoking JAXWSClientMetaDataBuilder.buildMetaData(). The error occurs in inherited method MetaDataBuilder.initEndpointBinding() when it tries to identify the binding associated with QName sc002:soap11port. The following 4 lines are where it goes wrong: WSDLDefinitions wsdlDefinitions = wsdlEndpoint.getWsdlService().getWsdlDefinitions(); WSDLInterface wsdlInterface = wsdlEndpoint.getInterface(); WSDLBinding wsdlBinding = wsdlDefinitions.getBindingByInterfaceName(wsdlInterface.getName()); String bindingType = wsdlBinding.getType(); The first two lines locate the port sc002 associated with the binding identified by sc002:soap11port. The 3rd line then reverse searches from the port to find a binding which binds it. Of course there are two of these and, mirabile dictu, the code even prints a warning about this. It then ignores the correct and returns the soap12 binding. Ironically before reaching this point the JAXWSClientMetaDataBuilder does this search correctly in method buildMetaDataInternal at lines 154/5 it executes this code WSDLBinding wsdlBinding = wsdlEndpoint.getWsdlService().getWsdlDefinitions().getBinding(bindingName); String bindingType = wsdlBinding.getType(); I assume the convoluted code in the inherited method is there for a reason but, then again, that's what overriding was invented for. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://jira.jboss.org/browse/JBWS-3131?page=com.atlassian.jira.plugin.sy... ] Fernando Rubbo updated JBWS-3131: --------------------------------- Description: I've cloned this issue because it was closed and I was not able to reopen it. The bug is on class org.jboss.ws.metadata.builder.MetaDataBuilder in initEndpointBinding(..) method. Below the explanation of the bug and the patch to fix the problem. protected void initEndpointBinding(WSDLEndpoint wsdlEndpoint, EndpointMetaData epMetaData) { WSDLDefinitions wsdlDefinitions = wsdlEndpoint.getWsdlService().getWsdlDefinitions(); //WSDLInterface wsdlInterface = wsdlEndpoint.getInterface(); //WSDLBinding wsdlBinding = wsdlDefinitions.getBindingByInterfaceName(wsdlInterface.getName()); /* BEGIN PATCH */ /* * Both commented lines above were replaced by the piece of code below. * * CAUSE: Once an interface may have more than one binding and once, at this point of the code, * we already know which binding to use (it is inside of the wsdlEndpoint), we can simple get the binding * of wsdlEndpoint INSTEAD of trying to guess which binding the user would like to use. * * NOTE: This bug is used to be intermittent because it depends on the name given for the binding. The bindings property * inside of WSDLBinding is a LinkedHashMap in which the key is the binding Name. Once the method wsdlDefinitions.getBindingByInterfaceName(..) * is called it iterates over bindings proprety and gets the last binding associated to the given interface. Note that we * MUST NOT get the last one. We know what binding we would like to use. * * KNOWN ISSUE: This issue was already submitted in the past (https://jira.jboss.org/browse/JBWS-2278?page=com.atlassian.jira.plugin.sy...) * but it seems that it was closed because the developer did not understood exactly what the problem was about. Moreover * it seems that up until now it was not fixed yet. Even in the newest version of the jbossws-core. */ WSDLBinding wsdlBinding; final QName binding = wsdlEndpoint.getBinding(); if(binding!=null){ wsdlBinding = wsdlDefinitions.getBinding(binding); }else{ // These lines are here only for backward compatibility. Supose a case which wsdlEndpoint does // not contain a binding associated. In this case, it will keep doing as it did before. WSDLInterface wsdlInterface = wsdlEndpoint.getInterface(); wsdlBinding = wsdlDefinitions.getBindingByInterfaceName(wsdlInterface.getName()); } /* END PATCH */ String bindingType = wsdlBinding.getType(); if (Constants.NS_SOAP11.equals(bindingType)) { epMetaData.setBindingId(Constants.SOAP11HTTP_BINDING); } else if (Constants.NS_SOAP12.equals(bindingType)) { epMetaData.setBindingId(Constants.SOAP12HTTP_BINDING); } } Also, I've done some unit testes to show the problem. P.S.1: I've attached two WSDL files that can be used to reproduce the this issue P.S.: Remember to set in the "VM Arguments" the following property. -Djava.endorsed.dirs="<JBOSS_HOME>\\lib\\endorsed" / ** * @author Fernando Rubbo */ public class TestWSDLWithTwoBindingsForOneInterface { private final String PATH_WSDL_FILE = "<PATH>\\NfeStatusServico2.wsdl"; private final String PATH_CHANGED_WSDL_FILE = "<PATH>\\NfeStatusServico21.wsdl"; private Service service; private Service serviceChanged; @Before public void setUp() throws IOException { URL wsdlLoc = new URL(PATH_WSDL_FILE); QName serviceName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2"); service = Service.create(wsdlLoc, serviceName); /* * After changing the line 72 of the WSDL as shown below. We have the order of the wsdlDefinitions.bindings property changed. * Original line * <wsdl:port name="NfeStatusServico2Soap" binding="tns:NfeStatusServico2Soap"> * Modified line * <wsdl:port name="NfeStatusServico2Soap1" binding="tns:NfeStatusServico2Soap"> */ wsdlLoc = new URL(PATH_CHANGED_WSDL_FILE); serviceName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2"); serviceChanged = Service.create(wsdlLoc, serviceName); } @Test public void testSOAP11OnOriginalWSDL() throws IOException { // asking for using the SOAP 1.1 binding. no problem... QName portName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2Soap"); Dispatch<Source> dispatch = service.createDispatch(portName, Source.class, Mode.MESSAGE); String bindingID = dispatch.getBinding().getBindingID(); Assert.assertEquals("http://schemas.xmlsoap.org/wsdl/soap/http";, bindingID); } @Test public void testSOAP12OnOriginalWSDL() throws IOException { // As you can see, the below code is asking for using the SOAP 1.2 binding. It should return a dispath for // the SOAP 1.2 binding, however it does not behaviour as expected... QName portName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2Soap12"); Dispatch<Source> dispatch = service.createDispatch(portName, Source.class, Mode.MESSAGE); String bindingID = dispatch.getBinding().getBindingID(); Assert.assertEquals("http://www.w3.org/2003/05/soap/bindings/HTTP/";, bindingID); } @Test public void testSOAP11OnChangedWSDL() throws IOException { // in this test we are asking for using the SOAP 1.1 binding but using the changed WSDL. Note that the order of the // the bindings property of WSDLBinding has changed. Therefore, it gets the wrong binding for the SOAP 1.1. QName portName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2Soap1"); Dispatch<Source> dispatch = serviceChanged.createDispatch(portName, Source.class, Mode.MESSAGE); String bindingID = dispatch.getBinding().getBindingID(); Assert.assertEquals("http://schemas.xmlsoap.org/wsdl/soap/http";, bindingID); } @Test public void testSOAP12OnChangedWSDL() throws IOException { // Since the order the bindings property of WSDLBinding has changed we have no problem asking for using the SOAP 1.2 binding. QName portName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2Soap12"); Dispatch<Source> dispatch = serviceChanged.createDispatch(portName, Source.class, Mode.MESSAGE); String bindingID = dispatch.getBinding().getBindingID(); Assert.assertEquals("http://www.w3.org/2003/05/soap/bindings/HTTP/";, bindingID); } } Hope this helps to fix the problem. Thanks in advance ---------------------------------------------------------------------------------------------------------------------------------------------------- I am implementing an interop test using WSDL organized as follows: <?xml version="1.0" encoding="UTF-8"?> <wsdl:definitions targetNamespace="http://www.wstf.org/docs/scenarios/sc002" xmlns:xs="http://www.w3.org/2001/XMLSchema" xmlns:soap="http://schemas.xmlsoap.org/wsdl/soap/" xmlns:soap12="http://schemas.xmlsoap.org/wsdl/soap12/" xmlns:wsdl="http://schemas.xmlsoap.org/wsdl/" xmlns:tns="http://www.wstf.org/docs/scenarios/sc002"> <wsdl:types> . . . </wsdl:types> <wsdl:message name="Begin"> <wsdl:part name="Begin" element="tns:Begin"/> . . . </wsdl:message> <wsdl:portType name="sc002Port"> <wsdl:operation name="Begin"> . . . </wsdl:portType> <wsdl:binding name="sc002SOAP11Binding" type="tns:sc002Port"> <soap:binding style="document" transport="http://schemas.xmlsoap.org/soap/http"/> <wsdl:operation name="Begin"> . . . </wsdl:binding> <wsdl:binding name="sc002SOAP12Binding" type="tns:sc002Port"> <soap12:binding style="document" transport="http://schemas.xmlsoap.org/soap/http"/> <wsdl:operation name="Begin"> . . . </wsdl:binding> <wsdl:service name="sc002Service"> <wsdl:port name="soap11port" binding="tns:sc002SOAP11Binding"> <soap:address location="http://www.wstf.org/sc002/sc002SOAP11"/> </wsdl:port> <wsdl:port name="soap12port" binding="tns:sc002SOAP12Binding"> <soap12:address location="http://www.wstf.org/sc002/sc002SOAP12"/> </wsdl:port> </wsdl:service> </wsdl:definitions> In other words, there is one port, two bindings and one service exporting the two bindings. I create a client proxy for the port named "soap11port" by invoking new Sc002Service().getPort(new QName("http://www.wstf.org/docs/scenarios/sc002";, "soap11port"), Sc002Port.class) The returned port uses a SOAP12 binding even though the WSDL labels it clearly as a 1.1 port. The problem arises because of an error in the construction of the ClientEndPointMetaData used as the endpointMetaData attached to the ClientProxy. This is constructed inside the ServiceDelegateImpl constructor by invoking JAXWSClientMetaDataBuilder.buildMetaData(). The error occurs in inherited method MetaDataBuilder.initEndpointBinding() when it tries to identify the binding associated with QName sc002:soap11port. The following 4 lines are where it goes wrong: WSDLDefinitions wsdlDefinitions = wsdlEndpoint.getWsdlService().getWsdlDefinitions(); WSDLInterface wsdlInterface = wsdlEndpoint.getInterface(); WSDLBinding wsdlBinding = wsdlDefinitions.getBindingByInterfaceName(wsdlInterface.getName()); String bindingType = wsdlBinding.getType(); The first two lines locate the port sc002 associated with the binding identified by sc002:soap11port. The 3rd line then reverse searches from the port to find a binding which binds it. Of course there are two of these and, mirabile dictu, the code even prints a warning about this. It then ignores the correct and returns the soap12 binding. Ironically before reaching this point the JAXWSClientMetaDataBuilder does this search correctly in method buildMetaDataInternal at lines 154/5 it executes this code WSDLBinding wsdlBinding = wsdlEndpoint.getWsdlService().getWsdlDefinitions().getBinding(bindingName); String bindingType = wsdlBinding.getType(); I assume the convoluted code in the inherited method is there for a reason but, then again, that's what overriding was invented for. was: I've cloned this issue because it was closed and I was not able to reopen it. The bug is on class org.jboss.ws.metadata.builder.MetaDataBuilder in initEndpointBinding(..) method. Below the explanation of the bug and the patch to fix the problem. protected void initEndpointBinding(WSDLEndpoint wsdlEndpoint, EndpointMetaData epMetaData) { WSDLDefinitions wsdlDefinitions = wsdlEndpoint.getWsdlService().getWsdlDefinitions(); //WSDLInterface wsdlInterface = wsdlEndpoint.getInterface(); //WSDLBinding wsdlBinding = wsdlDefinitions.getBindingByInterfaceName(wsdlInterface.getName()); /* BEGIN PATCH */ /* * Both commented lines above were replaced by the piece of code below. * * CAUSE: Once an interface may have more than one binding and once, at this point of the code, * we already know which binding to use (it is inside of the wsdlEndpoint), we can simple get the binding * of wsdlEndpoint INSTEAD of trying to guess which binding the user would like to use. * * NOTE: This bug is used to be intermittent because it depends on the name given for the binding. The bindings property * inside of WSDLBinding is a LinkedHashMap in which the key is the binding Name. Once the method wsdlDefinitions.getBindingByInterfaceName(..) * is called it iterates over bindings proprety and gets the last binding associated to the given interface. Note that we * MUST NOT get the last one. We know what binding we would like to use. * * KNOWN ISSUE: This issue was already submitted in the past (https://jira.jboss.org/browse/JBWS-2278?page=com.atlassian.jira.plugin.sy...) * but it seems that it was closed because the developer did not understood exactly what the problem was about. Moreover * it seems that up until now it was not fixed yet. Even in the newest version of the jbossws-core. */ WSDLBinding wsdlBinding; final QName binding = wsdlEndpoint.getBinding(); if(binding!=null){ wsdlBinding = wsdlDefinitions.getBinding(binding); }else{ // These lines are here only for backward compatibility. Supose a case which wsdlEndpoint does // not contain a binding associated. In this case, it will keep doing as it did before. WSDLInterface wsdlInterface = wsdlEndpoint.getInterface(); wsdlBinding = wsdlDefinitions.getBindingByInterfaceName(wsdlInterface.getName()); } /* END PATCH */ String bindingType = wsdlBinding.getType(); if (Constants.NS_SOAP11.equals(bindingType)) { epMetaData.setBindingId(Constants.SOAP11HTTP_BINDING); } else if (Constants.NS_SOAP12.equals(bindingType)) { epMetaData.setBindingId(Constants.SOAP12HTTP_BINDING); } } The WSDL file which can be used to reproduce the this issue: <?xml version="1.0" encoding="utf-8"?> <wsdl:definitions xmlns:soap="http://schemas.xmlsoap.org/wsdl/soap/" xmlns:tm="http://microsoft.com/wsdl/mime/textMatching/" xmlns:soapenc="http://schemas.xmlsoap.org/soap/encoding/" xmlns:mime="http://schemas.xmlsoap.org/wsdl/mime/" xmlns:tns="http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2" xmlns:s="http://www.w3.org/2001/XMLSchema" xmlns:soap12="http://schemas.xmlsoap.org/wsdl/soap12/" xmlns:http="http://schemas.xmlsoap.org/wsdl/http/" targetNamespace="http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2" xmlns:wsdl="http://schemas.xmlsoap.org/wsdl/"> <wsdl:types> <s:schema elementFormDefault="qualified" targetNamespace="http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2"> <s:element name="nfeDadosMsg"> <s:complexType mixed="true"> <s:sequence> <s:any /> </s:sequence> </s:complexType> </s:element> <s:element name="nfeStatusServicoNF2Result"> <s:complexType mixed="true"> <s:sequence> <s:any /> </s:sequence> </s:complexType> </s:element> <s:element name="nfeCabecMsg" type="tns:nfeCabecMsg" /> <s:complexType name="nfeCabecMsg"> <s:sequence> <s:element minOccurs="0" maxOccurs="1" name="versaoDados" type="s:string" /> <s:element minOccurs="0" maxOccurs="1" name="cUF" type="s:string" /> </s:sequence> <s:anyAttribute /> </s:complexType> </s:schema> </wsdl:types> <wsdl:message name="nfeStatusServicoNF2SoapIn"> <wsdl:part name="nfeDadosMsg" element="tns:nfeDadosMsg" /> </wsdl:message> <wsdl:message name="nfeStatusServicoNF2SoapOut"> <wsdl:part name="nfeStatusServicoNF2Result" element="tns:nfeStatusServicoNF2Result" /> </wsdl:message> <wsdl:message name="nfeStatusServicoNF2nfeCabecMsg"> <wsdl:part name="nfeCabecMsg" element="tns:nfeCabecMsg" /> </wsdl:message> <wsdl:portType name="NfeStatusServico2Soap"> <wsdl:operation name="nfeStatusServicoNF2"> <wsdl:documentation xmlns:wsdl="http://schemas.xmlsoap.org/wsdl/">Serviço destinado à consulta do status do serviço prestado pela Sefaz Virtual do Ambiente Nacional</wsdl:documentation> <wsdl:input message="tns:nfeStatusServicoNF2SoapIn" /> <wsdl:output message="tns:nfeStatusServicoNF2SoapOut" /> </wsdl:operation> </wsdl:portType> <wsdl:binding name="NfeStatusServico2Soap" type="tns:NfeStatusServico2Soap"> <soap:binding transport="http://schemas.xmlsoap.org/soap/http" /> <wsdl:operation name="nfeStatusServicoNF2"> <soap:operation soapAction="http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2/nfeStatusServicoNF2" style="document" /> <wsdl:input> <soap:body use="literal" /> <soap:header message="tns:nfeStatusServicoNF2nfeCabecMsg" part="nfeCabecMsg" use="literal" /> </wsdl:input> <wsdl:output> <soap:body use="literal" /> </wsdl:output> </wsdl:operation> </wsdl:binding> <wsdl:binding name="NfeStatusServico2Soap12" type="tns:NfeStatusServico2Soap"> <soap12:binding transport="http://schemas.xmlsoap.org/soap/http" /> <wsdl:operation name="nfeStatusServicoNF2"> <soap12:operation soapAction="http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2/nfeStatusServicoNF2" style="document" /> <wsdl:input> <soap12:body use="literal" /> <soap12:header message="tns:nfeStatusServicoNF2nfeCabecMsg" part="nfeCabecMsg" use="literal" /> </wsdl:input> <wsdl:output> <soap12:body use="literal" /> </wsdl:output> </wsdl:operation> </wsdl:binding> <wsdl:service name="NfeStatusServico2"> <wsdl:port name="NfeStatusServico2Soap" binding="tns:NfeStatusServico2Soap"> <soap:address location="https://hom.nfe.fazenda.gov.br/SCAN/NfeStatusServico2/NfeStatusServico2.asmx" /> </wsdl:port> <wsdl:port name="NfeStatusServico2Soap12" binding="tns:NfeStatusServico2Soap12"> <soap12:address location="https://hom.nfe.fazenda.gov.br/SCAN/NfeStatusServico2/NfeStatusServico2.asmx" /> </wsdl:port> </wsdl:service> </wsdl:definitions> Also, I've done some unit testes to show the problem. P.S.: Remember to set in the "VM Arguments" the following property. -Djava.endorsed.dirs="<JBOSS_HOME>\\lib\\endorsed" / ** * @author Fernando Rubbo */ public class TestWSDLWithTwoBindingsForOneInterface { private final String PATH_WSDL_FILE = "<PATH>\\NfeStatusServico2.wsdl"; private final String PATH_CHANGED_WSDL_FILE = "<PATH>\\NfeStatusServico21.wsdl"; private Service service; private Service serviceChanged; @Before public void setUp() throws IOException { URL wsdlLoc = new URL(PATH_WSDL_FILE); QName serviceName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2"); service = Service.create(wsdlLoc, serviceName); /* * After changing the line 72 of the WSDL as shown below. We have the order of the wsdlDefinitions.bindings property changed. * Original line * <wsdl:port name="NfeStatusServico2Soap" binding="tns:NfeStatusServico2Soap"> * Modified line * <wsdl:port name="NfeStatusServico2Soap1" binding="tns:NfeStatusServico2Soap"> */ wsdlLoc = new URL(PATH_CHANGED_WSDL_FILE); serviceName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2"); serviceChanged = Service.create(wsdlLoc, serviceName); } @Test public void testSOAP11OnOriginalWSDL() throws IOException { // asking for using the SOAP 1.1 binding. no problem... QName portName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2Soap"); Dispatch<Source> dispatch = service.createDispatch(portName, Source.class, Mode.MESSAGE); String bindingID = dispatch.getBinding().getBindingID(); Assert.assertEquals("http://schemas.xmlsoap.org/wsdl/soap/http";, bindingID); } @Test public void testSOAP12OnOriginalWSDL() throws IOException { // As you can see, the below code is asking for using the SOAP 1.2 binding. It should return a dispath for // the SOAP 1.2 binding, however it does not behaviour as expected... QName portName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2Soap12"); Dispatch<Source> dispatch = service.createDispatch(portName, Source.class, Mode.MESSAGE); String bindingID = dispatch.getBinding().getBindingID(); Assert.assertEquals("http://www.w3.org/2003/05/soap/bindings/HTTP/";, bindingID); } @Test public void testSOAP11OnChangedWSDL() throws IOException { // in this test we are asking for using the SOAP 1.1 binding but using the changed WSDL. Note that the order of the // the bindings property of WSDLBinding has changed. Therefore, it gets the wrong binding for the SOAP 1.1. QName portName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2Soap1"); Dispatch<Source> dispatch = serviceChanged.createDispatch(portName, Source.class, Mode.MESSAGE); String bindingID = dispatch.getBinding().getBindingID(); Assert.assertEquals("http://schemas.xmlsoap.org/wsdl/soap/http";, bindingID); } @Test public void testSOAP12OnChangedWSDL() throws IOException { // Since the order the bindings property of WSDLBinding has changed we have no problem asking for using the SOAP 1.2 binding. QName portName = new QName("http://www.portalfiscal.inf.br/nfe/wsdl/NfeStatusServico2","NfeStatusServico2Soap12"); Dispatch<Source> dispatch = serviceChanged.createDispatch(portName, Source.class, Mode.MESSAGE); String bindingID = dispatch.getBinding().getBindingID(); Assert.assertEquals("http://www.w3.org/2003/05/soap/bindings/HTTP/";, bindingID); } } Hope this helps to fix the problem. Thanks in advance ---------------------------------------------------------------------------------------------------------------------------------------------------- I am implementing an interop test using WSDL organized as follows: <?xml version="1.0" encoding="UTF-8"?> <wsdl:definitions targetNamespace="http://www.wstf.org/docs/scenarios/sc002" xmlns:xs="http://www.w3.org/2001/XMLSchema" xmlns:soap="http://schemas.xmlsoap.org/wsdl/soap/" xmlns:soap12="http://schemas.xmlsoap.org/wsdl/soap12/" xmlns:wsdl="http://schemas.xmlsoap.org/wsdl/" xmlns:tns="http://www.wstf.org/docs/scenarios/sc002"> <wsdl:types> . . . </wsdl:types> <wsdl:message name="Begin"> <wsdl:part name="Begin" element="tns:Begin"/> . . . </wsdl:message> <wsdl:portType name="sc002Port"> <wsdl:operation name="Begin"> . . . </wsdl:portType> <wsdl:binding name="sc002SOAP11Binding" type="tns:sc002Port"> <soap:binding style="document" transport="http://schemas.xmlsoap.org/soap/http"/> <wsdl:operation name="Begin"> . . . </wsdl:binding> <wsdl:binding name="sc002SOAP12Binding" type="tns:sc002Port"> <soap12:binding style="document" transport="http://schemas.xmlsoap.org/soap/http"/> <wsdl:operation name="Begin"> . . . </wsdl:binding> <wsdl:service name="sc002Service"> <wsdl:port name="soap11port" binding="tns:sc002SOAP11Binding"> <soap:address location="http://www.wstf.org/sc002/sc002SOAP11"/> </wsdl:port> <wsdl:port name="soap12port" binding="tns:sc002SOAP12Binding"> <soap12:address location="http://www.wstf.org/sc002/sc002SOAP12"/> </wsdl:port> </wsdl:service> </wsdl:definitions> In other words, there is one port, two bindings and one service exporting the two bindings. I create a client proxy for the port named "soap11port" by invoking new Sc002Service().getPort(new QName("http://www.wstf.org/docs/scenarios/sc002";, "soap11port"), Sc002Port.class) The returned port uses a SOAP12 binding even though the WSDL labels it clearly as a 1.1 port. The problem arises because of an error in the construction of the ClientEndPointMetaData used as the endpointMetaData attached to the ClientProxy. This is constructed inside the ServiceDelegateImpl constructor by invoking JAXWSClientMetaDataBuilder.buildMetaData(). The error occurs in inherited method MetaDataBuilder.initEndpointBinding() when it tries to identify the binding associated with QName sc002:soap11port. The following 4 lines are where it goes wrong: WSDLDefinitions wsdlDefinitions = wsdlEndpoint.getWsdlService().getWsdlDefinitions(); WSDLInterface wsdlInterface = wsdlEndpoint.getInterface(); WSDLBinding wsdlBinding = wsdlDefinitions.getBindingByInterfaceName(wsdlInterface.getName()); String bindingType = wsdlBinding.getType(); The first two lines locate the port sc002 associated with the binding identified by sc002:soap11port. The 3rd line then reverse searches from the port to find a binding which binds it. Of course there are two of these and, mirabile dictu, the code even prints a warning about this. It then ignores the correct and returns the soap12 binding. Ironically before reaching this point the JAXWSClientMetaDataBuilder does this search correctly in method buildMetaDataInternal at lines 154/5 it executes this code WSDLBinding wsdlBinding = wsdlEndpoint.getWsdlService().getWsdlDefinitions().getBinding(bindingName); String bindingType = wsdlBinding.getType(); I assume the convoluted code in the inherited method is there for a reason but, then again, that's what overriding was invented for. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://jira.jboss.org/browse/JBWS-3131?page=com.atlassian.jira.plugin.sy... ] Fernando Rubbo commented on JBWS-3131: -------------------------------------- Hi Alessio, First of all, thanks for your rapid response. About your question. I've run the tests against the version jbossws-native-2.0.1.SP2_CP06 (build=200905251649) because it is the version we are currently using. However, if you open the jbossws-native-core.jar of version 3.3.1.GA it is possible to see that the piece of code with problem is still there (the only difference is that someone did a minor refactoring in the MetaDataBuilder.initEndpointBinding(..) method). About the trunk. I've not tried to run the tests against the trunk. If I said that at any moment, sorry.. not my intention. Thanks, Fernando Rubbo -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://jira.jboss.org/browse/JBWS-3131?page=com.atlassian.jira.plugin.sy... ] Fernando Rubbo commented on JBWS-3131: -------------------------------------- It is true. I've not went deep enough in the code of JBossWS-3.0.4.GA to see that the two *ClientMetaDataBuilder classes were using the new extracted method. That solves the problem. Thank you very much for your help. Anyway, I'll open a customer case with jboss.com. Thanks again -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira