updated log4j library from 1.x to 2.x version

Author: ibalosh

pom.xml

@@ -5,7 +5,6 @@
     <modelVersion>4.0.0</modelVersion>
 
     <!-- base library details -->
-
     <groupId>com.wildbit.java</groupId>
     <artifactId>postmark</artifactId>
     <version>${postmark.version}</version>
@@ -96,6 +95,18 @@
             <version>2.1.3</version>
         </dependency>
 
+        <!-- Logging dependencies -->
+        <dependency>
+            <groupId>org.apache.logging.log4j</groupId>
+            <artifactId>log4j-api</artifactId>
+            <version>2.9.1</version>
+        </dependency>
+        <dependency>
+            <groupId>org.apache.logging.log4j</groupId>
+            <artifactId>log4j-core</artifactId>
+            <version>2.9.1</version>
+        </dependency>
+
         <!-- Testing dependencies -->
         <dependency>
             <groupId>org.junit.jupiter</groupId>
@@ -125,13 +136,6 @@
             <version>2.11.0</version>
             <scope>test</scope>
         </dependency>
-
-        <!-- https://mvnrepository.com/artifact/log4j/log4j -->
-        <dependency>
-            <groupId>log4j</groupId>
-            <artifactId>log4j</artifactId>
-            <version>1.2.17</version>
-        </dependency>
     </dependencies>
 
     <!-- all of the below is needed for releasing -->

src/main/java/com/wildbit/java/postmark/Postmark.java

@@ -2,13 +2,14 @@
 
 import com.wildbit.java.postmark.client.AccountApiClient;
 import com.wildbit.java.postmark.client.ApiClient;
-import org.apache.log4j.Logger;
 
 import javax.ws.rs.core.MultivaluedHashMap;
 import javax.ws.rs.core.MultivaluedMap;
 import java.io.InputStream;
 import java.util.Properties;
 
+import org.apache.logging.log4j.LogManager;
+import org.apache.logging.log4j.Logger;
 /**
  * Top level class allowing simple access to API clients for doing Postmark API calls.
  */
@@ -92,7 +93,7 @@ public static AccountApiClient getAccountApiClient(String apiToken, Boolean secu
 
     private Postmark() {}
 
-    private static Logger log = Logger.getLogger(Postmark.class);
+    private static Logger log = LogManager.getLogger(Postmark.class);
 
     private static MultivaluedMap<String,Object> getHeadersWithAuth(DEFAULTS authType, String apiToken) {
         MultivaluedMap headers = DefaultHeaders.headers();

src/main/java/com/wildbit/java/postmark/client/HttpClient.java

@@ -71,7 +71,7 @@ public ClientResponse execute(REQUEST_TYPES requestType, String url, String data
                 break;
 
             case PATCH:
-                // client doesn't have PATCH method, therefore workaround has to be used
+                // Jersey client doesn't have PATCH method, therefore workaround has to be used by using reflection
                 // https://stackoverflow.com/questions/55778145/how-to-use-patch-method-with-jersey-invocation-builder
                 target.property(HttpUrlConnectorProvider.SET_METHOD_WORKAROUND, true);
                 response = target.request().headers(headers).method("PATCH", Entity.json(data), Response.class);

src/main/java/com/wildbit/java/postmark/client/HttpClientHandler.java

@@ -2,11 +2,14 @@
 
 import com.wildbit.java.postmark.client.data.DataHandler;
 import com.wildbit.java.postmark.client.exception.*;
-import org.apache.log4j.spi.RootLogger;
+import org.apache.logging.log4j.LogManager;
+import org.apache.logging.log4j.Level;
+import org.apache.logging.log4j.core.LoggerContext;
+import org.apache.logging.log4j.core.config.Configuration;
+import org.apache.logging.log4j.core.config.LoggerConfig;
 
 import javax.ws.rs.core.MultivaluedMap;
 import java.io.IOException;
-import org.apache.log4j.Level;
 
 /**
  * Client class acts as handler between HTTP requests handler class (HttpClient) and class which provides access to all endpoints to call.
@@ -86,7 +89,12 @@ public void setDebugMode() {
      * @param level - debugging level - OFF, DEBUG, ERROR etc.
      */
     public void setLoggingLevel(Level level) {
-        RootLogger.getRootLogger().setLevel(level);
+        LoggerContext ctx = (LoggerContext) LogManager.getContext(false);
+        Configuration config = ctx.getConfiguration();
+
+        LoggerConfig loggerConfig = config.getLoggerConfig(LogManager.ROOT_LOGGER_NAME);
+        loggerConfig.setLevel(level);
+        ctx.updateLoggers();
     }
 
     public void setSecureConnection(boolean secureConnection) {

src/main/resources/log4j.properties

@@ -0,0 +1,17 @@
+name=PropertiesConfig
+appenders = console
+
+appender.console.type = Console
+appender.console.name = STDOUT
+appender.console.layout.type = PatternLayout
+appender.console.layout.pattern = [%-5level] %d{yyyy-MM-dd HH:mm:ss.SSS} [%t] %c{1} - %msg%n
+
+loggers=console
+logger.console.name = postmark.java
+logger.console.level = debug
+logger.console.appenderRef.console.ref = STDOUT
+logger.rolling.level = info
+
+rootLogger.level = off
+rootLogger.appenderRefs = stdout
+rootLogger.appenderRef.stdout.ref = STDOUT