fix(secure-fetch): route early-abort path through settledReject

waleedlatif1 · claude · waleedlatif1 · commit e7e2954259ab · 2026-05-04T20:41:09.000-07:00
Promise reject is idempotent so this wasn't a correctness bug, but
routing the already-aborted branch through settledReject keeps all
settling paths consistent and ensures cleanupAbort runs even if a
listener somehow gets registered later.

Co-Authored-By: Claude Opus 4.7 &lt;noreply@anthropic.com&gt;
diff --git a/apps/sim/lib/core/security/input-validation.server.ts b/apps/sim/lib/core/security/input-validation.server.ts
@@ -416,7 +416,7 @@ export async function secureFetchWithPinnedIP(
     if (options.signal) {
       if (options.signal.aborted) {
         req.destroy()
-        reject(options.signal.reason ?? new Error('Aborted'))
+        settledReject(options.signal.reason ?? new Error('Aborted'))
         return
       }
       onAbort = () => {

Original file line number	Diff line number	Diff line change
`@@ -416,7 +416,7 @@ export async function secureFetchWithPinnedIP(`
`416`	`416`	`if (options.signal) {`
`417`	`417`	`if (options.signal.aborted) {`
`418`	`418`	`req.destroy()`
`419`		`- reject(options.signal.reason ?? new Error('Aborted'))`
	`419`	`+ settledReject(options.signal.reason ?? new Error('Aborted'))`
`420`	`420`	`return`
`421`	`421`	`}`
`422`	`422`	`onAbort = () => {`