[Sentry Auto-Fix] DOCS-2JX: Error: Request timed out after 1500ms (#59286)

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Ebonsignori <17055832+Ebonsignori@users.noreply.github.com>

Author: Ebonsignori

src/archives/middleware/archived-enterprise-versions.ts

@@ -2,6 +2,7 @@ import type { Response, NextFunction } from 'express'
 import { fetchWithRetry } from '@/frame/lib/fetch-utils'
 
 import statsd from '@/observability/lib/statsd'
+import { createLogger } from '@/observability/logger'
 import {
   firstVersionDeprecatedOnNewSite,
   lastVersionWithoutArchivedRedirectsFile,
@@ -19,6 +20,8 @@ import getRedirect, { splitPathByLanguage } from '@/redirects/lib/get-redirect'
 import getRemoteJSON from '@/frame/lib/get-remote-json'
 import { ExtendedRequest } from '@/types'
 
+const logger = createLogger(import.meta.url)
+
 const OLD_PUBLIC_AZURE_BLOB_URL = 'https://githubdocs.azureedge.net'
 // Old Azure Blob Storage `enterprise` container.
 const OLD_AZURE_BLOB_ENTERPRISE_DIR = `${OLD_PUBLIC_AZURE_BLOB_URL}/enterprise`
@@ -78,11 +81,17 @@ const cacheAggressively = (res: Response) => {
 const retryConfiguration = { limit: 3 }
 // According to our Datadog metrics, the *average* time for the
 // the 'archive_enterprise_proxy' metric is ~70ms (excluding spikes)
-// which much less than 1500ms.
+// which is much less than 3000ms.
 // We have observed errors of timeout, in production, when it was
-// set to 500ms. Let's try to be very conservative here to avoid
-// unnecessary error reporting.
-const timeoutConfiguration = { response: 1500 }
+// set to 500ms and then 1500ms. Let's be more conservative here to
+// avoid unnecessary error reporting during occasional slow responses.
+const timeoutConfiguration = { response: 3000 }
+
+// Monitoring thresholds for logging response times
+// Log warnings when responses exceed half the timeout threshold
+const WARN_RESPONSE_THRESHOLD = timeoutConfiguration.response / 2 // 1500ms
+// Log info for responses that are noticeably slow but not concerning
+const SLOW_RESPONSE_THRESHOLD = 500 // ms
 
 // This module handles requests for deprecated GitHub Enterprise versions
 // by routing them to static content in
@@ -201,10 +210,44 @@ export default async function archivedEnterpriseVersions(
     )
 
   const statsdTags = [`version:${requestedVersion}`]
+  const startTime = Date.now()
   const r = await statsd.asyncTimer(doGet, 'archive_enterprise_proxy', [
     ...statsdTags,
     `path:${req.path}`,
   ])()
+  const responseTime = Date.now() - startTime
+
+  // Log warnings for slow responses to help identify degraded performance
+  // A response time over half the timeout indicates potential issues
+  if (responseTime > WARN_RESPONSE_THRESHOLD) {
+    logger.warn('Slow response from archived enterprise content', {
+      version: requestedVersion,
+      path: req.path,
+      responseTime: `${responseTime}ms`,
+      status: r.status,
+      threshold: `${WARN_RESPONSE_THRESHOLD}ms`,
+    })
+  }
+
+  // Log errors for non-200 responses to help identify issues with archived content
+  if (r.status !== 200) {
+    logger.error('Failed to fetch archived enterprise content', {
+      version: requestedVersion,
+      path: req.path,
+      status: r.status,
+      responseTime: `${responseTime}ms`,
+      url: getProxyPath(req.path, requestedVersion),
+    })
+  }
+
+  // Log successful responses with timing for monitoring trends
+  if (r.status === 200 && responseTime > SLOW_RESPONSE_THRESHOLD) {
+    logger.info('Archived enterprise content response', {
+      version: requestedVersion,
+      responseTime: `${responseTime}ms`,
+      status: r.status,
+    })
+  }
 
   if (r.status === 200) {
     const body = await r.text()
@@ -317,6 +360,7 @@ export default async function archivedEnterpriseVersions(
 
     return res.send(modifiedBody)
   }
+
   // In releases 2.13 - 2.17, we lost access to frontmatter redirects
   //  during the archival process. This workaround finds potentially
   // relevant frontmatter redirects in currently supported pages