improved logs

Alathreon · Alathreon · commit d18f85edec31 · 2024-05-18T16:45:00.000+02:00
diff --git a/JShellAPI/src/main/java/org/togetherjava/jshellapi/service/JShellSessionService.java b/JShellAPI/src/main/java/org/togetherjava/jshellapi/service/JShellSessionService.java
@@ -80,10 +80,10 @@ private synchronized JShellService createSession(SessionInfo sessionInfo) throws
         if(hasSession(sessionInfo.id())) {    //Just in case race condition happens just before createSession
             return jshellSessions.get(sessionInfo.id());
         }
-        LOGGER.info("Creating session : {}.", sessionInfo);
         if(jshellSessions.size() >= config.maxAliveSessions()) {
             throw new ResponseStatusException(HttpStatus.TOO_MANY_REQUESTS, "Too many sessions, try again later :(.");
         }
+        LOGGER.info("Creating session : {}.", sessionInfo);
         JShellService service = new JShellService(
                 dockerService,
                 this,

Original file line number	Diff line number	Diff line change
`@@ -80,10 +80,10 @@ private synchronized JShellService createSession(SessionInfo sessionInfo) throws`
`80`	`80`	`if(hasSession(sessionInfo.id())) { //Just in case race condition happens just before createSession`
`81`	`81`	`return jshellSessions.get(sessionInfo.id());`
`82`	`82`	`}`
`83`		`- LOGGER.info("Creating session : {}.", sessionInfo);`
`84`	`83`	`if(jshellSessions.size() >= config.maxAliveSessions()) {`
`85`	`84`	`throw new ResponseStatusException(HttpStatus.TOO_MANY_REQUESTS, "Too many sessions, try again later :(.");`
`86`	`85`	`}`
	`86`	`+ LOGGER.info("Creating session : {}.", sessionInfo);`
`87`	`87`	`JShellService service = new JShellService(`
`88`	`88`	`dockerService,`
`89`	`89`	`this,`