Add: Move logger creation in CleanupJob to constructor

This commit is contained in:
Lars Simon Winzer
2026-03-15 19:49:23 +01:00
parent 0a5eb5d11c
commit 68827e177e
@@ -10,12 +10,12 @@ import org.apache.logging.log4j.Logger;
* threshold. * threshold.
*/ */
public class UserCleanupJob implements Runnable { public class UserCleanupJob implements Runnable {
private static final Logger logger = LogManager.getLogger(UserCleanupJob.class); private final Logger logger;
private final UserRegistry registry; private final UserRegistry registry;
private final Duration reconnectThreshold; private final Duration reconnectThreshold;
public UserCleanupJob(UserRegistry registry, Duration reconnectThreshold) { public UserCleanupJob(UserRegistry registry, Duration reconnectThreshold) {
this.logger = LogManager.getLogger(UserCleanupJob.class);
this.registry = registry; this.registry = registry;
this.reconnectThreshold = reconnectThreshold; this.reconnectThreshold = reconnectThreshold;
} }
@@ -33,7 +33,8 @@ public class UserCleanupJob implements Runnable {
Instant disconnectedAt = user.getDisconnectedAt().get(); Instant disconnectedAt = user.getDisconnectedAt().get();
if (disconnectedAt.isBefore(threshold)) { if (disconnectedAt.isBefore(threshold)) {
if (registry.removeIfStillDisconnected(user.getId())) { if (registry.removeIfStillDisconnected(user.getId())) {
logger.info("Removed expired user {} ({})", user.getName(), user.getId().value()); logger.info(
"Removed expired user {} ({})", user.getName(), user.getId().value());
} }
} }
} }