Merge pull request #218 from jmluy/logs-no-session

Add session id to logs for no active session
ChargeTimeEU · Oct 20, 2022 · b163057 · b163057
2 parents 9fe3b79 + 1e45459
commit b163057
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/ocpp-common/src/main/java/eu/chargetime/ocpp/Server.java b/ocpp-common/src/main/java/eu/chargetime/ocpp/Server.java
@@ -117,8 +117,8 @@ public Confirmation handleRequest(Request request)
                             .handleRequest(sessionIdOptional.get(), request);
                       } else {
                         logger.error(
-                            "Unable to handle request ({}), the active session was not found.",
-                            request);
+                            "Unable to handle request ({}), the active session was not found for {}.",
+                            request, session.getSessionId());
                         throw new IllegalStateException("Active session not found");
                       }
                     } else {
@@ -154,7 +154,7 @@ public void handleConnectionClosed() {
                       serverEvents.lostSession(sessionIdOptional.get());
                       sessions.remove(sessionIdOptional.get());
                     } else {
-                      logger.warn("Active session not found");
+                      logger.warn("Active session not found for {}", session.getSessionId());
                     }
                   }