Merge pull request #19 from No3x/patch-1

Improve exception handling with InvocationTargetException

src/main/java/org/openhab/automation/jrule/internal/engine/JRuleEngine.java

@@ -14,6 +14,9 @@
 
 import java.beans.PropertyChangeEvent;
 import java.beans.PropertyChangeListener;
+import java.io.IOException;
+import java.io.PrintWriter;
+import java.io.StringWriter;
 import java.lang.reflect.Field;
 import java.lang.reflect.InvocationTargetException;
 import java.lang.reflect.Method;
@@ -466,8 +469,21 @@ private synchronized void invokeRule(JRuleExecutionContext context, JRuleEvent e
         rule.setRuleLogName(context.getLogName());
         try {
             final Object invoke = context.isEventParameterPresent() ? method.invoke(rule, event) : method.invoke(rule);
-        } catch (IllegalAccessException | IllegalArgumentException | InvocationTargetException | SecurityException e) {
-            JRuleLog.error(logger, context.getRuleName(), "Error", e);
+        } catch (IllegalAccessException | IllegalArgumentException | SecurityException e) {
+            JRuleLog.error(logger, context.getRuleName(), "Error {}", e);
+        } catch (InvocationTargetException e) {
+            Throwable ex = e.getCause() != null ? e.getCause() : null;
+            JRuleLog.error(logger, context.getRuleName(), "Error message: {}", ex.getMessage());
+            JRuleLog.error(logger, context.getRuleName(), "Error Stacktrace: {}", getStackTraceAsString(ex));
+        }
+    }
+
+    private synchronized static String getStackTraceAsString(Throwable throwable) {
+        try (StringWriter sw = new StringWriter(); PrintWriter pw = new PrintWriter(sw)) {
+            throwable.printStackTrace(pw);
+            return sw.toString();
+        } catch (IOException ioe) {
+            throw new IllegalStateException(ioe);
         }
     }
 }