Skip to content

Commit d240c5a

Browse files
committed
ApResolver: Make request inside a try-with-resources block
1 parent dae67be commit d240c5a

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

lib/src/main/java/xyz/gianlu/librespot/core/ApResolver.java

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
import okhttp3.OkHttpClient;
2424
import okhttp3.Request;
2525
import okhttp3.Response;
26+
import okhttp3.ResponseBody;
2627
import org.jetbrains.annotations.NotNull;
2728
import org.slf4j.Logger;
2829
import org.slf4j.LoggerFactory;
@@ -81,9 +82,10 @@ private void request(@NotNull String... types) throws IOException {
8182
Request request = new Request.Builder()
8283
.url(url.toString())
8384
.build();
84-
Response response = client.newCall(request).execute();
85-
if (response.isSuccessful()) {
86-
try (Reader reader = response.body().charStream()) {
85+
try (Response response = client.newCall(request).execute()) {
86+
ResponseBody body = response.body();
87+
if (body == null) throw new IOException("No body");
88+
try (Reader reader = body.charStream()) {
8789
JsonObject obj = JsonParser.parseReader(reader).getAsJsonObject();
8890
HashMap<String, List<String>> map = new HashMap<>();
8991
for (String type : types)

0 commit comments

Comments
 (0)