Skip to content

Change the default logger from java.util.logging to SLF4J #288

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Apr 27, 2025

Conversation

backpaper0
Copy link
Member

@backpaper0 backpaper0 commented Apr 24, 2025

Impact on users

  • Users who were using the default setting (JUL):
    • It will be switched to SLF4J.
    • If they wish to continue using JUL, they will need to add explicit configuration.
      doma.jdbc-logger=JUL
      
  • Users who were explicitly using JUL:
    • org.seasar.doma:doma-slf4j will be included as a dependency.
    • To exclude it, they need to explicitly add an exclusion in pom.xml:
      <dependency>
          <groupId>org.seasar.doma</groupId>
          <artifactId>doma-spring-boot</artifactId>
          <version>x.y.z</version>
          <exclusions>
              <exclusion>
                  <groupId>org.seasar.doma</groupId>
                  <artifactId>doma-slf4j</artifactId>
              </exclusion>
          </exclusions>
      </dependency>
  • Users who were using SLF4J:
    • No impact.
    • Explicit configuration is no longer necessary.

Related Issue

closes gh-287

Others

The "Configuration" page on the wiki needs to be updated.

Verified

This commit was signed with the committer’s verified signature. The key has expired.
backpaper0 Uragami Taichi

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@backpaper0 backpaper0 merged commit 6794f92 into master Apr 27, 2025
6 checks passed
@backpaper0 backpaper0 deleted the slf4j-logger branch April 27, 2025 04:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Change the default logger to SLF4J
1 participant