Merge pull request #1 from SentryMan/main

Default to System Logger on Error

Author: rbygrave

.github/dependabot.yml

@@ -0,0 +1,29 @@
+version: 2
+updates:
+  - package-ecosystem: "maven"
+    directory: "/"
+    schedule:
+      interval: weekly
+    open-pull-requests-limit: 10
+    groups:
+      dependencies:
+        patterns:
+        - "*"
+    labels:
+      - "dependencies"
+    target-branch: "master"
+
+  - package-ecosystem: "github-actions"
+    directory: "/"
+    schedule:
+      interval: "weekly"
+    open-pull-requests-limit: 5
+    commit-message:
+      prefix: "[workflow]"
+    labels:
+      - "dependencies"
+    target-branch: "master"
+    groups:
+      dependencies:
+        patterns:
+        - "*"

.github/workflows/build.yml

@@ -5,7 +5,6 @@ on: [push, pull_request]
 
 jobs:
   build:
-
     runs-on: ${{ matrix.os }}
     permissions:
       contents: read

.github/workflows/dependabot-merge.yml

@@ -0,0 +1,28 @@
+name: Dependabot auto-merge
+on: pull_request
+
+permissions:
+  contents: write
+  pull-requests: write
+
+jobs:
+  dependabot:
+    runs-on: ubuntu-latest
+    if: ${{ github.actor == 'dependabot[bot]' }}
+    steps:
+      - name: Dependabot metadata
+        id: metadata
+        uses: dependabot/fetch-metadata@v2
+        with:
+          github-token: "${{ secrets.GITHUB_TOKEN }}"
+      - name: Approve a PR
+        run: gh pr review --approve "$PR_URL"
+        env:
+          PR_URL: ${{github.event.pull_request.html_url}}
+          GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}
+      # Enable for automerge
+      - name: Enable auto-merge for Dependabot PRs
+        run: gh pr merge --auto --merge "$PR_URL"
+        env:
+          PR_URL: ${{github.event.pull_request.html_url}}
+          GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}

.gitignore

@@ -1,4 +1,11 @@
 .idea/
 *.iml
 target/
-
+build/
+.gradle
+*.prefs
+*.class*
+*.factorypath
+*.project
+*.processors
+*/bin/

pom.xml

@@ -18,6 +18,7 @@
 
   <properties>
     <surefire.useModulePath>false</surefire.useModulePath>
+    <project.build.outputTimestamp>2025-02-26T05:43:52Z</project.build.outputTimestamp>
   </properties>
 
   <dependencies>
@@ -27,7 +28,6 @@
       <version>1.1</version>
       <scope>test</scope>
     </dependency>
-
   </dependencies>
 
 </project>

src/main/java/io/avaje/applog/AppLog.java

@@ -1,6 +1,8 @@
 package io.avaje.applog;
 
+import java.lang.System.Logger.Level;
 import java.util.ResourceBundle;
+import java.util.ServiceConfigurationError;
 import java.util.ServiceLoader;
 
 /**
@@ -24,8 +26,6 @@
  * {@link AppLog.Provider} to provide System.Logger implementations returned by AppLog.
  * The reason to use AppLog is that it provides this 1 extra level of indirection that
  * gives applications control over the System.Logger implementation returned by AppLog.
- *
- *
  * <hr/>
  *
  * <h2>Main differences to slf4j-api</h2>
@@ -77,14 +77,18 @@
  */
 public final class AppLog {
 
-  private static final Provider provider = ServiceLoader.load(Provider.class)
-    .findFirst()
-    .orElseGet(DefaultProvider::new);
+  private static final Provider provider = loadProvider();
 
-  /**
-   * Not accessible.
-   */
-  private AppLog() {
+  private AppLog() {}
+
+  private static Provider loadProvider() {
+    try {
+      return ServiceLoader.load(Provider.class).findFirst().orElseGet(DefaultProvider::new);
+    } catch (ServiceConfigurationError e) {
+      getLogger(AppLog.class.getCanonicalName())
+          .log(Level.ERROR, "Failed to Service Load AppLog Provider, using System.Logger implementation", e);
+      return new DefaultProvider();
+    }
   }
 
   /**