Merge pull request #21 from ChristophEichhorn/performanceIssue

Performance issue

Author: altenhof

README.md

@@ -27,11 +27,11 @@ All in all, you should do the following:
 * adjust your logging configuration accordingly.
 
 
-Say, you want to make use of the *servlet filter* feature, then you need to add the following dependency to your POM with property `cf-logging-version` referring to the latest nexus version (currently `2.1.3`):
+Say, you want to make use of the *servlet filter* feature, then you need to add the following dependency to your POM with property `cf-logging-version` referring to the latest nexus version (currently `2.1.4`):
 
 ```xml
 <properties>
-	<cf-logging-version>2.1.3</cf-logging-version>
+	<cf-logging-version>2.1.4</cf-logging-version>
 </properties>
 ```
 

cf-java-logging-support-core/pom.xml

@@ -32,7 +32,7 @@
     <parent>
         <groupId>com.sap.hcp.cf.logging</groupId>
         <artifactId>cf-java-logging-support-parent</artifactId>
-        <version>2.1.3</version>
+        <version>2.1.4</version>
         <relativePath>../pom.xml</relativePath>
     </parent>
     <build>

cf-java-logging-support-jersey/pom.xml

@@ -9,7 +9,7 @@
         <relativePath>../pom.xml</relativePath>
         <groupId>com.sap.hcp.cf.logging</groupId>
         <artifactId>cf-java-logging-support-parent</artifactId>
-        <version>2.1.3</version>
+        <version>2.1.4</version>
     </parent>
 
     <name>cf-java-logging-support-jersey</name>

cf-java-logging-support-jersey/src/main/java/com/sap/hcp/cf/logging/jersey/filter/ResponseHandler.java

@@ -19,7 +19,7 @@ public void handle(ResponseContextAdapter responseContext, RequestRecord rr) {
             rr.addTag(Fields.RESPONSE_CONTENT_TYPE, responseContext.getHeader(HttpHeaders.CONTENT_TYPE));
             rr.addValue(Fields.RESPONSE_STATUS, new LongValue(responseContext.getStatus()));
             rr.stop();
-            LOGGER.info(Markers.REQUEST_MARKER, rr.toString());
+            LOGGER.info(Markers.REQUEST_MARKER, "{}", rr);
             /*
              * -- close this instance
              */

cf-java-logging-support-log4j2/pom.xml

@@ -11,7 +11,7 @@
         <relativePath>../pom.xml</relativePath>
         <groupId>com.sap.hcp.cf.logging</groupId>
         <artifactId>cf-java-logging-support-parent</artifactId>
-        <version>2.1.3</version>
+        <version>2.1.4</version>
     </parent>
 
     <dependencies>

cf-java-logging-support-logback/pom.xml

@@ -10,7 +10,7 @@
         <relativePath>../pom.xml</relativePath>
         <groupId>com.sap.hcp.cf.logging</groupId>
         <artifactId>cf-java-logging-support-parent</artifactId>
-        <version>2.1.3</version>
+        <version>2.1.4</version>
     </parent>
 
     <dependencies>

cf-java-logging-support-servlet/pom.xml

@@ -9,7 +9,7 @@
     <parent>
         <groupId>com.sap.hcp.cf.logging</groupId>
         <artifactId>cf-java-logging-support-parent</artifactId>
-        <version>2.1.3</version>
+        <version>2.1.4</version>
         <relativePath>../pom.xml</relativePath>
     </parent>
 

cf-java-logging-support-servlet/src/main/java/com/sap/hcp/cf/logging/servlet/filter/RequestLoggingFilter.java

@@ -143,7 +143,7 @@ private void doFilterRequest(HttpServletRequest httpRequest, HttpServletResponse
             /*
              * -- log info
              */
-            LOGGER.info(Markers.REQUEST_MARKER, rr.toString());
+            LOGGER.info(Markers.REQUEST_MARKER, "{}", rr);
             /*
              * -- close this
              */

pom.xml

@@ -4,7 +4,7 @@
 
     <groupId>com.sap.hcp.cf.logging</groupId>
     <artifactId>cf-java-logging-support-parent</artifactId>
-    <version>2.1.3</version>
+    <version>2.1.4</version>
     <packaging>pom</packaging>
 
     <name>Cloud Foundry Java logging support components</name>

sample/manifest.yml

@@ -6,7 +6,7 @@ applications:
 - name: logging-sample-app
   host: logging-sample-app
   instances: 1
-  path: target/logging-sample-app-2.1.3.war
+  path: target/logging-sample-app-2.1.4.war
   env:
     RANDOM_SLEEP: true
 # Set LOG_*: true to activate logging of respective field