Merge pull request jwtk#41 from josebarrueta/Issue-34

Issue-34 getting rid of slf4j.
svanscho · Aug 27, 2015 · 46a3c13 · 46a3c13
2 parents 6c2c90a + 5c66435
commit 46a3c13
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 29 deletions.
diff --git a/pom.xml b/pom.xml
@@ -61,7 +61,6 @@
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <buildNumber>${user.name}-${maven.build.timestamp}</buildNumber>
 
-        <slf4j.version>1.7.6</slf4j.version>
         <jackson.version>2.4.2</jackson.version>
 
         <!-- Optional Runtime Dependencies: -->
@@ -78,12 +77,6 @@
     </properties>
 
     <dependencies>
-
-        <dependency>
-            <groupId>org.slf4j</groupId>
-            <artifactId>slf4j-api</artifactId>
-            <version>${slf4j.version}</version>
-        </dependency>
         <dependency>
             <groupId>com.fasterxml.jackson.core</groupId>
             <artifactId>jackson-databind</artifactId>

diff --git a/src/main/java/io/jsonwebtoken/lang/Classes.java b/src/main/java/io/jsonwebtoken/lang/Classes.java
@@ -15,9 +15,6 @@
  */
 package io.jsonwebtoken.lang;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
 import java.io.InputStream;
 import java.lang.reflect.Constructor;
 
@@ -26,11 +23,6 @@
  */
 public class Classes {
 
-    /**
-     * Private internal log instance.
-     */
-    private static final Logger log = LoggerFactory.getLogger(Classes.class);
-
     /**
      * @since 0.1
      */
@@ -78,18 +70,10 @@ public static Class forName(String fqcn) throws UnknownClassException {
         Class clazz = THREAD_CL_ACCESSOR.loadClass(fqcn);
 
         if (clazz == null) {
-            if (log.isTraceEnabled()) {
-                log.trace("Unable to load class named [" + fqcn +
-                          "] from the thread context ClassLoader.  Trying the current ClassLoader...");
-            }
             clazz = CLASS_CL_ACCESSOR.loadClass(fqcn);
         }
 
         if (clazz == null) {
-            if (log.isTraceEnabled()) {
-                log.trace("Unable to load class named [" + fqcn + "] from the current ClassLoader.  " +
-                          "Trying the system/application ClassLoader...");
-            }
             clazz = SYSTEM_CL_ACCESSOR.loadClass(fqcn);
         }
 
@@ -212,9 +196,7 @@ public Class loadClass(String fqcn) {
                 try {
                     clazz = cl.loadClass(fqcn);
                 } catch (ClassNotFoundException e) {
-                    if (log.isTraceEnabled()) {
-                        log.trace("Unable to load clazz named [" + fqcn + "] from class loader [" + cl + "]");
-                    }
+                    //Class couldn't be found by loader
                 }
             }
             return clazz;
@@ -233,9 +215,7 @@ protected final ClassLoader getClassLoader() {
             try {
                 return doGetClassLoader();
             } catch (Throwable t) {
-                if (log.isDebugEnabled()) {
-                    log.debug("Unable to acquire ClassLoader.", t);
-                }
+                //Unable to get ClassLoader
             }
             return null;
         }