From 8dbcda0ca431f5340ae1a5b10f09f888fe9fd7bc Mon Sep 17 00:00:00 2001
From: Matthew Horridge <matthew.horridge@stanford.edu>
Date: Tue, 10 Dec 2024 14:17:49 -0800
Subject: [PATCH] Fixes #10

---
 .../webprotege/ipc/impl/RabbitMqCommandHandlerWrapper.java | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/main/java/edu/stanford/protege/webprotege/ipc/impl/RabbitMqCommandHandlerWrapper.java b/src/main/java/edu/stanford/protege/webprotege/ipc/impl/RabbitMqCommandHandlerWrapper.java
index e722a99..4667629 100644
--- a/src/main/java/edu/stanford/protege/webprotege/ipc/impl/RabbitMqCommandHandlerWrapper.java
+++ b/src/main/java/edu/stanford/protege/webprotege/ipc/impl/RabbitMqCommandHandlerWrapper.java
@@ -175,7 +175,7 @@ private void handleAndReplyToRequest(CommandHandler<Q,R> handler, Channel channe
             response.subscribe(r -> {
                 replyWithSuccessResponse(channel, message, userId, r);
                 long endtime = System.currentTimeMillis();
-                logger.info("Request executed " + request.getChannel() + ". Time taken for Execution is : " + (endtime-startTime) +"ms");
+                logger.info("Request executed {}. Time taken for Execution is : {}ms", request.getChannel(), endtime - startTime);
 
             }, throwable -> {
                 if (throwable instanceof CommandExecutionException ex) {
@@ -186,13 +186,12 @@ private void handleAndReplyToRequest(CommandHandler<Q,R> handler, Channel channe
                             request);
                     replyWithErrorResponse(message,channel, userId, ex.getStatus());
                     long endtime = System.currentTimeMillis();
-                    logger.info("Request failed " + request.getChannel() + "with error " + throwable.getMessage() + ". Time taken for Execution is : " + (endtime-startTime) +"ms");
+                    logger.info("Request failed {}with error {}. Time taken for Execution is : {}ms", request.getChannel(), throwable.getMessage(), endtime - startTime);
                 }
                 else {
                     replyWithErrorResponse(message, channel, userId, HttpStatus.INTERNAL_SERVER_ERROR);
                     long endtime = System.currentTimeMillis();
-
-                    logger.info("Request failed " + request.getChannel() + "with error " + throwable.getMessage() + ". Time taken for Execution is : " + (endtime-startTime) +"ms");
+                    logger.info("Request failed {}with error {}. Time taken for Execution is : {}ms", request.getChannel(), throwable.getMessage(), endtime - startTime);
 
                 }
             });