You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cxf.apache.org by "Gladwin (JIRA)" <ji...@apache.org> on 2011/02/25 04:20:38 UTC
[jira] Created: (CXF-3363) Use
MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL) instead of
MessageFactory.newInstance()
Use MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL) instead of MessageFactory.newInstance()
-------------------------------------------------------------------------------------------------------
Key: CXF-3363
URL: https://issues.apache.org/jira/browse/CXF-3363
Project: CXF
Issue Type: Improvement
Components: JAX-WS Runtime, Soap Binding
Affects Versions: 2.3.2
Reporter: Gladwin
When it is known that SOAP version used is 1.1 [ {{instanceof Soap11}} ], CXF should use {{MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL)}} instead of {{MessageFactory.newInstance()}}
Further more by default when version is neither of 1.1 or 1.2, then instead of returning {{null}}, CXF should return {{MessageFactory.newInstance(SOAPConstants.DEFAULT_SOAP_PROTOCOL)}} or {{MessageFactory.newInstance(SOAPConstants.DYNAMIC_SOAP_PROTOCOL)}}
*Given below is an example of cxf code where this refactoring can be done*
{code:title=org.apache.cxf.jaxws.binding.soap.SOAPBindingImpl.java|borderStyle=solid}
public MessageFactory getMessageFactory()
{
if (this.soapBinding instanceof SoapBindingInfo) {
SoapBindingInfo bindingInfo = (SoapBindingInfo) this.soapBinding;
try {
if (bindingInfo.getSoapVersion() instanceof Soap11) {
return {color:red}MessageFactory.newInstance(){color:red};
} else if (bindingInfo.getSoapVersion() instanceof Soap12) {
return MessageFactory.newInstance(SOAPConstants.SOAP_1_2_PROTOCOL);
}
} catch (SOAPException e) {
throw new WebServiceException(BUNDLE.getString("SAAJ_FACTORY_ERR"), e);
}
}
return {color:red}null{color:red};
}
{code}
*Advantages*
* In future, {{MessageFactory.newInstance()}} may not return object instance of the default implementation that does not supports "SOAP 1.1 Protocol"
* Some JEE Appserver implementations of {{MessageFactory.newInstance()}}, currently do not return factory that supports "SOAP 1.1 Protocol"
** Weblogic 10.3.x [ {{weblogic.webservice.core.soap.MessageFactoryImpl}} ] does not support SAAJ "SOAP 1.1 Protocol" -> "{{java.lang.UnsupportedOperationException: This class does not support SAAJ 1.1}}"
* This could help ease use of CXF libraries in JEE applications even if Application Server has it's own SOAP/SAAJ implementation. By above method, CXF will always get {{MessageFactory}} that supports required protocol.
*Related Isues*
* https://issues.apache.org/jira/browse/CXF-976
* https://issues.apache.org/jira/browse/CXF-1750
* https://issues.apache.org/jira/browse/CXF-3307
--
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Updated: (CXF-3363) Use
MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL) instead of
MessageFactory.newInstance()
Posted by "Gladwin (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CXF-3363?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Gladwin updated CXF-3363:
-------------------------
Description:
When it is known that SOAP version used is 1.1 [ {{instanceof Soap11}} ], CXF should use {{MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL)}} instead of {{MessageFactory.newInstance()}}
Further more by default when version is neither of 1.1 or 1.2, then instead of returning {{null}}, CXF should return {{MessageFactory.newInstance(SOAPConstants.DEFAULT_SOAP_PROTOCOL)}} or {{MessageFactory.newInstance(SOAPConstants.DYNAMIC_SOAP_PROTOCOL)}}
*Given below is an example of cxf code where this refactoring can be done*
{code:title=org.apache.cxf.jaxws.binding.soap.SOAPBindingImpl.java|borderStyle=solid}
public MessageFactory getMessageFactory()
{
if (this.soapBinding instanceof SoapBindingInfo) {
SoapBindingInfo bindingInfo = (SoapBindingInfo) this.soapBinding;
try {
if (bindingInfo.getSoapVersion() instanceof Soap11) {
return MessageFactory.newInstance();
} else if (bindingInfo.getSoapVersion() instanceof Soap12) {
return MessageFactory.newInstance(SOAPConstants.SOAP_1_2_PROTOCOL);
}
} catch (SOAPException e) {
throw new WebServiceException(BUNDLE.getString("SAAJ_FACTORY_ERR"), e);
}
}
return null;
}
{code}
*Advantages*
* In future, {{MessageFactory.newInstance()}} may not return object instance of the default implementation that does not supports "SOAP 1.1 Protocol"
* Some JEE Appserver implementations of {{MessageFactory.newInstance()}}, currently do not return factory that supports "SOAP 1.1 Protocol"
** Weblogic 10.3.x [ {{weblogic.webservice.core.soap.MessageFactoryImpl}} ] does not support SAAJ "SOAP 1.1 Protocol" -> "{{java.lang.UnsupportedOperationException: This class does not support SAAJ 1.1}}"
* This could help ease use of CXF libraries in JEE applications even if Application Server has it's own SOAP/SAAJ implementation. By above method, CXF will always get {{MessageFactory}} that supports required protocol.
*Related Isues*
* https://issues.apache.org/jira/browse/CXF-976
* https://issues.apache.org/jira/browse/CXF-1750
* https://issues.apache.org/jira/browse/CXF-3307
was:
When it is known that SOAP version used is 1.1 [ {{instanceof Soap11}} ], CXF should use {{MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL)}} instead of {{MessageFactory.newInstance()}}
Further more by default when version is neither of 1.1 or 1.2, then instead of returning {{null}}, CXF should return {{MessageFactory.newInstance(SOAPConstants.DEFAULT_SOAP_PROTOCOL)}} or {{MessageFactory.newInstance(SOAPConstants.DYNAMIC_SOAP_PROTOCOL)}}
*Given below is an example of cxf code where this refactoring can be done*
{code:title=org.apache.cxf.jaxws.binding.soap.SOAPBindingImpl.java|borderStyle=solid}
public MessageFactory getMessageFactory()
{
if (this.soapBinding instanceof SoapBindingInfo) {
SoapBindingInfo bindingInfo = (SoapBindingInfo) this.soapBinding;
try {
if (bindingInfo.getSoapVersion() instanceof Soap11) {
return {color:red}MessageFactory.newInstance(){color:red};
} else if (bindingInfo.getSoapVersion() instanceof Soap12) {
return MessageFactory.newInstance(SOAPConstants.SOAP_1_2_PROTOCOL);
}
} catch (SOAPException e) {
throw new WebServiceException(BUNDLE.getString("SAAJ_FACTORY_ERR"), e);
}
}
return {color:red}null{color:red};
}
{code}
*Advantages*
* In future, {{MessageFactory.newInstance()}} may not return object instance of the default implementation that does not supports "SOAP 1.1 Protocol"
* Some JEE Appserver implementations of {{MessageFactory.newInstance()}}, currently do not return factory that supports "SOAP 1.1 Protocol"
** Weblogic 10.3.x [ {{weblogic.webservice.core.soap.MessageFactoryImpl}} ] does not support SAAJ "SOAP 1.1 Protocol" -> "{{java.lang.UnsupportedOperationException: This class does not support SAAJ 1.1}}"
* This could help ease use of CXF libraries in JEE applications even if Application Server has it's own SOAP/SAAJ implementation. By above method, CXF will always get {{MessageFactory}} that supports required protocol.
*Related Isues*
* https://issues.apache.org/jira/browse/CXF-976
* https://issues.apache.org/jira/browse/CXF-1750
* https://issues.apache.org/jira/browse/CXF-3307
> Use MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL) instead of MessageFactory.newInstance()
> -------------------------------------------------------------------------------------------------------
>
> Key: CXF-3363
> URL: https://issues.apache.org/jira/browse/CXF-3363
> Project: CXF
> Issue Type: Improvement
> Components: JAX-WS Runtime, Soap Binding
> Affects Versions: 2.3.2
> Reporter: Gladwin
>
> When it is known that SOAP version used is 1.1 [ {{instanceof Soap11}} ], CXF should use {{MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL)}} instead of {{MessageFactory.newInstance()}}
> Further more by default when version is neither of 1.1 or 1.2, then instead of returning {{null}}, CXF should return {{MessageFactory.newInstance(SOAPConstants.DEFAULT_SOAP_PROTOCOL)}} or {{MessageFactory.newInstance(SOAPConstants.DYNAMIC_SOAP_PROTOCOL)}}
> *Given below is an example of cxf code where this refactoring can be done*
> {code:title=org.apache.cxf.jaxws.binding.soap.SOAPBindingImpl.java|borderStyle=solid}
> public MessageFactory getMessageFactory()
> {
> if (this.soapBinding instanceof SoapBindingInfo) {
> SoapBindingInfo bindingInfo = (SoapBindingInfo) this.soapBinding;
> try {
> if (bindingInfo.getSoapVersion() instanceof Soap11) {
> return MessageFactory.newInstance();
> } else if (bindingInfo.getSoapVersion() instanceof Soap12) {
> return MessageFactory.newInstance(SOAPConstants.SOAP_1_2_PROTOCOL);
> }
> } catch (SOAPException e) {
> throw new WebServiceException(BUNDLE.getString("SAAJ_FACTORY_ERR"), e);
> }
> }
> return null;
> }
> {code}
> *Advantages*
> * In future, {{MessageFactory.newInstance()}} may not return object instance of the default implementation that does not supports "SOAP 1.1 Protocol"
> * Some JEE Appserver implementations of {{MessageFactory.newInstance()}}, currently do not return factory that supports "SOAP 1.1 Protocol"
> ** Weblogic 10.3.x [ {{weblogic.webservice.core.soap.MessageFactoryImpl}} ] does not support SAAJ "SOAP 1.1 Protocol" -> "{{java.lang.UnsupportedOperationException: This class does not support SAAJ 1.1}}"
> * This could help ease use of CXF libraries in JEE applications even if Application Server has it's own SOAP/SAAJ implementation. By above method, CXF will always get {{MessageFactory}} that supports required protocol.
> *Related Isues*
> * https://issues.apache.org/jira/browse/CXF-976
> * https://issues.apache.org/jira/browse/CXF-1750
> * https://issues.apache.org/jira/browse/CXF-3307
--
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Issue Comment Edited] (CXF-3363) Use
MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL) instead of
MessageFactory.newInstance()
Posted by "Gladwin (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CXF-3363?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13019139#comment-13019139 ]
Gladwin edited comment on CXF-3363 at 4/13/11 12:37 AM:
--------------------------------------------------------
Tested and verified this fix on Weblogic 11g with CXF - 2.3.4
was (Author: gburboz):
Tested and verified this fix on Weblogic 11g with CXF-2.3.4
> Use MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL) instead of MessageFactory.newInstance()
> -------------------------------------------------------------------------------------------------------
>
> Key: CXF-3363
> URL: https://issues.apache.org/jira/browse/CXF-3363
> Project: CXF
> Issue Type: Improvement
> Components: JAX-WS Runtime, Soap Binding
> Affects Versions: 2.3.2
> Reporter: Gladwin
> Assignee: Daniel Kulp
> Fix For: 2.4, 2.3.4
>
>
> When it is known that SOAP version used is 1.1 [ {{instanceof Soap11}} ], CXF should use {{MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL)}} instead of {{MessageFactory.newInstance()}}
> Further more by default when version is neither of 1.1 or 1.2, then instead of returning {{null}}, CXF should return {{MessageFactory.newInstance(SOAPConstants.DEFAULT_SOAP_PROTOCOL)}} or {{MessageFactory.newInstance(SOAPConstants.DYNAMIC_SOAP_PROTOCOL)}}
> *Given below is an example of cxf code where this refactoring can be done*
> {code:title=org.apache.cxf.jaxws.binding.soap.SOAPBindingImpl.java|borderStyle=solid}
> public MessageFactory getMessageFactory()
> {
> if (this.soapBinding instanceof SoapBindingInfo) {
> SoapBindingInfo bindingInfo = (SoapBindingInfo) this.soapBinding;
> try {
> if (bindingInfo.getSoapVersion() instanceof Soap11) {
> return MessageFactory.newInstance();
> } else if (bindingInfo.getSoapVersion() instanceof Soap12) {
> return MessageFactory.newInstance(SOAPConstants.SOAP_1_2_PROTOCOL);
> }
> } catch (SOAPException e) {
> throw new WebServiceException(BUNDLE.getString("SAAJ_FACTORY_ERR"), e);
> }
> }
> return null;
> }
> {code}
> *Advantages*
> * In future, {{MessageFactory.newInstance()}} may not return object instance of the default implementation that does not supports "SOAP 1.1 Protocol"
> * Some JEE Appserver implementations of {{MessageFactory.newInstance()}}, currently do not return factory that supports "SOAP 1.1 Protocol"
> ** Weblogic 10.3.x [ {{weblogic.webservice.core.soap.MessageFactoryImpl}} ] does not support SAAJ "SOAP 1.1 Protocol" -> "{{java.lang.UnsupportedOperationException: This class does not support SAAJ 1.1}}"
> * This could help ease use of CXF libraries in JEE applications even if Application Server has it's own SOAP/SAAJ implementation. By above method, CXF will always get {{MessageFactory}} that supports required protocol.
> *Related Isues*
> * https://issues.apache.org/jira/browse/CXF-976
> * https://issues.apache.org/jira/browse/CXF-1750
> * https://issues.apache.org/jira/browse/CXF-3307
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Commented: (CXF-3363) Use
MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL) instead of
MessageFactory.newInstance()
Posted by "Gladwin (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CXF-3363?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13005898#comment-13005898 ]
Gladwin commented on CXF-3363:
------------------------------
*Refactor SAAJFactoryResolver*
* To handle Throwable similar to how it is handled for SOAPConstants.SOAP_1_1_PROTOCOL
* Rename method newInstanceCxfMessageFactory(...) to newInstanceCxfSAAJFactory(...)
{code:title=SAAJFactoryResolver.java|borderStyle=solid}
public final class SAAJFactoryResolver {
...
public static MessageFactory createMessageFactory(SoapVersion version) throws SOAPException {
...
if (messageFactoryClassName != null) {
messageFactory = newInstanceCxfSAAJFactory(messageFactoryClassName, MessageFactory.class);
} else if (version instanceof Soap11) {
...
} else if (version instanceof Soap12) {
try {
messageFactory = MessageFactory.newInstance(SOAPConstants.SOAP_1_2_PROTOCOL);
} catch (Throwable t) {
messageFactory = MessageFactory.newInstance();
}
} else {
...
}
public static SOAPFactory createSOAPFactory(SoapVersion version) throws SOAPException {
...
if (soapFactoryClassName != null) {
soapFactory = newInstanceCxfSAAJFactory(soapFactoryClassName, SOAPFactory.class);
} else if (version instanceof Soap11) {
...
} else if (version instanceof Soap12) {
try {
soapFactory = SOAPFactory.newInstance(SOAPConstants.SOAP_1_2_PROTOCOL);
} catch (Throwable t) {
soapFactory = SOAPFactory.newInstance();
}
soapFactory = SOAPFactory.newInstance(SOAPConstants.SOAP_1_2_PROTOCOL);
} else {
...
}
private static <T> T newInstanceCxfSAAJFactory(String factoryName, Class<T> cls)
throws SOAPException {
...
}
}
{code}
> Use MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL) instead of MessageFactory.newInstance()
> -------------------------------------------------------------------------------------------------------
>
> Key: CXF-3363
> URL: https://issues.apache.org/jira/browse/CXF-3363
> Project: CXF
> Issue Type: Improvement
> Components: JAX-WS Runtime, Soap Binding
> Affects Versions: 2.3.2
> Reporter: Gladwin
> Assignee: Daniel Kulp
> Fix For: 2.4, 2.3.4
>
>
> When it is known that SOAP version used is 1.1 [ {{instanceof Soap11}} ], CXF should use {{MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL)}} instead of {{MessageFactory.newInstance()}}
> Further more by default when version is neither of 1.1 or 1.2, then instead of returning {{null}}, CXF should return {{MessageFactory.newInstance(SOAPConstants.DEFAULT_SOAP_PROTOCOL)}} or {{MessageFactory.newInstance(SOAPConstants.DYNAMIC_SOAP_PROTOCOL)}}
> *Given below is an example of cxf code where this refactoring can be done*
> {code:title=org.apache.cxf.jaxws.binding.soap.SOAPBindingImpl.java|borderStyle=solid}
> public MessageFactory getMessageFactory()
> {
> if (this.soapBinding instanceof SoapBindingInfo) {
> SoapBindingInfo bindingInfo = (SoapBindingInfo) this.soapBinding;
> try {
> if (bindingInfo.getSoapVersion() instanceof Soap11) {
> return MessageFactory.newInstance();
> } else if (bindingInfo.getSoapVersion() instanceof Soap12) {
> return MessageFactory.newInstance(SOAPConstants.SOAP_1_2_PROTOCOL);
> }
> } catch (SOAPException e) {
> throw new WebServiceException(BUNDLE.getString("SAAJ_FACTORY_ERR"), e);
> }
> }
> return null;
> }
> {code}
> *Advantages*
> * In future, {{MessageFactory.newInstance()}} may not return object instance of the default implementation that does not supports "SOAP 1.1 Protocol"
> * Some JEE Appserver implementations of {{MessageFactory.newInstance()}}, currently do not return factory that supports "SOAP 1.1 Protocol"
> ** Weblogic 10.3.x [ {{weblogic.webservice.core.soap.MessageFactoryImpl}} ] does not support SAAJ "SOAP 1.1 Protocol" -> "{{java.lang.UnsupportedOperationException: This class does not support SAAJ 1.1}}"
> * This could help ease use of CXF libraries in JEE applications even if Application Server has it's own SOAP/SAAJ implementation. By above method, CXF will always get {{MessageFactory}} that supports required protocol.
> *Related Isues*
> * https://issues.apache.org/jira/browse/CXF-976
> * https://issues.apache.org/jira/browse/CXF-1750
> * https://issues.apache.org/jira/browse/CXF-3307
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (CXF-3363) Use
MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL) instead of
MessageFactory.newInstance()
Posted by "Gladwin (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CXF-3363?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13019139#comment-13019139 ]
Gladwin commented on CXF-3363:
------------------------------
Tested and verified this fix on Weblogic 11g with CXF-2.3.4
> Use MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL) instead of MessageFactory.newInstance()
> -------------------------------------------------------------------------------------------------------
>
> Key: CXF-3363
> URL: https://issues.apache.org/jira/browse/CXF-3363
> Project: CXF
> Issue Type: Improvement
> Components: JAX-WS Runtime, Soap Binding
> Affects Versions: 2.3.2
> Reporter: Gladwin
> Assignee: Daniel Kulp
> Fix For: 2.4, 2.3.4
>
>
> When it is known that SOAP version used is 1.1 [ {{instanceof Soap11}} ], CXF should use {{MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL)}} instead of {{MessageFactory.newInstance()}}
> Further more by default when version is neither of 1.1 or 1.2, then instead of returning {{null}}, CXF should return {{MessageFactory.newInstance(SOAPConstants.DEFAULT_SOAP_PROTOCOL)}} or {{MessageFactory.newInstance(SOAPConstants.DYNAMIC_SOAP_PROTOCOL)}}
> *Given below is an example of cxf code where this refactoring can be done*
> {code:title=org.apache.cxf.jaxws.binding.soap.SOAPBindingImpl.java|borderStyle=solid}
> public MessageFactory getMessageFactory()
> {
> if (this.soapBinding instanceof SoapBindingInfo) {
> SoapBindingInfo bindingInfo = (SoapBindingInfo) this.soapBinding;
> try {
> if (bindingInfo.getSoapVersion() instanceof Soap11) {
> return MessageFactory.newInstance();
> } else if (bindingInfo.getSoapVersion() instanceof Soap12) {
> return MessageFactory.newInstance(SOAPConstants.SOAP_1_2_PROTOCOL);
> }
> } catch (SOAPException e) {
> throw new WebServiceException(BUNDLE.getString("SAAJ_FACTORY_ERR"), e);
> }
> }
> return null;
> }
> {code}
> *Advantages*
> * In future, {{MessageFactory.newInstance()}} may not return object instance of the default implementation that does not supports "SOAP 1.1 Protocol"
> * Some JEE Appserver implementations of {{MessageFactory.newInstance()}}, currently do not return factory that supports "SOAP 1.1 Protocol"
> ** Weblogic 10.3.x [ {{weblogic.webservice.core.soap.MessageFactoryImpl}} ] does not support SAAJ "SOAP 1.1 Protocol" -> "{{java.lang.UnsupportedOperationException: This class does not support SAAJ 1.1}}"
> * This could help ease use of CXF libraries in JEE applications even if Application Server has it's own SOAP/SAAJ implementation. By above method, CXF will always get {{MessageFactory}} that supports required protocol.
> *Related Isues*
> * https://issues.apache.org/jira/browse/CXF-976
> * https://issues.apache.org/jira/browse/CXF-1750
> * https://issues.apache.org/jira/browse/CXF-3307
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Resolved: (CXF-3363) Use
MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL) instead of
MessageFactory.newInstance()
Posted by "Daniel Kulp (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CXF-3363?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Daniel Kulp resolved CXF-3363.
------------------------------
Resolution: Fixed
Fix Version/s: 2.3.4
2.4
Assignee: Daniel Kulp
> Use MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL) instead of MessageFactory.newInstance()
> -------------------------------------------------------------------------------------------------------
>
> Key: CXF-3363
> URL: https://issues.apache.org/jira/browse/CXF-3363
> Project: CXF
> Issue Type: Improvement
> Components: JAX-WS Runtime, Soap Binding
> Affects Versions: 2.3.2
> Reporter: Gladwin
> Assignee: Daniel Kulp
> Fix For: 2.4, 2.3.4
>
>
> When it is known that SOAP version used is 1.1 [ {{instanceof Soap11}} ], CXF should use {{MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL)}} instead of {{MessageFactory.newInstance()}}
> Further more by default when version is neither of 1.1 or 1.2, then instead of returning {{null}}, CXF should return {{MessageFactory.newInstance(SOAPConstants.DEFAULT_SOAP_PROTOCOL)}} or {{MessageFactory.newInstance(SOAPConstants.DYNAMIC_SOAP_PROTOCOL)}}
> *Given below is an example of cxf code where this refactoring can be done*
> {code:title=org.apache.cxf.jaxws.binding.soap.SOAPBindingImpl.java|borderStyle=solid}
> public MessageFactory getMessageFactory()
> {
> if (this.soapBinding instanceof SoapBindingInfo) {
> SoapBindingInfo bindingInfo = (SoapBindingInfo) this.soapBinding;
> try {
> if (bindingInfo.getSoapVersion() instanceof Soap11) {
> return MessageFactory.newInstance();
> } else if (bindingInfo.getSoapVersion() instanceof Soap12) {
> return MessageFactory.newInstance(SOAPConstants.SOAP_1_2_PROTOCOL);
> }
> } catch (SOAPException e) {
> throw new WebServiceException(BUNDLE.getString("SAAJ_FACTORY_ERR"), e);
> }
> }
> return null;
> }
> {code}
> *Advantages*
> * In future, {{MessageFactory.newInstance()}} may not return object instance of the default implementation that does not supports "SOAP 1.1 Protocol"
> * Some JEE Appserver implementations of {{MessageFactory.newInstance()}}, currently do not return factory that supports "SOAP 1.1 Protocol"
> ** Weblogic 10.3.x [ {{weblogic.webservice.core.soap.MessageFactoryImpl}} ] does not support SAAJ "SOAP 1.1 Protocol" -> "{{java.lang.UnsupportedOperationException: This class does not support SAAJ 1.1}}"
> * This could help ease use of CXF libraries in JEE applications even if Application Server has it's own SOAP/SAAJ implementation. By above method, CXF will always get {{MessageFactory}} that supports required protocol.
> *Related Isues*
> * https://issues.apache.org/jira/browse/CXF-976
> * https://issues.apache.org/jira/browse/CXF-1750
> * https://issues.apache.org/jira/browse/CXF-3307
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] Commented: (CXF-3363) Use
MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL) instead of
MessageFactory.newInstance()
Posted by "Gladwin (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CXF-3363?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12999636#comment-12999636 ]
Gladwin commented on CXF-3363:
------------------------------
*If CXF won't deal with deployment issues for popular JEE containers like Tomcat/WebSphere/Weblogic/JBoss/Glassfish, etc. then it will loose a big market.*
For CXF to deal with classloader issues in the various J2EE servers, another solution is as given below.
To resolve this we should be able to force CXF to use specific message factory we can set as java system property by which then CXF will always return specified message factory instance when no protocol parameter is passed. Further more it should not interfere with JEE application server unless server itself uses CXF libraries internally.
*java ... -D=org.apache.cxf.binding.soap.MessageFactoryCreator.messageFactoryClassName=com.sun.xml.messaging.saaj.soap.ver1_1SOAPMessageFactory1_1Impl*
*Solution*
*In CXF code, replace* all {{MessageFactory.newInstance}} calls in CXF with {{MessageFactoryCreator.create}} as mentioned below:
{code:title=Original calls to be replaced|borderStyle=solid}
MessageFactory.newInstance();
MessageFactory.newInstance(someProtocol);
{code}
{code:title=New calls to be replaced with|borderStyle=solid}
MessageFactoryCreator.create();
MessageFactoryCreator.create(someProtocol);
{code}
*Define* {{org.apache.cxf.binding.soap.MessageFactoryCreator}} class in CXF
{code:title=org.apache.cxf.binding.soap.MessageFactoryCreator (This code is not tested)|borderStyle=solid}
package org.apache.cxf.binding.soap;
import javax.xml.soap.MessageFactory;
import javax.xml.soap.SOAPException;
public class MessageFactoryCreator {
public static final String MESSAGE_FACTORY_KEY = "org.apache.cxf.binding.soap.MessageFactoryCreator.messageFactoryClassName";
public MessageFactoryCreator() {
}
public static MessageFactory create() throws SOAPException {
MessageFactory messageFactory;
String messageFactoryClassName = System
.getProperty(MessageFactoryCreator.MESSAGE_FACTORY_KEY);
if (messageFactoryClassName != null) {
messageFactory = newInstanceCxfMessageFactory(messageFactoryClassName);
} else {
messageFactory = MessageFactory.newInstance();
}
return messageFactory;
}
public static MessageFactory create(String protocol)
throws SOAPException {
MessageFactory messageFactory = MessageFactory.newInstance(protocol);
return messageFactory;
}
@SuppressWarnings("unchecked")
private static MessageFactory newInstanceCxfMessageFactory(
String messageFactoryClassName) throws SOAPException {
try {
Class<MessageFactory> klass = (Class<MessageFactory>) Class
.forName(messageFactoryClassName);
MessageFactory messageFactory = klass.newInstance();
return messageFactory;
} catch (ClassNotFoundException cnfe) {
throw new SOAPException("Provider " + messageFactoryClassName
+ " could not be instantiated: " + cnfe, cnfe);
} catch (InstantiationException ie) {
throw new SOAPException("Provider " + messageFactoryClassName
+ " could not be instantiated: " + ie, ie);
} catch (IllegalAccessException iae) {
throw new SOAPException("Provider " + messageFactoryClassName
+ " could not be instantiated: " + iae, iae);
}
}
}
{code}
*Note:* This solution is derived from the fact that in Weblogic 10.3.3 server, there are issues as it automatically set the system property {{javax.xml.soap.MessageFactory}}={{weblogic.webservice.core.soap.MessageFactoryImpl}} hence forcing all the {{MessageFactory.newInstance()}} calls to return this factory by default (see javadoc of [javax.xml.soap.MessageFactory.getInstance()|http://download.oracle.com/javase/6/docs/api/javax/xml/soap/MessageFactory.html#newInstance%28%29] for explaination).
> Use MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL) instead of MessageFactory.newInstance()
> -------------------------------------------------------------------------------------------------------
>
> Key: CXF-3363
> URL: https://issues.apache.org/jira/browse/CXF-3363
> Project: CXF
> Issue Type: Improvement
> Components: JAX-WS Runtime, Soap Binding
> Affects Versions: 2.3.2
> Reporter: Gladwin
>
> When it is known that SOAP version used is 1.1 [ {{instanceof Soap11}} ], CXF should use {{MessageFactory.newInstance(SOAPConstants.SOAP_1_1_PROTOCOL)}} instead of {{MessageFactory.newInstance()}}
> Further more by default when version is neither of 1.1 or 1.2, then instead of returning {{null}}, CXF should return {{MessageFactory.newInstance(SOAPConstants.DEFAULT_SOAP_PROTOCOL)}} or {{MessageFactory.newInstance(SOAPConstants.DYNAMIC_SOAP_PROTOCOL)}}
> *Given below is an example of cxf code where this refactoring can be done*
> {code:title=org.apache.cxf.jaxws.binding.soap.SOAPBindingImpl.java|borderStyle=solid}
> public MessageFactory getMessageFactory()
> {
> if (this.soapBinding instanceof SoapBindingInfo) {
> SoapBindingInfo bindingInfo = (SoapBindingInfo) this.soapBinding;
> try {
> if (bindingInfo.getSoapVersion() instanceof Soap11) {
> return MessageFactory.newInstance();
> } else if (bindingInfo.getSoapVersion() instanceof Soap12) {
> return MessageFactory.newInstance(SOAPConstants.SOAP_1_2_PROTOCOL);
> }
> } catch (SOAPException e) {
> throw new WebServiceException(BUNDLE.getString("SAAJ_FACTORY_ERR"), e);
> }
> }
> return null;
> }
> {code}
> *Advantages*
> * In future, {{MessageFactory.newInstance()}} may not return object instance of the default implementation that does not supports "SOAP 1.1 Protocol"
> * Some JEE Appserver implementations of {{MessageFactory.newInstance()}}, currently do not return factory that supports "SOAP 1.1 Protocol"
> ** Weblogic 10.3.x [ {{weblogic.webservice.core.soap.MessageFactoryImpl}} ] does not support SAAJ "SOAP 1.1 Protocol" -> "{{java.lang.UnsupportedOperationException: This class does not support SAAJ 1.1}}"
> * This could help ease use of CXF libraries in JEE applications even if Application Server has it's own SOAP/SAAJ implementation. By above method, CXF will always get {{MessageFactory}} that supports required protocol.
> *Related Isues*
> * https://issues.apache.org/jira/browse/CXF-976
> * https://issues.apache.org/jira/browse/CXF-1750
> * https://issues.apache.org/jira/browse/CXF-3307
--
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira