perf($WebLog): trim printed JSON string

johnnymillergh · Mar 9, 2020 · 48e7984 · 48e7984
1 parent 2f1f63a
commit 48e7984
Show file tree

Hide file tree

Showing 7 changed files with 30 additions and 12 deletions.
diff --git a/common/src/main/java/com/jmsoftware/common/domain/.empty b/common/src/main/java/com/jmsoftware/common/domain/.empty
diff --git a/demo-one/src/main/java/com/jmsoftware/demoone/universal/aspect/WebRequestLogAspect.java b/demo-one/src/main/java/com/jmsoftware/demoone/universal/aspect/WebRequestLogAspect.java
@@ -84,8 +84,11 @@ public Object aroundHandleRequest(ProceedingJoinPoint proceedingJoinPoint) throw
         Object result = proceedingJoinPoint.proceed();
         long elapsedTime = System.currentTimeMillis() - startTime;
         try {
-            log.info("Response           :{}{}", LINE_SEPARATOR,
-                     JSONUtil.formatJsonStr(mapper.writeValueAsString(result)));
+            var formattedStringifiedJson = JSONUtil.formatJsonStr(mapper.writeValueAsString(result));
+            if (formattedStringifiedJson.length() > 500) {
+                formattedStringifiedJson = formattedStringifiedJson.substring(0, 499).concat("…");
+            }
+            log.info("Response           :{}{}", LINE_SEPARATOR, formattedStringifiedJson);
         } catch (JsonProcessingException e) {
             log.info("Response (non-JSON): {}", result);
         }

diff --git a/demo-two/src/main/java/com/jmsoftware/demotwo/universal/aspect/WebRequestLogAspect.java b/demo-two/src/main/java/com/jmsoftware/demotwo/universal/aspect/WebRequestLogAspect.java
@@ -84,8 +84,11 @@ public Object aroundHandleRequest(ProceedingJoinPoint proceedingJoinPoint) throw
         Object result = proceedingJoinPoint.proceed();
         long elapsedTime = System.currentTimeMillis() - startTime;
         try {
-            log.info("Response           :{}{}", LINE_SEPARATOR,
-                     JSONUtil.formatJsonStr(mapper.writeValueAsString(result)));
+            var formattedStringifiedJson = JSONUtil.formatJsonStr(mapper.writeValueAsString(result));
+            if (formattedStringifiedJson.length() > 500) {
+                formattedStringifiedJson = formattedStringifiedJson.substring(0, 499).concat("…");
+            }
+            log.info("Response           :{}{}", LINE_SEPARATOR, formattedStringifiedJson);
         } catch (JsonProcessingException e) {
             log.info("Response (non-JSON): {}", result);
         }

diff --git a/...se-mis/src/main/java/com/jmsoftware/exercisemis/universal/aspect/WebRequestLogAspect.java b/...se-mis/src/main/java/com/jmsoftware/exercisemis/universal/aspect/WebRequestLogAspect.java
@@ -84,8 +84,11 @@ public Object aroundHandleRequest(ProceedingJoinPoint proceedingJoinPoint) throw
         Object result = proceedingJoinPoint.proceed();
         long elapsedTime = System.currentTimeMillis() - startTime;
         try {
-            log.info("Response           :{}{}", LINE_SEPARATOR,
-                     JSONUtil.formatJsonStr(mapper.writeValueAsString(result)));
+            var formattedStringifiedJson = JSONUtil.formatJsonStr(mapper.writeValueAsString(result));
+            if (formattedStringifiedJson.length() > 500) {
+                formattedStringifiedJson = formattedStringifiedJson.substring(0, 499).concat("…");
+            }
+            log.info("Response           :{}{}", LINE_SEPARATOR, formattedStringifiedJson);
         } catch (JsonProcessingException e) {
             log.info("Response (non-JSON): {}", result);
         }

diff --git a/...le-mis/src/main/java/com/jmsoftware/exercisemis/universal/aspect/WebRequestLogAspect.java b/...le-mis/src/main/java/com/jmsoftware/exercisemis/universal/aspect/WebRequestLogAspect.java
@@ -84,8 +84,11 @@ public Object aroundHandleRequest(ProceedingJoinPoint proceedingJoinPoint) throw
         Object result = proceedingJoinPoint.proceed();
         long elapsedTime = System.currentTimeMillis() - startTime;
         try {
-            log.info("Response           :{}{}", LINE_SEPARATOR,
-                     JSONUtil.formatJsonStr(mapper.writeValueAsString(result)));
+            var formattedStringifiedJson = JSONUtil.formatJsonStr(mapper.writeValueAsString(result));
+            if (formattedStringifiedJson.length() > 500) {
+                formattedStringifiedJson = formattedStringifiedJson.substring(0, 499).concat("…");
+            }
+            log.info("Response           :{}{}", LINE_SEPARATOR, formattedStringifiedJson);
         } catch (JsonProcessingException e) {
             log.info("Response (non-JSON): {}", result);
         }

diff --git a/...ry/src/main/java/com/jmsoftware/serviceregistry/universal/aspect/WebRequestLogAspect.java b/...ry/src/main/java/com/jmsoftware/serviceregistry/universal/aspect/WebRequestLogAspect.java
@@ -84,8 +84,11 @@ public Object aroundHandleRequest(ProceedingJoinPoint proceedingJoinPoint) throw
         Object result = proceedingJoinPoint.proceed();
         long elapsedTime = System.currentTimeMillis() - startTime;
         try {
-            log.info("Response           :{}{}", LINE_SEPARATOR,
-                     JSONUtil.formatJsonStr(mapper.writeValueAsString(result)));
+            var formattedStringifiedJson = JSONUtil.formatJsonStr(mapper.writeValueAsString(result));
+            if (formattedStringifiedJson.length() > 500) {
+                formattedStringifiedJson = formattedStringifiedJson.substring(0, 499).concat("…");
+            }
+            log.info("Response           :{}{}", LINE_SEPARATOR, formattedStringifiedJson);
         } catch (JsonProcessingException e) {
             log.info("Response (non-JSON): {}", result);
         }

diff --git a/...in/src/main/java/com/jmsoftware/springbootadmin/universal/aspect/WebRequestLogAspect.java b/...in/src/main/java/com/jmsoftware/springbootadmin/universal/aspect/WebRequestLogAspect.java
@@ -84,8 +84,11 @@ public Object aroundHandleRequest(ProceedingJoinPoint proceedingJoinPoint) throw
         Object result = proceedingJoinPoint.proceed();
         long elapsedTime = System.currentTimeMillis() - startTime;
         try {
-            log.info("Response           :{}{}", LINE_SEPARATOR,
-                     JSONUtil.formatJsonStr(mapper.writeValueAsString(result)));
+            var formattedStringifiedJson = JSONUtil.formatJsonStr(mapper.writeValueAsString(result));
+            if (formattedStringifiedJson.length() > 500) {
+                formattedStringifiedJson = formattedStringifiedJson.substring(0, 499).concat("…");
+            }
+            log.info("Response           :{}{}", LINE_SEPARATOR, formattedStringifiedJson);
         } catch (JsonProcessingException e) {
             log.info("Response (non-JSON): {}", result);
         }