From 963e04e50cd16ee1383c871fed5cec0ca92998dc Mon Sep 17 00:00:00 2001 From: Noah Levitt Date: Tue, 13 May 2014 15:47:21 -0700 Subject: [PATCH] handle exception determining whether to apply overlay --- .../org/archive/crawler/spring/SheetOverlaysManager.java | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/engine/src/main/java/org/archive/crawler/spring/SheetOverlaysManager.java b/engine/src/main/java/org/archive/crawler/spring/SheetOverlaysManager.java index c25bfaeb7..50d541fd6 100644 --- a/engine/src/main/java/org/archive/crawler/spring/SheetOverlaysManager.java +++ b/engine/src/main/java/org/archive/crawler/spring/SheetOverlaysManager.java @@ -30,6 +30,7 @@ import java.util.concurrent.ConcurrentHashMap; import java.util.concurrent.ConcurrentSkipListMap; import java.util.concurrent.ConcurrentSkipListSet; +import java.util.logging.Level; import java.util.logging.Logger; import org.archive.modules.CrawlURI; @@ -326,8 +327,12 @@ public void applyOverlaysTo(CrawlURI curi) { } // apply deciderule-based overlays for(DecideRuledSheetAssociation assoc : ruleAssociations) { - if(assoc.getRules().accepts(curi)) { - curi.getOverlayNames().addAll(assoc.getTargetSheetNames()); + try { + if(assoc.getRules().accepts(curi)) { + curi.getOverlayNames().addAll(assoc.getTargetSheetNames()); + } + } catch (Exception e) { + logger.log(Level.SEVERE, "problem determining whether to apply overlays, so not applying " + assoc.getTargetSheetNames() + " to " + curi, e); } } // even if no overlays set, let creation of empty list signal