From fb8d534e8ef3359ea5d6039123ab454bbaa5f9b6 Mon Sep 17 00:00:00 2001 From: Pinguet62 Date: Sun, 17 Sep 2017 22:31:26 +0200 Subject: [PATCH] Fix logging message when transition action throws exception - Backport #413 - Relates #307 --- .../statemachine/support/AbstractStateMachine.java | 2 +- .../statemachine/support/DefaultStateMachineExecutor.java | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/spring-statemachine-core/src/main/java/org/springframework/statemachine/support/AbstractStateMachine.java b/spring-statemachine-core/src/main/java/org/springframework/statemachine/support/AbstractStateMachine.java index 342ce9d3..c04a9c4f 100644 --- a/spring-statemachine-core/src/main/java/org/springframework/statemachine/support/AbstractStateMachine.java +++ b/spring-statemachine-core/src/main/java/org/springframework/statemachine/support/AbstractStateMachine.java @@ -294,7 +294,7 @@ public abstract class AbstractStateMachine extends StateMachineObjectSuppo } catch (Exception e) { // aborting, executor should stop possible loop checking possible transitions // causing infinite execution - log.warn("Aborting as transition " + t + " caused error " + e); + log.warn("Aborting as transition " + t, e); throw new StateMachineException("Aborting as transition " + t + " caused error ", e); } notifyTransition(buildStateContext(Stage.TRANSITION, message, t, getRelayStateMachine())); diff --git a/spring-statemachine-core/src/main/java/org/springframework/statemachine/support/DefaultStateMachineExecutor.java b/spring-statemachine-core/src/main/java/org/springframework/statemachine/support/DefaultStateMachineExecutor.java index 3e9a6f1a..65dd4d40 100644 --- a/spring-statemachine-core/src/main/java/org/springframework/statemachine/support/DefaultStateMachineExecutor.java +++ b/spring-statemachine-core/src/main/java/org/springframework/statemachine/support/DefaultStateMachineExecutor.java @@ -243,7 +243,7 @@ public class DefaultStateMachineExecutor extends LifecycleObjectSupport im try { transit = t.transit(stateContext); } catch (Exception e) { - log.warn("Aborting as transition " + t + " caused error " + e); + log.warn("Aborting as transition " + t, e); } if (transit) { // if executor transit is raising exception, stop here