[java-metadata-aggregator COMMIT] in /trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata/dom: NamespaceS...

noreply at shibboleth.net noreply at shibboleth.net
Sun Jun 30 12:12:36 EDT 2013


Author: iay
Date: Sun Jun 30 12:12:36 2013
New Revision: 245

URL: http://svn.shibboleth.net/view/java-metadata-aggregator?rev=245&view=rev
Log:
MDA-98: review use of debug vs. trace logging in aggregator stages

Modified:
    trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata/dom/NamespaceStrippingStage.java
    trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata/dom/XMLSignatureValidationStage.java

Modified: trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata/dom/NamespaceStrippingStage.java
URL: http://svn.shibboleth.net/view/java-metadata-aggregator/trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata/dom/NamespaceStrippingStage.java?rev=245&r1=244&r2=245&view=diff
==============================================================================
--- trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata/dom/NamespaceStrippingStage.java (original)
+++ trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata/dom/NamespaceStrippingStage.java Sun Jun 30 12:12:36 2013
@@ -126,15 +126,15 @@
             final Attr attribute = (Attr) attributes.item(aIndex);
             final String attrNamespace = attribute.getNamespaceURI();
             final String attrLocalName = attribute.getLocalName();
-            log.debug("checking attribute {{}}:{}", attrNamespace, attrLocalName);
+            log.trace("checking attribute {{}}:{}", attrNamespace, attrLocalName);
             if (namespace.equals(attrNamespace)) {
                 // remove attribute in target namespace
-                log.debug("   in target namespace; will remove");
+                log.trace("   in target namespace; will remove");
                 removeTarget.add(attribute);
             } else if (XMLConstants.XMLNS_ATTRIBUTE_NS_URI.equals(attrNamespace)
                     && namespace.equals(attribute.getTextContent())) {
                 // remove prefix definition
-                log.debug("   prefix {} definition; will remove", attrLocalName);
+                log.trace("   prefix {} definition; will remove", attrLocalName);
                 removeTarget.add(attribute);
             }
         }
@@ -160,13 +160,13 @@
      */
     private void processElement(@Nonnull final Element element, final int depth) {
         Constraint.isNotNull(element, "Element can not be null");
-        log.debug("{}: element {}", depth, element.getLocalName());
+        log.trace("{}: element {}", depth, element.getLocalName());
 
         /*
          * If this element is in the target namespace, remove it from the DOM entirely and we're done.
          */
         if (namespace.equals(element.getNamespaceURI())) {
-            log.debug("{}: removing element entirely", depth);
+            log.trace("{}: removing element entirely", depth);
             element.getParentNode().removeChild(element);
             return;
         }

Modified: trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata/dom/XMLSignatureValidationStage.java
URL: http://svn.shibboleth.net/view/java-metadata-aggregator/trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata/dom/XMLSignatureValidationStage.java?rev=245&r1=244&r2=245&view=diff
==============================================================================
--- trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata/dom/XMLSignatureValidationStage.java (original)
+++ trunk/aggregator-pipeline/src/main/java/net/shibboleth/metadata/dom/XMLSignatureValidationStage.java Sun Jun 30 12:12:36 2013
@@ -203,7 +203,7 @@
      * @throws StageProcessingException thrown if the given root element contains more than one signature
      */
     protected boolean signatureVerified(@Nonnull final Element signatureElement) throws StageProcessingException {
-        log.debug("Creating XML security library XMLSignature object");
+        log.trace("Creating XML security library XMLSignature object");
         final XMLSignature signature;
         try {
             signature = new XMLSignature(signatureElement, "");



More information about the commits mailing list