Replace logError() with Session.getLog() in SessionConnector

[Copilot]

SessionConnector was logging errors to its class-level logger instead of using session-specific logs where a Session object was available.

Changes

• Replaced three logError() calls with LogUtil.logThrowable(session.getLog(), ...) in:
  • logoutAllSessions(): logout errors
  • logoutAllSessions(): disconnect errors
  • SessionTimerTask.run(): timer processing errors
• Removed the now-unused logError(SessionID, IoSession, String, Throwable) method
• Removed the helper method getLogSuffix(SessionID, IoSession) which was only used by logError()

Before:

logError(session.getSessionID(), null, "Error during logout", e);

After:

LogUtil.logThrowable(session.getLog(), "Error during logout", e);

This routes errors to per-session log configuration rather than the generic connector logger, consistent with error logging patterns in Session itself. The cleanup removes dead code that is no longer needed.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Replace logError()´ with call to Session.getLog()where applicable inSessionConnector`</issue_title>
<issue_description></issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Replace logError() with call to Session.getLog() where applicable in SessionConnector #1107

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[chrjohn]

@copilot , is this method still used anywhere? Please remove it after changing the callers to use a session-specific log.

[Copilot]

The logError() method was not used anywhere else in the codebase. I've removed both the logError() method and its helper method getLogSuffix() in commit 9e29ecf.