Merge pull request #568 from kwin/template-engines-loading-resilience

Catch all classloading issues
jbake-org · Nov 15, 2018 · 1903219 · 1903219
2 parents aebf496 + 0f8adbf
commit 1903219
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/jbake-core/src/main/java/org/jbake/template/TemplateEngines.java b/jbake-core/src/main/java/org/jbake/template/TemplateEngines.java
@@ -77,8 +77,9 @@ private static AbstractTemplateEngine tryLoadEngine(final JBakeConfiguration con
             Class<? extends AbstractTemplateEngine> engineClass = (Class<? extends AbstractTemplateEngine>) Class.forName(engineClassName, false, TemplateEngines.class.getClassLoader());
             Constructor<? extends AbstractTemplateEngine> ctor = engineClass.getConstructor(JBakeConfiguration.class, ContentStore.class);
             return ctor.newInstance(config, db);
-        } catch (Exception e) {
-            LOGGER.error("unable to load engine", e);
+        } catch (Throwable e) {
+            // not all engines might be necessary, therefore only emit class loading issue with level warn
+            LOGGER.warn("unable to load engine", e);
             return null;
         }
     }