yahoo / imapnio

Java imap nio client that is designed to scale well for thousands of connections per machine and reduce contention when using large number of threads and cpus.
57 stars 50 forks source link

ISSUE-145 Remove logback from main dependencies #146

Closed chibenwa closed 1 year ago

chibenwa commented 1 year ago

Keep it as a testing dependency

Description

java.lang.NoSuchMethodError: 'java.lang.String ch.qos.logback.core.util.EnvUtil.logbackVersion()'

    at ch.qos.logback.classic.util.ContextInitializer.autoConfig(ContextInitializer.java:81)
    at ch.qos.logback.classic.util.ContextInitializer.autoConfig(ContextInitializer.java:77)
    at ch.qos.logback.classic.spi.LogbackServiceProvider.initializeLoggerContext(LogbackServiceProvider.java:50)
    at ch.qos.logback.classic.spi.LogbackServiceProvider.initialize(LogbackServiceProvider.java:41)
    at org.slf4j.LoggerFactory.bind(LoggerFactory.java:183)
    at org.slf4j.LoggerFactory.performInitialization(LoggerFactory.java:170)
    at org.slf4j.LoggerFactory.getProvider(LoggerFactory.java:455)
    at org.slf4j.LoggerFactory.getILoggerFactory(LoggerFactory.java:441)
    at org.slf4j.LoggerFactory.getLogger(LoggerFactory.java:390)
    at org.slf4j.LoggerFactory.getLogger(LoggerFactory.java:416)
    at org.apache.james.mock.smtp.server.testing.MockSmtpServerExtension$DockerMockSmtp.<clinit>(MockSmtpServerExtension.java:43)
    at org.apache.james.mock.smtp.server.testing.MockSmtpServerExtension.<init>(MockSmtpServerExtension.java:77)
    at org.apache.james.mailets.RemoteDeliveryForwardIntegrationTest.<clinit>(RemoteDeliveryForwardIntegrationTest.java:76)

Motivation and Context

SLF4J should be enough (logging facade)

How Has This Been Tested?

Existing tests passes

Screenshots (if appropriate):

Types of changes

Checklist:

chibenwa commented 1 year ago

Hello @fansu

What's the ETA of this?