diff --git a/services/git-bridge/src/main/java/uk/ac/ic/wlgitbridge/snapshot/push/PostbackPromise.java b/services/git-bridge/src/main/java/uk/ac/ic/wlgitbridge/snapshot/push/PostbackPromise.java index 766ed23a1d..420237d729 100644 --- a/services/git-bridge/src/main/java/uk/ac/ic/wlgitbridge/snapshot/push/PostbackPromise.java +++ b/services/git-bridge/src/main/java/uk/ac/ic/wlgitbridge/snapshot/push/PostbackPromise.java @@ -14,6 +14,8 @@ import java.util.concurrent.locks.ReentrantLock; */ public class PostbackPromise { + private static int TIMEOUT_SECONDS = 60 * 3; + private final String postbackKey; private final ReentrantLock lock; private final Condition cond; @@ -35,8 +37,8 @@ public class PostbackPromise { try { while (!received) { try { - if (!cond.await(60 * 3, TimeUnit.SECONDS)) { - throw new PostbackTimeoutException(); + if (!cond.await(TIMEOUT_SECONDS, TimeUnit.SECONDS)) { + throw new PostbackTimeoutException(TIMEOUT_SECONDS); } } catch (InterruptedException e) { throw new InternalErrorException(); diff --git a/services/git-bridge/src/main/java/uk/ac/ic/wlgitbridge/snapshot/push/exception/PostbackTimeoutException.java b/services/git-bridge/src/main/java/uk/ac/ic/wlgitbridge/snapshot/push/exception/PostbackTimeoutException.java index 2b1579cd3e..ca19448c3e 100644 --- a/services/git-bridge/src/main/java/uk/ac/ic/wlgitbridge/snapshot/push/exception/PostbackTimeoutException.java +++ b/services/git-bridge/src/main/java/uk/ac/ic/wlgitbridge/snapshot/push/exception/PostbackTimeoutException.java @@ -11,9 +11,15 @@ import java.util.List; */ public class PostbackTimeoutException extends SevereSnapshotPostException { + private int timeout; + + public PostbackTimeoutException(int timeout) { + this.timeout = timeout; + } + @Override public String getMessage() { - return "timeout"; + return "Request timed out (after " + this.timeout + " seconds)"; } @Override