Add default timeout to gh queries

Summary: Add more timeouts to other gh queries. I wouldn't expect these queries to take more than a few seconds. If they are, something is wrong. This also means we have a cap on how many processes we'll have going at the same time.

Reviewed By: muirdm

Differential Revision: D76072068

fbshipit-source-id: d1ff8ed1576110a430abc25c71ea8a6f87bca96e

Author: evangrayk

addons/isl-server/src/github/githubCodeReviewProvider.ts

@@ -62,6 +62,8 @@ export type GitHubDiffSummary = {
   branchName?: string;
 };
 
+const DEFAULT_GH_FETCH_TIMEOUT = 60_000; // 1 minute
+
 type GitHubCodeReviewSystem = CodeReviewSystem & {type: 'github'};
 export class GitHubCodeReviewProvider implements CodeReviewProvider {
   constructor(private codeReviewSystem: GitHubCodeReviewSystem, private logger: Logger) {}
@@ -234,7 +236,12 @@ export class GitHubCodeReviewProvider implements CodeReviewProvider {
   }
 
   private query<D, V>(query: string, variables: V, timeoutMs?: number): Promise<D | undefined> {
-    return queryGraphQL<D, V>(query, variables, this.codeReviewSystem.hostname, timeoutMs);
+    return queryGraphQL<D, V>(
+      query,
+      variables,
+      this.codeReviewSystem.hostname,
+      timeoutMs ?? DEFAULT_GH_FETCH_TIMEOUT,
+    );
   }
 
   public dispose() {