Use checkSessionTimerRunning() and remove debug logging

Copilot · chrjohn · Copilot · commit dd82c5eeb89e · 2025-12-17T11:32:32.000Z
Co-authored-by: chrjohn &lt;6644028+chrjohn@users.noreply.github.com&gt;
diff --git a/quickfixj-core/src/main/java/quickfix/mina/SessionConnector.java b/quickfixj-core/src/main/java/quickfix/mina/SessionConnector.java
@@ -312,8 +312,7 @@ private String getLogSuffix(SessionID sessionID, IoSession protocolSession) {
 
     protected void startSessionTimer() {
         // Check if a session timer is already running to avoid creating multiple timers
-        if (sessionTimerFuture != null && !sessionTimerFuture.isDone()) {
-            log.debug("SessionTimer already running, not creating a new one");
+        if (checkSessionTimerRunning()) {
             return;
         }
         Runnable timerTask = new SessionTimerTask();

Original file line number	Diff line number	Diff line change
`@@ -312,8 +312,7 @@ private String getLogSuffix(SessionID sessionID, IoSession protocolSession) {`
`312`	`312`
`313`	`313`	`protected void startSessionTimer() {`
`314`	`314`	`// Check if a session timer is already running to avoid creating multiple timers`
`315`		`- if (sessionTimerFuture != null && !sessionTimerFuture.isDone()) {`
`316`		`- log.debug("SessionTimer already running, not creating a new one");`
	`315`	`+ if (checkSessionTimerRunning()) {`
`317`	`316`	`return;`
`318`	`317`	`}`
`319`	`318`	`Runnable timerTask = new SessionTimerTask();`