prevent string concatenation for disabled loggers

1 files changed, 4 insertions, 3 deletions

diff --git a/juick-api/src/main/java/com/juick/api/controllers/Post.java b/juick-api/src/main/java/com/juick/api/controllers/Post.java
index 18f417d8..0fbf87e8 100644
--- a/juick-api/src/main/java/com/juick/api/controllers/Post.java
+++ b/juick-api/src/main/java/com/juick/api/controllers/Post.java
@@ -99,7 +99,7 @@ public class Post {
         }
 
         String attachmentFName = HttpUtils.receiveMultiPartFile(attach, env.getProperty("upload_tmp_dir",
-                    "/var/www/juick.com/i/tmp/"));
+                "/var/www/juick.com/i/tmp/"));
 
         if (StringUtils.isBlank(attachmentFName) && img != null && img.length() > 10) {
             try {
@@ -142,7 +142,7 @@ public class Post {
                     OobX xoob = new OobX(new URI(attachmentURL));
                     xmsg.addExtension(xoob);
                 } catch (URISyntaxException e) {
-                    logger.error("invalid uri: " + attachmentURL, e);
+                    logger.error("invalid uri: {}, exception {}", attachmentURL, e);
                 }
             }
 
@@ -173,6 +173,7 @@ public class Post {
         }
         return jmsg;
     }
+
     @RequestMapping(value = "/comment", method = RequestMethod.GET, produces = MediaType.APPLICATION_JSON_UTF8_VALUE)
     public com.juick.Message doPostComment(
             Principal principal,
@@ -263,7 +264,7 @@ public class Post {
                 try {
                     xmsg.addExtension(new OobX(new URI(attachmentURL)));
                 } catch (URISyntaxException e) {
-                    logger.error("invalid uri: " + attachmentURL, e);
+                    logger.error("invalid uri: {}, exception {}", attachmentURL, e);
                 }
             }