Slf4jJdbcLogger backward compatible

Author: backpaper0

.github/workflows/ci.yml

@@ -48,7 +48,7 @@ jobs:
     steps:
       - uses: actions/checkout@v2
 
-      - name: Set up JDK ${{ matrix.java }}
+      - name: Set up JDK 11
         uses: actions/setup-java@v2
         with:
           distribution: zulu
@@ -71,10 +71,38 @@ jobs:
           name: reports
           path: ./**/target/surefire-reports
 
+  test-slf4j-backward-compatible:
+    if: contains(github.event.head_commit.message, '[skip ci]') == false
+    runs-on: ubuntu-latest
+    timeout-minutes: 30
+
+    steps:
+      - uses: actions/checkout@v2
+
+      - name: Set up JDK 8
+        uses: actions/setup-java@v2
+        with:
+          distribution: zulu
+          java-version: 8
+          cache: maven
+
+      - name: Grant execute permission for mvnw
+        run: chmod +x mvnw
+
+      - name: Test with Maven
+        run: ./mvnw -B compile -pl doma-spring-boot-core,doma-spring-boot-autoconfigure && ./mvnw -B test -pl doma-spring-boot-core,doma-spring-boot-autoconfigure -Dmaven.main.skip=true -Ddoma.version=4.49.0
+
+      - name: Upload reports
+        if: failure()
+        uses: actions/upload-artifact@v2
+        with:
+          name: reports
+          path: ./**/target/surefire-reports
+
   deploy:
     if: github.event_name == 'push' && contains(github.event.head_commit.message, '[skip ci]') == false
     runs-on: ubuntu-latest
-    needs: [ test, test-module ]
+    needs: [ test, test-module, test-slf4j-backward-compatible ]
 
     steps:
       - uses: actions/checkout@v2

doma-spring-boot-autoconfigure/src/main/java/org/seasar/doma/boot/autoconfigure/DomaProperties.java

@@ -4,6 +4,8 @@
 
 import java.util.function.Supplier;
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 import org.seasar.doma.jdbc.*;
 import org.seasar.doma.jdbc.dialect.*;
 import org.seasar.doma.slf4j.Slf4jJdbcLogger;
@@ -228,7 +230,24 @@ public Naming naming() {
 
 	public static enum JdbcLoggerType {
 		JUL(UtilLoggingJdbcLogger::new),
-		SLF4J(() -> new Slf4jJdbcLogger());
+		SLF4J(JdbcLoggerType::slf4jJdbcLogger);
+
+		private static JdbcLogger slf4jJdbcLogger() {
+			try {
+				return new Slf4jJdbcLogger();
+			} catch (NoClassDefFoundError e) {
+				Log logger = LogFactory.getLog(JdbcLoggerType.class);
+				logger.info(
+						"org.seasar.doma.slf4j.Slf4jJdbcLogger is not found, fallback to org.seasar.doma.jdbc.Slf4jJdbcLogger");
+				try {
+					return (JdbcLogger) Class.forName("org.seasar.doma.jdbc.Slf4jJdbcLogger")
+							.getConstructor().newInstance();
+				} catch (ReflectiveOperationException roe) {
+					logger.warn("org.seasar.doma.jdbc.Slf4jJdbcLogger could not be instantiated either.", roe);
+				}
+				throw e;
+			}
+		}
 
 		private final Supplier<JdbcLogger> constructor;