From 0213858d0a9eeeb1d1ac3c60ceb1c721c8569a01 Mon Sep 17 00:00:00 2001
From: Vitaly Takmazov
Date: Sat, 31 Dec 2022 01:12:21 +0300
Subject: ActivityPub: cleanup logging

---
 .../java/com/juick/config/ActivityPubClientErrorHandler.java | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

(limited to 'src/main/java/com/juick/config/ActivityPubClientErrorHandler.java')

diff --git a/src/main/java/com/juick/config/ActivityPubClientErrorHandler.java b/src/main/java/com/juick/config/ActivityPubClientErrorHandler.java
index 69958bbd..23576ad5 100644
--- a/src/main/java/com/juick/config/ActivityPubClientErrorHandler.java
+++ b/src/main/java/com/juick/config/ActivityPubClientErrorHandler.java
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2008-2020, Juick
+ * Copyright (C) 2008-2022, Juick
  *
  * This program is free software: you can redistribute it and/or modify
  * it under the terms of the GNU Affero General Public License as
@@ -28,6 +28,7 @@ import org.springframework.http.client.ClientHttpResponse;
 import org.springframework.stereotype.Component;
 import org.springframework.web.client.DefaultResponseErrorHandler;
 
+import javax.annotation.Nonnull;
 import javax.inject.Inject;
 import java.io.IOException;
 import java.net.URI;
@@ -36,16 +37,17 @@ import java.nio.charset.StandardCharsets;
 @Component
 public class ActivityPubClientErrorHandler extends DefaultResponseErrorHandler {
     private static final Logger logger = LoggerFactory.getLogger("ActivityPub");
-
     @Inject
     private ApplicationEventPublisher applicationEventPublisher;
     @Override
-    public void handleError(URI contextUri, HttpMethod method, ClientHttpResponse response) throws IOException {
-        logger.warn("HTTP ERROR {} {} : {}", response.getStatusCode().value(),
-                response.getStatusText(), IOUtils.toString(response.getBody(), StandardCharsets.UTF_8));
+    public void handleError(@Nonnull URI contextUri, @Nonnull HttpMethod method, ClientHttpResponse response)
+            throws IOException {
         if (response.getStatusCode().equals(HttpStatus.GONE)) {
             logger.warn("Server report {} is gone, deleting", contextUri.toASCIIString());
             applicationEventPublisher.publishEvent(new DeleteUserEvent(this, contextUri.toASCIIString()));
+        } else {
+            logger.warn("HTTP ERROR {} {} : {}", response.getStatusCode().value(),
+                    response.getStatusText(), IOUtils.toString(response.getBody(), StandardCharsets.UTF_8));
         }
     }
 }
-- 
cgit v1.2.3