Author: coheigea
Date: Fri Oct 5 15:57:35 2012
New Revision: 1394612
URL: http://svn.apache.org/viewvc?rev=1394612&view=rev
Log:
Minor performance improvement when evaluating multiple XPaths
Modified:
cxf/trunk/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/PolicyBasedWSS4JInInterceptor.java
Modified:
cxf/trunk/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/PolicyBasedWSS4JInInterceptor.java
URL:
http://svn.apache.org/viewvc/cxf/trunk/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/PolicyBasedWSS4JInInterceptor.java?rev=1394612&r1=1394611&r2=1394612&view=diff
==============================================================================
---
cxf/trunk/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/PolicyBasedWSS4JInInterceptor.java
(original)
+++
cxf/trunk/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/PolicyBasedWSS4JInInterceptor.java
Fri Oct 5 15:57:35 2012
@@ -23,6 +23,7 @@ import java.io.IOException;
import java.io.InputStream;
import java.net.URL;
import java.util.ArrayList;
+import java.util.Arrays;
import java.util.Collection;
import java.util.HashSet;
import java.util.List;
@@ -408,15 +409,14 @@ public class PolicyBasedWSS4JInIntercept
}
if (xpaths != null) {
- for (String xPath : xpaths) {
- try {
- CryptoCoverageUtil.checkCoverage(soapEnvelope,
refs,
- namespaces, xPath, type, scope);
- } catch (WSSecurityException e) {
- ai.setNotAsserted("No " + type
- + " element found matching XPath " +
xPath);
- return false;
- }
+ try {
+ CryptoCoverageUtil.checkCoverage(soapEnvelope, refs,
+ namespaces, xpaths, type, scope);
+ } catch (WSSecurityException e) {
+ ai.setNotAsserted("No " + type
+ + " element found matching one of the XPaths "
+ + Arrays.toString(xpaths.toArray()));
+ return false;
}
}
}