Better logging.
authorwassa <wassa@ab3bd59b-922f-494d-bb5f-6f0a3c29deca>
Thu, 17 Mar 2005 19:59:20 +0000 (19:59 +0000)
committerwassa <wassa@ab3bd59b-922f-494d-bb5f-6f0a3c29deca>
Thu, 17 Mar 2005 19:59:20 +0000 (19:59 +0000)
git-svn-id: https://subversion.switch.ch/svn/shibboleth/java-idp/trunk@1306 ab3bd59b-922f-494d-bb5f-6f0a3c29deca

src/edu/internet2/middleware/shibboleth/idp/provider/SAMLv1_1ArtifactQueryHandler.java

index 436cb3e..2822c14 100644 (file)
@@ -103,12 +103,16 @@ public class SAMLv1_1ArtifactQueryHandler extends BaseServiceHandler implements
                // for transaction log
                StringBuffer dereferencedArtifacts = new StringBuffer();
 
+               // TODO make sure we don't work on artifacts that are expired
+
                while (artifacts.hasNext()) {
                        queriedArtifacts++;
                        Artifact artifact = (Artifact) artifacts.next();
-                       log.debug("Attempting to dereference artifact: (" + artifact.toString() + ").");
+                       log.info("Dereferencing artifact: (" + artifact.toString() + ").");
                        ArtifactMapping mapping = support.getArtifactMapper().recoverAssertion(artifact);
-                       if (mapping != null) {
+                       if (mapping == null) {
+                               log.info("Could not map artifact to a SAML Assertion.");
+                       } else {
                                SAMLAssertion assertion = mapping.getAssertion();
                                // See if we have metadata for this provider
                                EntityDescriptor provider = support.lookup(mapping.getServiceProviderId());