From 96c282ec0689e64f9bf8d36a2860a19f7331d88d Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Thu, 18 Jun 2026 21:00:40 +0000
Subject: [PATCH] =?UTF-8?q?=F0=9F=9B=A1=EF=B8=8F=20Sentinel:=20[Medium]=20?=
 =?UTF-8?q?Add=20explicit=20timeouts=20to=20all=20external=20fetch=20calls?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

🚨 Severity: MEDIUM
💡 Vulnerability: The native \`fetch\` API does not have a default timeout. Several external API calls to Dexscreener, Farcaster public node info endpoints, and GitHub lacked explicit timeouts.
🎯 Impact: If these external services are slow or unresponsive, requests can hang indefinitely, potentially exhausting server connections and causing denial of service (DoS) or very slow page loads.
🔧 Fix: Added \`signal: AbortSignal.timeout(TIMEOUT_MS)\` to all native \`fetch\` calls in \`src/lib/network.ts\`, \`src/lib/snap-market.ts\`, and \`scripts/sync-sources.mjs\`.
✅ Verification: Ran \`npm run lint\`, \`npm run typecheck\`, and \`npm run build\` successfully to ensure no build-time or type errors were introduced.

Co-authored-by: felirami <6752178+felirami@users.noreply.github.com>
---
 .jules/sentinel.md       | 5 +++++
 scripts/sync-sources.mjs | 3 +++
 src/lib/network.ts       | 1 +
 src/lib/snap-market.ts   | 2 ++
 4 files changed, 11 insertions(+)

diff --git a/.jules/sentinel.md b/.jules/sentinel.md
index b5d9250..e5b2a42 100644
--- a/.jules/sentinel.md
+++ b/.jules/sentinel.md
@@ -12,3 +12,8 @@
 **Vulnerability:** XSS risk via unsanitized `<` characters in `JSON.stringify` output injected into `<script type="application/ld+json">`.
 **Learning:** `JSON.stringify()` does not automatically escape `<` as `\u003c`. If dynamic or unsanitized content is serialized into a `<script>` tag via `dangerouslySetInnerHTML`, an attacker can include `</script>` to break out of the context and inject malicious scripts.
 **Prevention:** Always replace `<` with `\u003c` when injecting JSON output into script tags, e.g., `JSON.stringify(data).replace(/</g, '\\u003c')`.
+
+## 2026-06-18 - Native Fetch Timeouts
+**Vulnerability:** External fetch calls hanging indefinitely.
+**Learning:** The native `fetch` API does not have a default timeout. Without an explicit abort signal, external API calls can hang indefinitely, leading to resource exhaustion, slow page loads, and potential Denial of Service (DoS).
+**Prevention:** Always include a timeout signal for external `fetch` calls using `signal: AbortSignal.timeout(TIMEOUT_MS)`.
diff --git a/scripts/sync-sources.mjs b/scripts/sync-sources.mjs
index 7dbc002..ae93aa3 100644
--- a/scripts/sync-sources.mjs
+++ b/scripts/sync-sources.mjs
@@ -24,6 +24,7 @@ const DOCS_LINK_LIMIT = 80;
 const README_HEADING_LIMIT = 10;
 
 const githubToken = process.env.GITHUB_TOKEN || process.env.GH_TOKEN;
+const GITHUB_FETCH_TIMEOUT_MS = 15000;
 
 async function fetchGithubJson(url, { optional = false } = {}) {
   const response = await fetch(url, {
@@ -32,6 +33,7 @@ async function fetchGithubJson(url, { optional = false } = {}) {
       "X-GitHub-Api-Version": "2022-11-28",
       ...(githubToken ? { Authorization: `Bearer ${githubToken}` } : {}),
     },
+    signal: AbortSignal.timeout(GITHUB_FETCH_TIMEOUT_MS),
   });
 
   if ((response.status === 404 || response.status === 409) && optional) {
@@ -53,6 +55,7 @@ async function fetchGithubText(url, { optional = false } = {}) {
       "X-GitHub-Api-Version": "2022-11-28",
       ...(githubToken ? { Authorization: `Bearer ${githubToken}` } : {}),
     },
+    signal: AbortSignal.timeout(GITHUB_FETCH_TIMEOUT_MS),
   });
 
   if ((response.status === 404 || response.status === 409) && optional) {
diff --git a/src/lib/network.ts b/src/lib/network.ts
index 049cc1e..8a390b6 100644
--- a/src/lib/network.ts
+++ b/src/lib/network.ts
@@ -85,6 +85,7 @@ export async function getNetworkStatus(): Promise<NetworkStatus> {
       headers: {
         Accept: "application/json",
       },
+      signal: AbortSignal.timeout(NODE_PROBE_TIMEOUT_MS),
     });
 
     if (!response.ok) {
diff --git a/src/lib/snap-market.ts b/src/lib/snap-market.ts
index 19b7c1a..7996ea4 100644
--- a/src/lib/snap-market.ts
+++ b/src/lib/snap-market.ts
@@ -1,6 +1,7 @@
 import { SNAP, correctedSnapFdv } from "@/lib/snap";
 
 export const SNAP_MARKET_REVALIDATE = 30;
+export const SNAP_MARKET_TIMEOUT_MS = 10_000;
 
 type DexPeriod = {
   buys?: number;
@@ -79,6 +80,7 @@ export async function getSnapMarketData(): Promise<SnapMarketResponse> {
         "user-agent": "hypersnap.org market data checker",
       },
       next: { revalidate: SNAP_MARKET_REVALIDATE },
+      signal: AbortSignal.timeout(SNAP_MARKET_TIMEOUT_MS),
     });
 
     if (!response.ok) {