Merge pull request #571 from kubeshop/f1ames/fix/synchronizer-timeouts

fix(synchronizer): use explicit timeouts for API and auth queries
kubeshop · Nov 23, 2023 · 077a5e9 · 077a5e9
2 parents ae4b86b + 3bbb688
commit 077a5e9
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 2 deletions.
diff --git a/.changeset/nervous-badgers-call.md b/.changeset/nervous-badgers-call.md
@@ -0,0 +1,5 @@
+---
+"@monokle/synchronizer": patch
+---
+
+Add explicit timeouts for API communication
diff --git a/packages/synchronizer/src/handlers/apiHandler.ts b/packages/synchronizer/src/handlers/apiHandler.ts
@@ -287,6 +287,7 @@ export class ApiHandler {
         query,
         variables,
       }),
+      timeout: 30 * 1000,
     });
   }
 

diff --git a/packages/synchronizer/src/handlers/deviceFlowHandler.ts b/packages/synchronizer/src/handlers/deviceFlowHandler.ts
@@ -1,4 +1,4 @@
-import {Issuer} from 'openid-client';
+import {Issuer, custom} from 'openid-client';
 import {
   DEFAULT_DEVICE_FLOW_IDP_URL,
   DEFAULT_DEVICE_FLOW_CLIENT_ID,
@@ -28,7 +28,11 @@ export class DeviceFlowHandler {
       id_token_signed_response_alg: DEFAULT_DEVICE_FLOW_ALG,
     },
     private _clientScope: string = DEFAULT_DEVICE_FLOW_CLIENT_SCOPE
-  ) {}
+  ) {
+    custom.setHttpOptionsDefaults({
+      timeout: 10 * 1000,
+    });
+  }
 
   async initializeAuthFlow(): Promise<DeviceFlowHandle> {
     const client = await this.getClient();