This is an automated email from the ASF dual-hosted git repository. billblough pushed a commit to branch RAMPART-234 in repository https://gitbox.apache.org/repos/asf/axis-axis2-java-rampart.git
commit ef34b1004aad1d22ef6c9f9c2b706156d0515dbf Author: Andreas Veithen <veit...@apache.org> AuthorDate: Sun Jan 29 15:30:18 2017 +0000 Avoid unnecessary code changes with respect to trunk. --- .../src/main/java/org/apache/rampart/util/RampartUtil.java | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/modules/rampart-core/src/main/java/org/apache/rampart/util/RampartUtil.java b/modules/rampart-core/src/main/java/org/apache/rampart/util/RampartUtil.java index 2de0201..0bfda53 100644 --- a/modules/rampart-core/src/main/java/org/apache/rampart/util/RampartUtil.java +++ b/modules/rampart-core/src/main/java/org/apache/rampart/util/RampartUtil.java @@ -1823,12 +1823,16 @@ public class RampartUtil { * @throws RampartException */ public static void validateTransport(RampartMessageData rmd) throws RampartException { - + MessageContext msgContext = rmd.getMsgContext(); RampartPolicyData rpd = rmd.getPolicyData(); AxisConfiguration axisConf = msgContext.getConfigurationContext().getAxisConfiguration(); - - if(rpd != null && rpd.isTransportBinding() && !rmd.isInitiator()){ + + if (rpd == null) { + return; + } + + if (rpd.isTransportBinding() && !rmd.isInitiator()) { if (rpd.getTransportToken() instanceof HttpsToken) { try { TransportInDescription transportIn = msgContext.getTransportIn(); @@ -1868,8 +1872,7 @@ public class RampartUtil { // verify client certificate used // try to obtain the client certificate chain directly from the message context // and then from the servlet request - HttpsToken token = (HttpsToken)rpd.getTransportToken(); - if (token.isRequireClientCertificate()) { + if (((HttpsToken)rpd.getTransportToken()).isRequireClientCertificate()) { Object certificateChainProperty = msgContext.getProperty(RampartConstants.HTTPS_CLIENT_CERT_KEY); if (certificateChainProperty instanceof X509Certificate[]) { // HTTPS client certificate chain found @@ -1889,6 +1892,7 @@ public class RampartUtil { // HTTPS client certificate chain NOT found throw new RampartException("httpsClientCertValidationFailed"); } + } } }