Merge pull request #86 from overleaf/em-handle-history-error

Handle errors from the history service
2024-11-21 20:47:08 -05:00 · 2020-12-09 09:05:24 -05:00 · 2020-12-09 09:05:24 -05:00 · 8a5027255a
commit 8a5027255a
parent 614f57f778 2430d5fe1d
1 changed files with 7 additions and 2 deletions
--- a/services/git-bridge/src/main/java/uk/ac/ic/wlgitbridge/io/http/ning/NingHttpClient.java
+++ b/services/git-bridge/src/main/java/uk/ac/ic/wlgitbridge/io/http/ning/NingHttpClient.java
@ -52,11 +52,16 @@ public class NingHttpClient implements NingHttpClientFacade {
                @Override
                public byte[] onCompleted(
                        Response response
-                ) throws IOException {
+                ) throws Exception {
+                    int statusCode = response.getStatusCode();
+                    if (statusCode >= 400) {
+                        throw new Exception("got status " + statusCode +
+                                            " fetching " + url);
+                    }
                    byte[] ret = bytes.toByteArray();
                    bytes.close();
                    log.info(
-                            response.getStatusCode()
+                            statusCode
                                    + " "
                                    + response.getStatusText()
                                    + " ("