6bf8bebf51
CI / Test and Build (push) Failing after 1s
CI / Migrate Dev DB (push) Has been skipped
CI / Migrate DB (push) Has been skipped
CodeQL / Analyze actions (push) Has been cancelled
CodeQL / Analyze javascript-typescript (push) Has been cancelled
CI / Detect Version (push) Has been cancelled
CI / Detect Desktop Changes (push) Has been cancelled
CI / Build AMD64 (blacksmith-2vcpu-ubuntu-2404, ./docker/cron.Dockerfile, ubuntu-latest, ghcr.io/simstudioai/cron) (push) Has been cancelled
CI / Build AMD64 (blacksmith-2vcpu-ubuntu-2404, ./docker/db.Dockerfile, ECR_MIGRATIONS, ubuntu-latest, ghcr.io/simstudioai/migrations) (push) Has been cancelled
CI / Build AMD64 (blacksmith-4vcpu-ubuntu-2404, ./docker/pii.Dockerfile, ECR_PII, ubuntu-latest, ghcr.io/simstudioai/pii) (push) Has been cancelled
CI / Build AMD64 (blacksmith-4vcpu-ubuntu-2404, ./docker/realtime.Dockerfile, ECR_REALTIME, ubuntu-latest, ghcr.io/simstudioai/realtime) (push) Has been cancelled
CI / Build AMD64 (blacksmith-8vcpu-ubuntu-2404, ./docker/app.Dockerfile, ECR_APP, linux-x64-8-core, ghcr.io/simstudioai/simstudio) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (blacksmith-4vcpu-ubuntu-2404-arm, ./docker/cron.Dockerfile, ubuntu-24.04-arm, ghcr.io/simstudioai/cron) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (blacksmith-4vcpu-ubuntu-2404-arm, ./docker/db.Dockerfile, ubuntu-24.04-arm, ghcr.io/simstudioai/migrations) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (blacksmith-4vcpu-ubuntu-2404-arm, ./docker/pii.Dockerfile, ubuntu-24.04-arm, ghcr.io/simstudioai/pii) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (blacksmith-4vcpu-ubuntu-2404-arm, ./docker/realtime.Dockerfile, ubuntu-24.04-arm, ghcr.io/simstudioai/realtime) (push) Has been cancelled
CI / Build ARM64 (GHCR Only) (blacksmith-8vcpu-ubuntu-2404-arm, ./docker/app.Dockerfile, linux-arm64-8-core, ghcr.io/simstudioai/simstudio) (push) Has been cancelled
CI / Check Docs Changes (push) Has been cancelled
Publish CLI Package / publish-npm (push) Has been cancelled
Publish Python SDK / publish-pypi (push) Has been cancelled
CI / Deploy Trigger.dev (Dev) (push) Has been cancelled
Helm Chart / Lint, test, and validate chart (push) Has been cancelled
Helm Chart / Chart version bumped (push) Has been cancelled
Publish TypeScript SDK / publish-npm (push) Has been cancelled
CI / Build Dev ECR (blacksmith-8vcpu-ubuntu-2404, ./docker/app.Dockerfile, ECR_APP, linux-x64-8-core) (push) Has been cancelled
CI / Promote Images (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/cron) (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/migrations) (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/pii) (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/realtime) (push) Has been cancelled
CI / Build Dev ECR (blacksmith-2vcpu-ubuntu-2404, ./docker/db.Dockerfile, ECR_MIGRATIONS, ubuntu-latest) (push) Has been cancelled
CI / Build Dev ECR (blacksmith-4vcpu-ubuntu-2404, ./docker/pii.Dockerfile, ECR_PII, ubuntu-latest) (push) Has been cancelled
CI / Build Dev ECR (blacksmith-4vcpu-ubuntu-2404, ./docker/realtime.Dockerfile, ECR_REALTIME, ubuntu-latest) (push) Has been cancelled
CI / Create GHCR Manifests (ghcr.io/simstudioai/simstudio) (push) Has been cancelled
CI / Process Docs (push) Has been cancelled
CI / Create GitHub Release (push) Has been cancelled
CI / Check Desktop Signing Secrets (push) Has been cancelled
CI / Desktop Release (push) Has been cancelled
CI / Create Desktop Prerelease (push) Has been cancelled
CI / Desktop Prerelease Build (push) Has been cancelled
CI / Publish Desktop Prerelease (push) Has been cancelled
CI / Prune Desktop Prereleases (push) Has been cancelled
Helm Chart / Install on kind and run helm test (push) Has been cancelled
100 lines
3.5 KiB
TypeScript
100 lines
3.5 KiB
TypeScript
/**
|
|
* @vitest-environment node
|
|
*/
|
|
import { beforeEach, describe, expect, it, vi } from 'vitest'
|
|
|
|
const { mockResolve } = vi.hoisted(() => ({ mockResolve: vi.fn() }))
|
|
|
|
vi.mock('@sim/security/dns', () => ({
|
|
resolveHostAddresses: mockResolve,
|
|
preferIpv4: (addresses: string[]) =>
|
|
addresses.find((address) => address.includes('.')) ?? addresses[0],
|
|
}))
|
|
|
|
vi.mock('@/lib/core/config/env-flags', () => ({
|
|
isHosted: false,
|
|
isPrivateDatabaseHostsAllowed: false,
|
|
getProxyUrl: () => undefined,
|
|
}))
|
|
|
|
import { validateUrlWithDNS } from '@/lib/core/security/input-validation.server'
|
|
|
|
/**
|
|
* Shapes a resolver answer the way `resolveHostAddresses` does, including its
|
|
* IPv4-first preference — so `preferred` can differ from `addresses[0]`, which
|
|
* is the whole reason the field exists.
|
|
*/
|
|
function resolved(addresses: string[]) {
|
|
const preferred = addresses.find((address) => address.includes('.')) ?? addresses[0]
|
|
return { addresses, preferred }
|
|
}
|
|
|
|
describe('validateUrlWithDNS address classification', () => {
|
|
beforeEach(() => {
|
|
vi.clearAllMocks()
|
|
})
|
|
|
|
it('drops a private co-record and pins the public one', async () => {
|
|
// The gap this closes: one address used to be classified, so which record
|
|
// got judged was a matter of resolver order.
|
|
mockResolve.mockResolvedValue(resolved(['93.184.216.34', '10.0.0.5']))
|
|
|
|
const result = await validateUrlWithDNS('https://mixed.example/api')
|
|
|
|
expect(result.isValid).toBe(true)
|
|
expect(result.resolvedIP).toBe('93.184.216.34')
|
|
})
|
|
|
|
it('rejects when every record is private', async () => {
|
|
mockResolve.mockResolvedValue(resolved(['10.0.0.5', '192.168.1.9']))
|
|
|
|
const result = await validateUrlWithDNS('https://internal.example/api')
|
|
|
|
expect(result.isValid).toBe(false)
|
|
expect(result.error).toContain('blocked IP address')
|
|
})
|
|
|
|
it('never pins an address the filter refused', async () => {
|
|
// The private record sorts first AND is the IPv4 one, so a pin taken from
|
|
// the unfiltered set would land on 10.0.0.5.
|
|
mockResolve.mockResolvedValue(resolved(['10.0.0.5', '2606:2800:220:1::248']))
|
|
|
|
const result = await validateUrlWithDNS('https://mixed.example/api')
|
|
|
|
expect(result.isValid).toBe(true)
|
|
expect(result.resolvedIP).toBe('2606:2800:220:1::248')
|
|
})
|
|
|
|
it('accepts a host whose every record is public, pinning the preferred one', async () => {
|
|
mockResolve.mockResolvedValue(resolved(['93.184.216.34', '93.184.216.35']))
|
|
|
|
const result = await validateUrlWithDNS('https://example.com/api')
|
|
|
|
expect(result.isValid).toBe(true)
|
|
expect(result.resolvedIP).toBe('93.184.216.34')
|
|
})
|
|
|
|
it('keeps the self-hosted localhost carve-out when every record is loopback', async () => {
|
|
mockResolve.mockResolvedValue(resolved(['127.0.0.1', '::1']))
|
|
|
|
expect((await validateUrlWithDNS('https://localhost/api')).isValid).toBe(true)
|
|
})
|
|
|
|
it('drops an off-loopback record from localhost rather than pinning it', async () => {
|
|
// The carve-out covers loopback only, so the LAN record is filtered out and
|
|
// the pin stays on the machine the carve-out was written for.
|
|
mockResolve.mockResolvedValue(resolved(['127.0.0.1', '10.0.0.5']))
|
|
|
|
const result = await validateUrlWithDNS('https://localhost/api')
|
|
|
|
expect(result.isValid).toBe(true)
|
|
expect(result.resolvedIP).toBe('127.0.0.1')
|
|
})
|
|
|
|
it('reports an unresolvable host rather than treating it as public', async () => {
|
|
mockResolve.mockRejectedValue(new Error('ENOTFOUND'))
|
|
|
|
expect((await validateUrlWithDNS('https://missing.example/api')).isValid).toBe(false)
|
|
})
|
|
})
|