chore: minor cleanup after 0.7.1 release (remove redundant logger dependency)

Author: bsayli

customer-service-client/README.md

@@ -112,7 +112,7 @@ public class CustomerApiClientConfig {
       return builder -> builder.defaultStatusHandler(
               HttpStatusCode::isError,
               (request, response) -> {
-                 ProblemDetail pd = ProblemDetailSupport.extract(om, response, log);
+                 ProblemDetail pd = ProblemDetailSupport.extract(om, response);
                  throw new ClientProblemException(pd, response.getStatusCode().value());
               });
    }

customer-service-client/src/main/java/io/github/bsayli/openapi/client/adapter/config/CustomerApiClientConfig.java

@@ -11,8 +11,6 @@
 import org.apache.hc.client5.http.impl.classic.CloseableHttpClient;
 import org.apache.hc.client5.http.impl.classic.HttpClients;
 import org.apache.hc.client5.http.impl.io.PoolingHttpClientConnectionManagerBuilder;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 import org.springframework.beans.factory.annotation.Value;
 import org.springframework.boot.web.client.RestClientCustomizer;
 import org.springframework.context.annotation.Bean;
@@ -23,16 +21,15 @@
 
 @Configuration
 public class CustomerApiClientConfig {
-
-  private static final Logger log = LoggerFactory.getLogger(CustomerApiClientConfig.class);
+  
 
   @Bean
   RestClientCustomizer problemDetailStatusHandler(ObjectMapper om) {
     return builder ->
         builder.defaultStatusHandler(
             HttpStatusCode::isError,
             (request, response) -> {
-              ProblemDetail pd = ProblemDetailSupport.extract(om, response, log);
+              ProblemDetail pd = ProblemDetailSupport.extract(om, response);
               throw new ClientProblemException(pd, response.getStatusCode().value());
             });
   }

customer-service-client/src/main/java/io/github/bsayli/openapi/client/adapter/support/ProblemDetailSupport.java

@@ -6,18 +6,21 @@
 import java.io.InputStream;
 import java.util.Optional;
 import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.springframework.http.HttpStatusCode;
 import org.springframework.http.MediaType;
 import org.springframework.http.client.ClientHttpResponse;
 
 public final class ProblemDetailSupport {
 
+  private static final Logger log = LoggerFactory.getLogger(ProblemDetailSupport.class);
+
   private ProblemDetailSupport() {}
 
-  public static ProblemDetail extract(ObjectMapper om, ClientHttpResponse response, Logger log) {
+  public static ProblemDetail extract(ObjectMapper om, ClientHttpResponse response) {
     ProblemDetail pd;
     MediaType contentType =
-        Optional.ofNullable(response.getHeaders().getContentType()).orElse(MediaType.ALL);
+            Optional.ofNullable(response.getHeaders().getContentType()).orElse(MediaType.ALL);
     HttpStatusCode status;
 
     try {
@@ -36,10 +39,10 @@ public static ProblemDetail extract(ObjectMapper om, ClientHttpResponse response
       }
     } catch (IOException e) {
       log.warn(
-          "Unable to deserialize ProblemDetail (status={}, contentType={}); using generic fallback",
-          status,
-          contentType,
-          e);
+              "Unable to deserialize ProblemDetail (status={}, contentType={}); using generic fallback",
+              status,
+              contentType,
+              e);
       pd = fallback(status, "Unparseable problem response");
     } catch (Exception e) {
       log.warn("Unexpected error while parsing ProblemDetail", e);
@@ -56,4 +59,4 @@ private static ProblemDetail fallback(HttpStatusCode status, String detail) {
     pd.setDetail(detail);
     return pd;
   }
-}
+}

docs/adoption/client-side-adoption.md

@@ -277,7 +277,7 @@ public class YourApiClientConfig {
       return builder -> builder.defaultStatusHandler(
               HttpStatusCode::isError,
               (request, response) -> {
-                 ProblemDetail pd = ProblemDetailSupport.extract(om, response, log);
+                 ProblemDetail pd = ProblemDetailSupport.extract(om, response);
                  throw new ClientProblemException(pd, response.getStatusCode().value());
               });
    }