@elisra-devops/docgen-data-provider 1.21.0 → 1.23.0

package/src/helpers/tfs.ts

@@ -7,12 +7,12 @@ export class TFSServices {
     httpAgent: new (require('http').Agent)({
       keepAlive: true,
       maxSockets: 50, // Allow more connections
-      keepAliveMsecs: 30000, // Keep connections alive longer
+      keepAliveMsecs: 300000, // Keep connections alive longer
     }),
     httpsAgent: new (require('https').Agent)({
       keepAlive: true,
       maxSockets: 50,
-      keepAliveMsecs: 30000,
+      keepAliveMsecs: 300000,
     }),
   };
 
@@ -28,7 +28,6 @@ export class TFSServices {
         url: url,
         headers: { 'Content-Type': 'application/zip' },
         auth: { username: '', password: pat },
-        timeout: 15000, // Increased timeout for large files
       });
       return res;
     } catch (e) {
@@ -51,7 +50,6 @@ export class TFSServices {
       auth: { username: '', password: pat },
       data: data,
       responseType: 'arraybuffer', // Important for binary data
-      timeout: 8000, // Increased timeout for images
     };
 
     return this.executeWithRetry(url, config, printError, (response) => {
@@ -79,7 +77,7 @@ export class TFSServices {
       method: requestMethod,
       auth: { username: '', password: pat },
       data: data,
-      timeout: 10000, // More reasonable timeout
+      timeout: requestMethod.toLocaleLowerCase() === 'get' ? 10000 : undefined, // More reasonable timeout
     };
 
     return this.executeWithRetry(cleanUrl, config, printError, (response) => {
@@ -93,7 +91,6 @@ export class TFSServices {
       url: url,
       method: 'GET',
       headers: header,
-      timeout: 8000, // Reasonable timeout,
     };
 
     try {
@@ -118,7 +115,6 @@ export class TFSServices {
       method: requestMethod,
       auth: { username: '', password: pat },
       data: data,
-      timeout: 10000, // More reasonable timeout
     };
 
     try {
@@ -154,7 +150,7 @@ export class TFSServices {
         // Handle not found errors
         if (errorMessage.includes('could not be found')) {
           logger.info(`File does not exist, or you do not have permissions to read it.`);
-          return undefined;
+          throw new Error(`File not found or insufficient permissions: ${url}`);
         }
 
         // Check if we should retry

package/src/modules/GitDataProvider.ts

@@ -475,17 +475,17 @@ export default class GitDataProvider {
       let body =
         specificItemPath === ''
           ? {
-              itemVersion,
-              compareVersion,
-              includeWorkItems: true,
-            }
+            itemVersion,
+            compareVersion,
+            includeWorkItems: true,
+          }
           : {
-              itemVersion,
-              compareVersion,
-              includeWorkItems: true,
-              itemPath: specificItemPath,
-              historyMode: 'fullHistory',
-            };
+            itemVersion,
+            compareVersion,
+            includeWorkItems: true,
+            itemPath: specificItemPath,
+            historyMode: 'fullHistory',
+          };
 
       let url = `${gitUrl}/commitsbatch?$skip=${skipping}&$top=${chunkSize}&api-version=5.1`;
       let commitsResponse = await TFSServices.postRequest(url, this.token, undefined, body);

package/src/modules/PipelinesDataProvider.ts

@@ -97,9 +97,9 @@ export default class PipelinesDataProvider {
     searchPrevPipelineFromDifferentCommit: boolean
   ): boolean {
     const fromRepo =
-      fromPipeline.resources.repositories[0].self || fromPipeline.resources.repositories.__designer_repo;
+      fromPipeline.resources.repositories[0]?.self || fromPipeline.resources.repositories.__designer_repo;
     const targetRepo =
-      targetPipeline.resources.repositories[0].self || targetPipeline.resources.repositories.__designer_repo;
+      targetPipeline.resources.repositories[0]?.self || targetPipeline.resources.repositories.__designer_repo;
 
     if (fromRepo.repository.id !== targetRepo.repository.id) {
       return false;