From 874a926bb3cbacfc466f240c9b0aee24ad9ac59c Mon Sep 17 00:00:00 2001
From: Kevin Viet <kevin.viet@gmail.com>
Date: Wed, 18 Oct 2023 05:04:47 +0200
Subject: [PATCH] fix: downgrade SLF4J to 1.7

Closes: #990
Co-authored-by: Gauthier Roebroeck <gauthier.roebroeck@gmail.com>
---
 Makefile                                      |  2 +-
 Makefile.common                               |  2 +-
 pom.xml                                       |  2 +-
 src/main/java/org/sqlite/JDBC.java            |  2 +-
 .../java/org/sqlite/SQLiteJDBCLoader.java     | 30 ++++++++-----------
 src/main/java/org/sqlite/core/NativeDB.java   | 18 +++++------
 .../sqlite/jdbc3/JDBC3DatabaseMetaData.java   |  6 ++--
 .../java/org/sqlite/jdbc3/JDBC3Statement.java |  2 +-
 src/main/java/org/sqlite/util/OSInfo.java     |  8 ++---
 9 files changed, 31 insertions(+), 41 deletions(-)

diff --git a/Makefile b/Makefile
index 5bf5df041c..0315b6585b 100644
--- a/Makefile
+++ b/Makefile
@@ -45,7 +45,7 @@ $(SQLITE_UNPACKED): $(SQLITE_ARCHIVE)
 
 $(JAVA_CLASSPATH):
 	@mkdir -p $(@D)
-	curl -L -f -o$@ https://search.maven.org/remotecontent?filepath=org/slf4j/slf4j-api/2.0.9/slf4j-api-2.0.9.jar
+	curl -L -f -o$@ https://search.maven.org/remotecontent?filepath=org/slf4j/slf4j-api/1.7.36/slf4j-api-1.7.36.jar
 
 $(TARGET)/common-lib/org/sqlite/%.class: src/main/java/org/sqlite/%.java
 	@mkdir -p $(@D)
diff --git a/Makefile.common b/Makefile.common
index e78ce60c19..c435e7cebb 100644
--- a/Makefile.common
+++ b/Makefile.common
@@ -18,7 +18,7 @@ JAVA_CLASSPATH:=$(TARGET)/classpath/slf4j-api.jar
 ifeq ("$(wildcard $(OSINFO_PROG))","")
 $(info Building OSInfo tool)
 $(shell mkdir -p $(TARGET)/classpath)
-$(shell curl -L -f -o$(JAVA_CLASSPATH) https://search.maven.org/remotecontent?filepath=org/slf4j/slf4j-api/2.0.9/slf4j-api-2.0.9.jar)
+$(shell curl -L -f -o$(JAVA_CLASSPATH) https://search.maven.org/remotecontent?filepath=org/slf4j/slf4j-api/1.7.36/slf4j-api-1.7.36.jar)
 $(shell $(JAVAC) -cp $(JAVA_CLASSPATH) -sourcepath $(SRC) -d lib src/main/java/org/sqlite/util/OSInfo.java)
 endif
 
diff --git a/pom.xml b/pom.xml
index b2f42ab0eb..f654d08981 100644
--- a/pom.xml
+++ b/pom.xml
@@ -408,7 +408,7 @@
         <dependency>
             <groupId>org.slf4j</groupId>
             <artifactId>slf4j-api</artifactId>
-            <version>2.0.9</version>
+            <version>1.7.36</version>
         </dependency>
         <!--
         This dependency makes compilation on non-GraalVM versions possible.
diff --git a/src/main/java/org/sqlite/JDBC.java b/src/main/java/org/sqlite/JDBC.java
index d449b166d0..42f4499f49 100644
--- a/src/main/java/org/sqlite/JDBC.java
+++ b/src/main/java/org/sqlite/JDBC.java
@@ -30,7 +30,7 @@ public class JDBC implements Driver {
         try {
             DriverManager.registerDriver(new JDBC());
         } catch (SQLException e) {
-            logger.atError().setCause(e).log();
+            logger.error("Could not register driver", e);
         }
     }
 
diff --git a/src/main/java/org/sqlite/SQLiteJDBCLoader.java b/src/main/java/org/sqlite/SQLiteJDBCLoader.java
index 2a5dd65cae..e054eedd3c 100644
--- a/src/main/java/org/sqlite/SQLiteJDBCLoader.java
+++ b/src/main/java/org/sqlite/SQLiteJDBCLoader.java
@@ -101,14 +101,12 @@ static void cleanup() {
                                     try {
                                         Files.delete(nativeLib);
                                     } catch (Exception e) {
-                                        logger.atError()
-                                                .setCause(e)
-                                                .log("Failed to delete old native lib");
+                                        logger.error("Failed to delete old native lib", e);
                                     }
                                 }
                             });
         } catch (IOException e) {
-            logger.atError().setCause(e).log("Failed to open directory");
+            logger.error("Failed to open directory", e);
         }
     }
 
@@ -224,7 +222,7 @@ private static boolean extractAndLoadLibraryFile(
             }
             return loadNativeLibrary(targetFolder, extractedLibFileName);
         } catch (IOException e) {
-            logger.atError().setCause(e).log();
+            logger.error("Unexpected IOException", e);
             return false;
         }
     }
@@ -248,7 +246,7 @@ private static InputStream getResourceAsStream(String name) {
             connection.setUseCaches(false);
             return connection.getInputStream();
         } catch (IOException e) {
-            logger.atError().setCause(e).log();
+            logger.error("Could not connect", e);
             return null;
         }
     }
@@ -268,12 +266,12 @@ private static boolean loadNativeLibrary(String path, String name) {
                 System.load(new File(path, name).getAbsolutePath());
                 return true;
             } catch (UnsatisfiedLinkError e) {
-                logger.atError()
-                        .setCause(e)
-                        .setMessage("Failed to load native library: {}. osinfo: {}")
-                        .addArgument(name)
-                        .addArgument(OSInfo::getNativeLibFolderPathForCurrentOS)
-                        .log();
+
+                logger.error(
+                        "Failed to load native library: {}. osinfo: {}",
+                        name,
+                        OSInfo.getNativeLibFolderPathForCurrentOS(),
+                        e);
                 return false;
             }
 
@@ -287,9 +285,7 @@ private static boolean loadNativeLibraryJdk() {
             System.loadLibrary(LibraryLoaderUtil.NATIVE_LIB_BASE_NAME);
             return true;
         } catch (UnsatisfiedLinkError e) {
-            logger.atError()
-                    .setCause(e)
-                    .log("Failed to load native library through System.loadLibrary");
+            logger.error("Failed to load native library through System.loadLibrary", e);
             return false;
         }
     }
@@ -422,9 +418,7 @@ public static final class VersionHolder {
                 // inline creation of logger to avoid build-time initialization of the logging
                 // framework in native-image
                 LoggerFactory.getLogger(VersionHolder.class)
-                        .atError()
-                        .setCause(e)
-                        .log("Could not read version from file: {}", versionFile);
+                        .error("Could not read version from file: {}", versionFile, e);
             }
             VERSION = version;
         }
diff --git a/src/main/java/org/sqlite/core/NativeDB.java b/src/main/java/org/sqlite/core/NativeDB.java
index 03b368ccf8..1e6cf73c4b 100644
--- a/src/main/java/org/sqlite/core/NativeDB.java
+++ b/src/main/java/org/sqlite/core/NativeDB.java
@@ -90,11 +90,10 @@ protected synchronized void _open(String file, int openFlags) throws SQLExceptio
     /** @see org.sqlite.core.DB#_exec(java.lang.String) */
     @Override
     public synchronized int _exec(String sql) throws SQLException {
-        logger.atTrace()
-                .setMessage("DriverManager [{}] [SQLite EXEC] {}")
-                .addArgument(() -> Thread.currentThread().getName())
-                .addArgument(sql)
-                .log();
+        if (logger.isTraceEnabled()) {
+            logger.trace(
+                    "DriverManager [{}] [SQLite EXEC] {}", Thread.currentThread().getName(), sql);
+        }
         return _exec_utf8(stringToUtf8ByteArray(sql));
     }
 
@@ -126,11 +125,10 @@ public synchronized int _exec(String sql) throws SQLException {
     /** @see org.sqlite.core.DB#prepare(java.lang.String) */
     @Override
     protected synchronized SafeStmtPtr prepare(String sql) throws SQLException {
-        logger.atTrace()
-                .setMessage("DriverManager [{}] [SQLite EXEC] {}")
-                .addArgument(() -> Thread.currentThread().getName())
-                .addArgument(sql)
-                .log();
+        if (logger.isTraceEnabled()) {
+            logger.trace(
+                    "DriverManager [{}] [SQLite EXEC] {}", Thread.currentThread().getName(), sql);
+        }
         return new SafeStmtPtr(this, prepare_utf8(stringToUtf8ByteArray(sql)));
     }
 
diff --git a/src/main/java/org/sqlite/jdbc3/JDBC3DatabaseMetaData.java b/src/main/java/org/sqlite/jdbc3/JDBC3DatabaseMetaData.java
index a062cbeeed..d91daa6612 100644
--- a/src/main/java/org/sqlite/jdbc3/JDBC3DatabaseMetaData.java
+++ b/src/main/java/org/sqlite/jdbc3/JDBC3DatabaseMetaData.java
@@ -964,14 +964,14 @@ public ResultSet getColumns(String c, String s, String tblNamePattern, String co
                         try {
                             rsColAutoinc.close();
                         } catch (Exception e) {
-                            LogHolder.logger.atError().setCause(e).log();
+                            LogHolder.logger.error("Could not close ResultSet", e);
                         }
                     }
                     if (statColAutoinc != null) {
                         try {
                             statColAutoinc.close();
                         } catch (Exception e) {
-                            LogHolder.logger.atError().setCause(e).log();
+                            LogHolder.logger.error("Could not close statement", e);
                         }
                     }
                 }
@@ -1122,7 +1122,7 @@ public ResultSet getColumns(String c, String s, String tblNamePattern, String co
                 try {
                     rs.close();
                 } catch (Exception e) {
-                    LogHolder.logger.atError().setCause(e).log();
+                    LogHolder.logger.error("Could not close ResultSet", e);
                 }
             }
         }
diff --git a/src/main/java/org/sqlite/jdbc3/JDBC3Statement.java b/src/main/java/org/sqlite/jdbc3/JDBC3Statement.java
index b8713d1b22..30210228a6 100644
--- a/src/main/java/org/sqlite/jdbc3/JDBC3Statement.java
+++ b/src/main/java/org/sqlite/jdbc3/JDBC3Statement.java
@@ -97,7 +97,7 @@ static class BackupObserver implements ProgressObserver {
         private static final Logger logger = LoggerFactory.getLogger(BackupObserver.class);
 
         public void progress(int remaining, int pageCount) {
-            logger.atInfo().log("remaining:{}, page count:{}", remaining, pageCount);
+            logger.info("remaining:{}, page count:{}", remaining, pageCount);
         }
     }
 
diff --git a/src/main/java/org/sqlite/util/OSInfo.java b/src/main/java/org/sqlite/util/OSInfo.java
index 926788a052..9af366ee73 100644
--- a/src/main/java/org/sqlite/util/OSInfo.java
+++ b/src/main/java/org/sqlite/util/OSInfo.java
@@ -160,7 +160,7 @@ static String getHardwareName() {
         try {
             return processRunner.runAndWaitFor("uname -m");
         } catch (Throwable e) {
-            LogHolder.logger.atError().setCause(e).log("Error while running uname -m");
+            LogHolder.logger.error("Error while running uname -m", e);
             return "unknown";
         }
     }
@@ -221,10 +221,8 @@ static String resolveArmArchType() {
                         return "armv7";
                     }
                 } else {
-                    LogHolder.logger
-                            .atWarn()
-                            .log(
-                                    "readelf not found. Cannot check if running on an armhf system, armel architecture will be presumed");
+                    LogHolder.logger.warn(
+                            "readelf not found. Cannot check if running on an armhf system, armel architecture will be presumed");
                 }
             } catch (IOException | InterruptedException e) {
                 // ignored: fall back to "arm" arch (soft-float ABI)