Merge f96a5edbb9 into f1f4e8baff

- removed redundant try block containing checker with `rsp.body` instead of using peekBody, resulting in closing the connection before even reading it.

app/src/main/java/fr/free/nrw/commons/OkHttpConnectionFactory.java

@@ -95,9 +95,7 @@ public final class OkHttpConnectionFactory {
             if (rsp.isSuccessful()) {
                 try (final ResponseBody responseBody = rsp.peekBody(ERRORS_PREFIX.length())) {
                     if (ERRORS_PREFIX.equals(responseBody.string())) {
-                        try (final ResponseBody body = rsp.body()) {
-                            throw new IOException(body.string());
-                        }
+                        throw new IOException(responseBody.string());
                     }
                 } catch (final IOException e) {
                     // Log the error as debug (and therefore, "expected") or at error level