Skip to content

Commit 3242b80

Browse files
committed
better log
1 parent af9d4b6 commit 3242b80

File tree

1 file changed

+12
-8
lines changed

1 file changed

+12
-8
lines changed

src/main/java/mindustrytool/handler/ApiGateway.java

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3,33 +3,29 @@
33
import java.lang.ref.WeakReference;
44
import java.net.URI;
55
import java.time.Duration;
6-
import java.util.ArrayList;
7-
import java.util.List;
8-
import java.util.concurrent.BlockingQueue;
96
import java.util.concurrent.CompletableFuture;
107
import java.util.concurrent.ConcurrentHashMap;
11-
import java.util.concurrent.LinkedBlockingQueue;
128
import java.util.concurrent.TimeUnit;
139

1410
import com.github.benmanes.caffeine.cache.Cache;
1511
import com.github.benmanes.caffeine.cache.Caffeine;
1612

1713
import arc.util.Http;
1814
import arc.util.Http.HttpRequest;
15+
import arc.util.Http.HttpStatusException;
1916
import arc.util.Log;
2017
import arc.util.Strings;
2118
import mindustrytool.utils.JsonUtils;
2219
import mindustrytool.utils.Utils;
2320
import mindustrytool.ServerController;
24-
import mindustrytool.type.BuildLogDto;
2521
import mindustrytool.type.MindustryPlayerDto;
2622
import mindustrytool.type.PaginationRequest;
2723
import mindustrytool.type.PlayerDto;
2824
import mindustrytool.type.ServerDto;
2925

3026
public class ApiGateway {
3127

32-
private final WeakReference<ServerController> context;
28+
final WeakReference<ServerController> context;
3329
private final ConcurrentHashMap<String, Object> locks = new ConcurrentHashMap<>();
3430

3531
public Cache<PaginationRequest, ServerDto> serverQueryCache = Caffeine.newBuilder()
@@ -83,7 +79,15 @@ private <T> T send(HttpRequest req, int timeout, Class<T> clazz) {
8379
.header("X-SERVER-ID", ServerController.SERVER_ID.toString())
8480
.timeout(timeout * 1000)
8581
.redirects(true)
86-
.error(error -> res.completeExceptionally(new RuntimeException(req.url, error)))
82+
.error(error -> {
83+
if (error instanceof HttpStatusException) {
84+
HttpStatusException e = (HttpStatusException) error;
85+
res.completeExceptionally(
86+
new RuntimeException(req.url + " " + e.response.getResultAsString(), e));
87+
} else {
88+
res.completeExceptionally(new RuntimeException(req.url, error));
89+
}
90+
})
8791
.submit(response -> {
8892
try {
8993
if (clazz.equals(Void.class)) {
@@ -106,7 +110,7 @@ private <T> T send(HttpRequest req, int timeout, Class<T> clazz) {
106110
try {
107111
return res.get(timeout, TimeUnit.SECONDS);
108112
} catch (Throwable e) {
109-
throw new RuntimeException(req.method + " " + req.url, e);
113+
throw new RuntimeException(req.method + " " + req.url, e);
110114
}
111115
}
112116

0 commit comments

Comments
 (0)