[jbossws-commits] JBossWS SVN: r17682 - in stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782: modules/core and 4 other directories.

Author: klape Date: 2013-06-14 21:07:28 -0400 (Fri, 14 Jun 2013) New Revision: 17682 Modified: stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/ stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/ stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/core/soap/SOAPFaultImpl.java stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/WSSecurityDispatcher.java stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/FailedAuthenticationException.java stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/FailedCheckException.java stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/InvalidSecurityHeaderException.java stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/InvalidSecurityTokenException.java stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/SecurityTokenUnavailableException.java stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/UnsupportedAlgorithmException.java stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/UnsupportedSecurityTokenException.java stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/WSSecurityException.java stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/testsuite/native-tests/ Log: [JBPAPP-10782] Customer's fix: WSSecurityException conversion to SOAP 1.2 fault fails Property changes on: stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782 ___________________________________________________________________ Modified: svn:mergeinfo - /stack/native/branches/jbossws-native-3.1.2:17030 /stack/native/branches/jbossws-native-3.1.2.SP10_JBPAPP-8128:15667 /stack/native/branches/jbossws-native-3.1.2.SP10_JBPAPP6864:14776-14778,14786,14894,14905,14907-14908 /stack/native/branches/jbossws-native-3.1.2.SP3-patch-02_JBPAPP-6365:14192 /stack/native/branches/jbossws-native-3.1.2.SP7_JBPAPP-8007:15579 /stack/native/branches/jbossws-native-3.4.0.SP1:13928,13933,13936,13946 /stack/native/trunk:12502,13992,14157,14160,14181,14183,14842 + /stack/native/branches/jbossws-native-3.1.2:17030,17378 /stack/native/branches/jbossws-native-3.1.2.SP10_JBPAPP-8128:15667 /stack/native/branches/jbossws-native-3.1.2.SP10_JBPAPP6864:14776-14778,14786,14894,14905,14907-14908 /stack/native/branches/jbossws-native-3.1.2.SP3-patch-02_JBPAPP-6365:14192 /stack/native/branches/jbossws-native-3.1.2.SP7_JBPAPP-8007:15579 /stack/native/branches/jbossws-native-3.4.0.SP1:13928,13933,13936,13946 /stack/native/trunk:12502,13992,14157,14160,14181,14183,14842 Property changes on: stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core ___________________________________________________________________ Modified: svn:mergeinfo - /stack/native/branches/jbossws-native-3.1.2/modules/core:17030 /stack/native/branches/jbossws-native-3.1.2.SP10_JBPAPP-8128/modules/core:15667 /stack/native/branches/jbossws-native-3.1.2.SP10_JBPAPP6864/modules/core:14776-14778,14786,14894,14905,14907-14908 /stack/native/branches/jbossws-native-3.1.2.SP3-patch-02_JBPAPP-6365/modules/core:14192 /stack/native/branches/jbossws-native-3.1.2.SP7_JBPAPP-8007/modules/core:15579 /stack/native/branches/jbossws-native-3.4.0.SP1/modules/core:13928,13933,13936,13946 /stack/native/trunk/modules/core:12502,13992,14011,14157,14160,14181,14183,14842 + /stack/native/branches/jbossws-native-3.1.2/modules/core:17030,17378 /stack/native/branches/jbossws-native-3.1.2.SP10_JBPAPP-8128/modules/core:15667 /stack/native/branches/jbossws-native-3.1.2.SP10_JBPAPP6864/modules/core:14776-14778,14786,14894,14905,14907-14908 /stack/native/branches/jbossws-native-3.1.2.SP3-patch-02_JBPAPP-6365/modules/core:14192 /stack/native/branches/jbossws-native-3.1.2.SP7_JBPAPP-8007/modules/core:15579 /stack/native/branches/jbossws-native-3.4.0.SP1/modules/core:13928,13933,13936,13946 /stack/native/trunk/modules/core:12502,13992,14011,14157,14160,14181,14183,14842 Modified: stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/core/soap/SOAPFaultImpl.java =================================================================== --- stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/core/soap/SOAPFaultImpl.java 2013-06-15 00:51:26 UTC (rev 17681) +++ stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/core/soap/SOAPFaultImpl.java 2013-06-15 01:07:28 UTC (rev 17682) @@ -800,7 +800,7 @@ public void writeElement(Writer writer) { - new DOMWriter(writer).print(this); + new DOMWriter(writer).setCompleteNamespaces(false).print(this); } private static SOAPElement getChildElement(SOAPElement element, QName name) Modified: stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/WSSecurityDispatcher.java =================================================================== --- stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/WSSecurityDispatcher.java 2013-06-15 00:51:26 UTC (rev 17681) +++ stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/WSSecurityDispatcher.java 2013-06-15 01:07:28 UTC (rev 17682) @@ -25,16 +25,20 @@ import java.util.List; import javax.xml.namespace.QName; +import javax.xml.soap.SOAPConstants; import javax.xml.soap.SOAPException; import javax.xml.soap.SOAPHeader; import javax.xml.soap.SOAPMessage; import javax.xml.ws.WebServiceException; +import javax.xml.ws.soap.SOAPFaultException; +import javax.xml.ws.soap.SOAPBinding; import org.jboss.logging.Logger; import org.jboss.ws.core.CommonMessageContext; import org.jboss.ws.core.CommonSOAPFaultException; import org.jboss.ws.core.soap.MessageContextAssociation; import org.jboss.ws.core.soap.SOAPMessageImpl; +import org.jboss.ws.core.soap.SOAPFaultImpl; import org.jboss.ws.extensions.security.exception.InvalidSecurityHeaderException; import org.jboss.ws.extensions.security.exception.WSSecurityException; import org.jboss.ws.extensions.security.nonce.DefaultNonceFactory; @@ -263,21 +267,65 @@ return newList; } - private CommonSOAPFaultException convertToFault(WSSecurityException e) + private RuntimeException convertToFault(WSSecurityException e) throws SOAPException { return convertToFault(e, VERBOSE_EXCEPTION_REPORTING); } - private CommonSOAPFaultException convertToFault(WSSecurityException e, boolean verbose) + private RuntimeException convertToFault(WSSecurityException e, boolean verbose) throws SOAPException { - if (verbose) { - return new CommonSOAPFaultException(e.getFaultCode(), e.getFaultString()); - } else { - QName faultCode = new QName(Constants.JBOSS_WSSE_NS, "GenericError", Constants.JBOSS_WSSE_PREFIX); - return new CommonSOAPFaultException(faultCode, "A WS-Security error occurred."); + //Try to reduce redundant stack trace elements printed to log + chopStackTrace(e); + log.error("Original WSSecurityException: ", e); + + if(isSOAP12()) + { + SOAPFaultImpl fault = new SOAPFaultImpl( + org.jboss.ws.Constants.PREFIX_ENV, + org.jboss.ws.Constants.NS_SOAP12_ENV + ); + + if(e.isInternalError()) + fault.setFaultCode(SOAPConstants.SOAP_RECEIVER_FAULT); + else + fault.setFaultCode(SOAPConstants.SOAP_SENDER_FAULT); + + fault.appendFaultSubcode(e.getFaultCode()); + fault.setFaultString(e.getFaultString()); + + return new SOAPFaultException(fault); } + else + { + if (verbose) + { + return new CommonSOAPFaultException(e.getFaultCode(), e.getFaultString()); + } + else + { + QName faultCode = new QName(Constants.JBOSS_WSSE_NS, "GenericError", Constants.JBOSS_WSSE_PREFIX); + return new CommonSOAPFaultException(faultCode, "A WS-Security error occurred."); + } + } } + private void chopStackTrace(Exception e) + { + StackTraceElement[] original = e.getStackTrace(); + int cutOffElement = 0; + for(; cutOffElement < original.length; cutOffElement++) + { + StackTraceElement elem = original[cutOffElement]; + String className = elem.getClassName(); + if("javax.servlet.http.HttpServlet".equals(className)) + { + StackTraceElement[] newSte = new StackTraceElement[cutOffElement + 1]; + System.arraycopy(original, 0, newSte, 0, cutOffElement + 1); + e.setStackTrace(newSte); + } + } + } + private List<RequireOperation> buildRequireOperations(Config operationConfig, boolean fault) { if (operationConfig == null) @@ -368,4 +416,20 @@ return requires != null && (!fault || requires.includesFaults()); } + + private static boolean isSOAP12() + { + CommonMessageContext msgContext = MessageContextAssociation.peekMessageContext(); + if (msgContext != null) + { + EndpointMetaData emd = msgContext.getEndpointMetaData(); + String bindingId = emd.getBindingId(); + if (SOAPBinding.SOAP12HTTP_BINDING.equals(bindingId) || SOAPBinding.SOAP12HTTP_MTOM_BINDING.equals(bindingId)) + { + return true; + } + } + + return false; + } } Modified: stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/FailedAuthenticationException.java =================================================================== --- stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/FailedAuthenticationException.java 2013-06-15 00:51:26 UTC (rev 17681) +++ stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/FailedAuthenticationException.java 2013-06-15 01:07:28 UTC (rev 17682) @@ -39,6 +39,7 @@ super(faultString); setFaultCode(faultCode); setFaultString(faultString); + internal = false; } public FailedAuthenticationException(Throwable cause) @@ -46,6 +47,7 @@ super(faultString); setFaultCode(faultCode); setFaultString(faultString); + internal = false; } public FailedAuthenticationException(String message) @@ -53,6 +55,7 @@ super(message); setFaultCode(faultCode); setFaultString(message); + internal = false; } public FailedAuthenticationException(String message, Throwable cause) @@ -60,5 +63,6 @@ super(message, cause); setFaultCode(faultCode); setFaultString(message); + internal = false; } } Modified: stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/FailedCheckException.java =================================================================== --- stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/FailedCheckException.java 2013-06-15 00:51:26 UTC (rev 17681) +++ stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/FailedCheckException.java 2013-06-15 01:07:28 UTC (rev 17682) @@ -39,6 +39,7 @@ super(faultString); setFaultCode(faultCode); setFaultString(faultString); + internal = false; } public FailedCheckException(Throwable cause) @@ -46,6 +47,7 @@ super(faultString); setFaultCode(faultCode); setFaultString(faultString); + internal = false; } public FailedCheckException(String message) @@ -60,5 +62,6 @@ super(message, cause); setFaultCode(faultCode); setFaultString(message); + internal = false; } } Modified: stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/InvalidSecurityHeaderException.java =================================================================== --- stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/InvalidSecurityHeaderException.java 2013-06-15 00:51:26 UTC (rev 17681) +++ stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/InvalidSecurityHeaderException.java 2013-06-15 01:07:28 UTC (rev 17682) @@ -39,6 +39,7 @@ super(faultString); setFaultCode(faultCode); setFaultString(faultString); + internal = false; } public InvalidSecurityHeaderException(Throwable cause) @@ -46,6 +47,7 @@ super(faultString); setFaultCode(faultCode); setFaultString(faultString); + internal = false; } public InvalidSecurityHeaderException(String message) @@ -53,6 +55,7 @@ super(message); setFaultCode(faultCode); setFaultString(message); + internal = false; } public InvalidSecurityHeaderException(String message, Throwable cause) @@ -60,5 +63,6 @@ super(message, cause); setFaultCode(faultCode); setFaultString(message); + internal = false; } } Modified: stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/InvalidSecurityTokenException.java =================================================================== --- stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/InvalidSecurityTokenException.java 2013-06-15 00:51:26 UTC (rev 17681) +++ stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/InvalidSecurityTokenException.java 2013-06-15 01:07:28 UTC (rev 17682) @@ -39,6 +39,7 @@ super(faultString); setFaultCode(faultCode); setFaultString(faultString); + internal = false; } public InvalidSecurityTokenException(Throwable cause) @@ -46,6 +47,7 @@ super(faultString); setFaultCode(faultCode); setFaultString(faultString); + internal = false; } public InvalidSecurityTokenException(String message) @@ -53,6 +55,7 @@ super(message); setFaultCode(faultCode); setFaultString(message); + internal = false; } public InvalidSecurityTokenException(String message, Throwable cause) @@ -60,5 +63,6 @@ super(message, cause); setFaultCode(faultCode); setFaultString(message); + internal = false; } } Modified: stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/SecurityTokenUnavailableException.java =================================================================== --- stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/SecurityTokenUnavailableException.java 2013-06-15 00:51:26 UTC (rev 17681) +++ stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/SecurityTokenUnavailableException.java 2013-06-15 01:07:28 UTC (rev 17682) @@ -39,6 +39,7 @@ super(faultString); setFaultCode(faultCode); setFaultString(faultString); + internal = false; } public SecurityTokenUnavailableException(Throwable cause) @@ -46,6 +47,7 @@ super(faultString); setFaultCode(faultCode); setFaultString(faultString); + internal = false; } public SecurityTokenUnavailableException(String message) @@ -53,6 +55,7 @@ super(message); setFaultCode(faultCode); setFaultString(message); + internal = false; } public SecurityTokenUnavailableException(String message, Throwable cause) @@ -60,5 +63,6 @@ super(message, cause); setFaultCode(faultCode); setFaultString(message); + internal = false; } } Modified: stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/UnsupportedAlgorithmException.java =================================================================== --- stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/UnsupportedAlgorithmException.java 2013-06-15 00:51:26 UTC (rev 17681) +++ stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/UnsupportedAlgorithmException.java 2013-06-15 01:07:28 UTC (rev 17682) @@ -39,6 +39,7 @@ super(faultString); setFaultCode(faultCode); setFaultString(faultString); + internal = false; } public UnsupportedAlgorithmException(Throwable cause) @@ -46,6 +47,7 @@ super(faultString); setFaultCode(faultCode); setFaultString(faultString); + internal = false; } public UnsupportedAlgorithmException(String message) @@ -53,6 +55,7 @@ super(message); setFaultCode(faultCode); setFaultString(message); + internal = false; } public UnsupportedAlgorithmException(String message, Throwable cause) @@ -60,5 +63,6 @@ super(message, cause); setFaultCode(faultCode); setFaultString(message); + internal = false; } } Modified: stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/UnsupportedSecurityTokenException.java =================================================================== --- stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/UnsupportedSecurityTokenException.java 2013-06-15 00:51:26 UTC (rev 17681) +++ stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/UnsupportedSecurityTokenException.java 2013-06-15 01:07:28 UTC (rev 17682) @@ -39,6 +39,7 @@ super(faultString); setFaultCode(faultCode); setFaultString(faultString); + internal = false; } public UnsupportedSecurityTokenException(Throwable cause) @@ -46,6 +47,7 @@ super(faultString); setFaultCode(faultCode); setFaultString(faultString); + internal = false; } public UnsupportedSecurityTokenException(String message) @@ -53,6 +55,7 @@ super(message); setFaultCode(faultCode); setFaultString(message); + internal = false; } public UnsupportedSecurityTokenException(String message, Throwable cause) @@ -60,5 +63,6 @@ super(message, cause); setFaultCode(faultCode); setFaultString(message); + internal = false; } } Modified: stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/WSSecurityException.java =================================================================== --- stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/WSSecurityException.java 2013-06-15 00:51:26 UTC (rev 17681) +++ stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/core/src/main/java/org/jboss/ws/extensions/security/exception/WSSecurityException.java 2013-06-15 01:07:28 UTC (rev 17682) @@ -30,7 +30,7 @@ */ public class WSSecurityException extends Exception { - private boolean internal = false; + protected boolean internal = false; private QName faultCode = new QName(Constants.JBOSS_WSSE_NS, "InternalError", Constants.JBOSS_WSSE_PREFIX); Property changes on: stack/native/branches/jbossws-native-3.1.2.SP15_JBPAPP-10782/modules/testsuite/native-tests ___________________________________________________________________ Modified: svn:mergeinfo - /stack/native/branches/jbossws-native-3.1.2/modules/testsuite/native-tests:17030 /stack/native/branches/jbossws-native-3.1.2.SP10_JBPAPP-8128/modules/testsuite/native-tests:15667 /stack/native/branches/jbossws-native-3.1.2.SP10_JBPAPP6864/modules/testsuite/native-tests:14776-14778,14786,14894,14905,14907-14908 /stack/native/branches/jbossws-native-3.1.2.SP3-patch-02_JBPAPP-6365/modules/testsuite/native-tests:14192 /stack/native/branches/jbossws-native-3.1.2.SP7_JBPAPP-8007/modules/testsuite/native-tests:15579 /stack/native/branches/jbossws-native-3.4.0.SP1/modules/testsuite/native-tests:13928,13933,13936,13946 /stack/native/trunk/modules/testsuite/native-tests:12502,13992,14013,14157,14160,14181,14183,14842 + /stack/native/branches/jbossws-native-3.1.2/modules/testsuite/native-tests:17030,17378 /stack/native/branches/jbossws-native-3.1.2.SP10_JBPAPP-8128/modules/testsuite/native-tests:15667 /stack/native/branches/jbossws-native-3.1.2.SP10_JBPAPP6864/modules/testsuite/native-tests:14776-14778,14786,14894,14905,14907-14908 /stack/native/branches/jbossws-native-3.1.2.SP3-patch-02_JBPAPP-6365/modules/testsuite/native-tests:14192 /stack/native/branches/jbossws-native-3.1.2.SP7_JBPAPP-8007/modules/testsuite/native-tests:15579 /stack/native/branches/jbossws-native-3.4.0.SP1/modules/testsuite/native-tests:13928,13933,13936,13946 /stack/native/trunk/modules/testsuite/native-tests:12502,13992,14013,14157,14160,14181,14183,14842