fix: Remove TempLoggerWrapper and use logback as the logging implementation. (#158)

Hi @kabir , as we discussed previously:
#94. This PR removes the
`TempLoggerWrapper` and uses `logback` as the default logging
implementation.

`logback` automatically loads the `logback.xml` file from the classpath,
and we can easily adjust the logging level by modifying this file.

Fixes #94

---------

Signed-off-by: Sun Yuhan <[email protected]>
Co-authored-by: Sun Yuhan <[email protected]>

Author: sunyuhan1998

pom.xml

@@ -29,6 +29,7 @@
         <quarkus.platform.version>3.22.3</quarkus.platform.version>
         <rest-assured.version>5.5.1</rest-assured.version>
         <slf4j.version>2.0.17</slf4j.version>
+        <logback.version>1.5.18</logback.version>
 
         <!-- Redirect test output to file -->
         <maven.test.redirectTestOutputToFile>true</maven.test.redirectTestOutputToFile>
@@ -111,6 +112,12 @@
                 <version>${mockserver.version}</version>
                 <scope>test</scope>
             </dependency>
+            <dependency>
+                <groupId>ch.qos.logback</groupId>
+                <artifactId>logback-classic</artifactId>
+                <version>${logback.version}</version>
+                <scope>test</scope>
+            </dependency>
         </dependencies>
     </dependencyManagement>
 

sdk-server-common/pom.xml

@@ -50,6 +50,24 @@
             <groupId>io.quarkus</groupId>
             <artifactId>quarkus-arc</artifactId>
             <scope>test</scope>
+            <exclusions>
+                <exclusion>
+                    <groupId>org.jboss.logging</groupId>
+                    <artifactId>jboss-logging</artifactId>
+                </exclusion>
+                <exclusion>
+                    <groupId>org.jboss.logmanager</groupId>
+                    <artifactId>jboss-logmanager</artifactId>
+                </exclusion>
+                <exclusion>
+                    <groupId>org.jboss.threads</groupId>
+                    <artifactId>jboss-threads</artifactId>
+                </exclusion>
+                <exclusion>
+                    <groupId>org.jboss.slf4j</groupId>
+                    <artifactId>slf4j-jboss-logmanager</artifactId>
+                </exclusion>
+            </exclusions>
         </dependency>
         <dependency>
             <groupId>org.junit.jupiter</groupId>
@@ -66,6 +84,11 @@
             <artifactId>mockserver-netty</artifactId>
             <scope>test</scope>
         </dependency>
+        <dependency>
+            <groupId>ch.qos.logback</groupId>
+            <artifactId>logback-classic</artifactId>
+            <scope>test</scope>
+        </dependency>
     </dependencies>
 
 </project>

sdk-server-common/src/main/java/io/a2a/server/events/EventQueue.java

@@ -10,14 +10,13 @@
 import java.util.concurrent.TimeUnit;
 import java.util.concurrent.atomic.AtomicBoolean;
 
-import io.a2a.server.util.TempLoggerWrapper;
 import io.a2a.spec.Event;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 public abstract class EventQueue implements AutoCloseable {
 
-    private static final Logger log = new TempLoggerWrapper(LoggerFactory.getLogger(EventQueue.class));
+    private static final Logger log = LoggerFactory.getLogger(EventQueue.class);
 
     // TODO decide on a capacity
     private static final int queueSize = 1000;

sdk-server-common/src/main/java/io/a2a/server/requesthandlers/DefaultRequestHandler.java

@@ -31,7 +31,6 @@
 import io.a2a.server.tasks.ResultAggregator;
 import io.a2a.server.tasks.TaskManager;
 import io.a2a.server.tasks.TaskStore;
-import io.a2a.server.util.TempLoggerWrapper;
 import io.a2a.server.util.async.Internal;
 import io.a2a.spec.Event;
 import io.a2a.spec.EventKind;
@@ -53,7 +52,7 @@
 @ApplicationScoped
 public class DefaultRequestHandler implements RequestHandler {
 
-    private static final Logger log = new TempLoggerWrapper(LoggerFactory.getLogger(DefaultRequestHandler.class));
+    private static final Logger log = LoggerFactory.getLogger(DefaultRequestHandler.class);
 
     private final AgentExecutor agentExecutor;
     private final TaskStore taskStore;

sdk-server-common/src/main/java/io/a2a/server/tasks/ResultAggregator.java

@@ -9,7 +9,6 @@
 import java.util.concurrent.atomic.AtomicReference;
 
 import io.a2a.server.events.EventConsumer;
-import io.a2a.server.util.TempLoggerWrapper;
 import io.a2a.spec.A2AServerException;
 import io.a2a.spec.Event;
 import io.a2a.spec.EventKind;
@@ -23,7 +22,7 @@
 import org.slf4j.LoggerFactory;
 
 public class ResultAggregator {
-    private static final Logger log = new TempLoggerWrapper(LoggerFactory.getLogger(ResultAggregator.class));
+    private static final Logger log = LoggerFactory.getLogger(ResultAggregator.class);
 
     private final TaskManager taskManager;
     private volatile Message message;