sam/dev_arc_aws
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Surface underlying network error instead of generic 'fetch failed'

Browse source 
undici's fetch() collapses DNS/TLS/connection-refused/timeout failures into a vague TypeError. Unwrap err.cause so Test Connection shows the real reason (e.g. ECONNREFUSED, certificate error) instead of just 'fetch failed'.
This commit is contained in:
Claude 2026-06-20 10:31:11 +00:00
parent 9157a2647a
commit cbd666fe60
No known key found for this signature in database
1 changed files with 12 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
backend/src/integrations/proxmox.ts
View file

@ -16,6 +16,17 @@ function authHeader(apiKey: string): Record<string, string> {
// Proxmox ships with a self-signed cert by default, which Node's fetch rejects out of the box.
const insecureDispatcher = new Agent({ connect: { rejectUnauthorized: false } })
// undici's fetch() wraps the real failure (DNS, TLS, connection refused/timed out) in a generic
// "fetch failed" TypeError — unwrap err.cause so the actual reason reaches the UI.
function describeFetchError(err: unknown): string {
if (err instanceof Error) {
const cause = (err as { cause?: unknown }).cause
if (cause instanceof Error) return `${err.message}: ${cause.message}`
return err.message
}
return 'Connection failed'
}
export const proxmox: IntegrationAdapter = {
async testConnection(config, secrets) {
const baseUrl = config.baseUrl?.replace(/\/$/, '')
@ -30,7 +41,7 @@ export const proxmox: IntegrationAdapter = {
if (!res.ok) return { ok: false, message: `HTTP ${res.status}` }
return { ok: true, message: 'Connected' }
} catch (err) {
return { ok: false, message: err instanceof Error ? err.message : 'Connection failed' }
return { ok: false, message: describeFetchError(err) }
}
},