QA: Add log level guards for logger.error statements

.mvn/wrapper/MavenWrapperDownloader.java

@@ -18,8 +18,11 @@
 import java.nio.channels.*;
 import java.util.Properties;
 
-public class MavenWrapperDownloader {
+public final class MavenWrapperDownloader {
 
+    private MavenWrapperDownloader() {
+        throw new UnsupportedOperationException("Utility class");
+    }
     private static final String WRAPPER_VERSION = "0.5.6";
     /**
      * Default URL to download the maven-wrapper.jar from, if no 'downloadUrl' is provided.
@@ -106,12 +109,10 @@ protected PasswordAuthentication getPasswordAuthentication() {
             });
         }
         URL website = new URL(urlString);
-        ReadableByteChannel rbc;
-        rbc = Channels.newChannel(website.openStream());
-        FileOutputStream fos = new FileOutputStream(destination);
-        fos.getChannel().transferFrom(rbc, 0, Long.MAX_VALUE);
-        fos.close();
-        rbc.close();
-    }
 
-}
+        try (ReadableByteChannel rbc = Channels.newChannel(website.openStream());
+             FileOutputStream fos = new FileOutputStream(destination)) {
+            fos.getChannel().transferFrom(rbc, 0, Long.MAX_VALUE);
+        }
+
+}}

src/main/java/com/bezkoder/spring/login/security/jwt/AuthEntryPointJwt.java

@@ -25,7 +25,8 @@ public class AuthEntryPointJwt implements AuthenticationEntryPoint {
   @Override
   public void commence(HttpServletRequest request, HttpServletResponse response, AuthenticationException authException)
       throws IOException, ServletException {
-    logger.error("Unauthorized error: {}", authException.getMessage());
+    if (logger.isErrorEnabled()) {
+       logger.error("Unauthorized error: {}", authException.getMessage());
 
     response.setContentType(MediaType.APPLICATION_JSON_VALUE);
     response.setStatus(HttpServletResponse.SC_UNAUTHORIZED);
@@ -39,5 +40,5 @@ public void commence(HttpServletRequest request, HttpServletResponse response, A
     final ObjectMapper mapper = new ObjectMapper();
     mapper.writeValue(response.getOutputStream(), body);
   }
-
+      }
 }

src/main/java/com/bezkoder/spring/login/security/jwt/JwtUtils.java

@@ -56,26 +56,38 @@ public String getUserNameFromJwtToken(String token) {
         .parseClaimsJws(token).getBody().getSubject();
   }
 
-  private Key key() {
+private Key key() {
     return Keys.hmacShaKeyFor(Decoders.BASE64.decode(jwtSecret));
   }
+  private void logJwtError(String message, Exception e) {
+    if (logger.isErrorEnabled()) {
+        logger.error(message, e.getMessage());
+    }
+}
 
   public boolean validateJwtToken(String authToken) {
     try {
-      Jwts.parserBuilder().setSigningKey(key()).build().parse(authToken);
-      return true;
+        Jwts.parserBuilder()
+            .setSigningKey(key())
+            .build()
+            .parse(authToken);
+        return true;
+
     } catch (MalformedJwtException e) {
-      logger.error("Invalid JWT token: {}", e.getMessage());
+        logJwtError("Invalid JWT token: {}", e);
+
     } catch (ExpiredJwtException e) {
-      logger.error("JWT token is expired: {}", e.getMessage());
+        logJwtError("JWT token is expired: {}", e);
+
     } catch (UnsupportedJwtException e) {
-      logger.error("JWT token is unsupported: {}", e.getMessage());
+        logJwtError("JWT token is unsupported: {}", e);
+
     } catch (IllegalArgumentException e) {
-      logger.error("JWT claims string is empty: {}", e.getMessage());
+        logJwtError("JWT claims string is empty: {}", e);
     }
 
     return false;
-  }
+}
 
   public String generateTokenFromUsername(String username) {   
     return Jwts.builder()
@@ -85,4 +97,4 @@ public String generateTokenFromUsername(String username) {
               .signWith(key(), SignatureAlgorithm.HS256)
               .compact();
   }
-}
+  }