From f28f78c045840e0fd760a76f00cae7eb6760a6f7 Mon Sep 17 00:00:00 2001 From: Dave Borowitz Date: Tue, 7 Jul 2015 15:59:21 -0400 Subject: [PATCH] Treat redirects as success when checking for documentation Documentation may be served elsewhere due to a strange reverse proxy configuration or authentication setup. Assume that a 301 or 302 response for Documentation/index.html indicates documentation is present. Change-Id: I07cfe8fc1721aeb3be690051b94517c22a08c5cd --- .../main/java/com/google/gerrit/client/Gerrit.java | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/Gerrit.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/Gerrit.java index 8a2d6457e9..3cb95149c7 100644 --- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/Gerrit.java +++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/Gerrit.java @@ -822,10 +822,15 @@ public class Gerrit implements EntryPoint { req.setCallback(new RequestCallback() { @Override public void onResponseReceived(Request req, Response resp) { - if (resp.getStatusCode() == Response.SC_OK) { - cb.onSuccess(DocInfo.create()); - } else { - cb.onSuccess(null); + switch (resp.getStatusCode()) { + case Response.SC_OK: + case Response.SC_MOVED_PERMANENTLY: + case Response.SC_MOVED_TEMPORARILY: + cb.onSuccess(DocInfo.create()); + break; + default: + cb.onSuccess(null); + break; } }