[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