Defensive tpl engine

Author: krystian-panek-vmltech

core/src/main/java/dev/vml/es/acm/core/code/Executor.java

@@ -33,6 +33,8 @@
 import org.osgi.service.metatype.annotations.AttributeDefinition;
 import org.osgi.service.metatype.annotations.Designate;
 import org.osgi.service.metatype.annotations.ObjectClassDefinition;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 @Component(
         immediate = true,
@@ -43,6 +45,8 @@ public class Executor implements EventListener {
 
     public static final String LOCK_DIR = "executor";
 
+    private static final Logger LOG = LoggerFactory.getLogger(Executor.class);
+
     @ObjectClassDefinition(name = "AEM Content Manager - Code Executor")
     public @interface Config {
 
@@ -278,10 +282,8 @@ private void handleNotifications(ContextualExecution execution) {
                         ? "✅"
                         : (execution.getStatus() == ExecutionStatus.FAILED ? "❌" : "⚠️"));
         templateVars.put("statusHere", execution.getStatus() == ExecutionStatus.SUCCEEDED ? "" : "@here");
-        TemplateFormatter templateFormatter =
-                execution.getContext().getCodeContext().getFormatter().getTemplate();
-        String title = StringUtils.trim(templateFormatter.renderString(config.notificationTitle(), templateVars));
-        String text = StringUtils.trim(templateFormatter.renderString(config.notificationText(), templateVars));
+        String title = StringUtils.trim(formatTemplate(config.notificationTitle(), templateVars));
+        String text = StringUtils.trim(formatTemplate(config.notificationText(), templateVars));
 
         Map<String, Object> fields = new LinkedHashMap<>();
         fields.put("Status", execution.getStatus().name().toLowerCase());
@@ -372,4 +374,13 @@ private void useLocker(ResourceResolverFactory resolverFactory, Consumer<Locker>
     private void useHistory(ResourceResolverFactory resolverFactory, Consumer<ExecutionHistory> consumer) {
         ResolverUtils.useContentResolver(resolverFactory, null, r -> consumer.accept(new ExecutionHistory(r)));
     }
+
+    private String formatTemplate(String template, Map<String, Object> vars) {
+        try {
+            return new TemplateFormatter().renderString(template, vars);
+        } catch (Exception e) {
+            LOG.warn("Cannot format template '{}'!", template, e);
+            return template;
+        }
+    }
 }