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

Merge pull request #7 from SamuelSJames/claude/wonderful-faraday-qxym5t

Browse source 
Surface underlying network error instead of generic 'fetch failed'
This commit is contained in:
Samuel James 2026-06-20 06:32:20 -04:00 • committed by GitHub
commit af4d2bac31
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
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) }
}
},