Skip to content
11 changes: 11 additions & 0 deletions apps/docs/components/icons.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -2261,6 +2261,17 @@ export function BrandfetchIcon(props: SVGProps<SVGSVGElement>) {
)
}

export function BrexIcon(props: SVGProps<SVGSVGElement>) {
return (
<svg {...props} viewBox='0 0 223 179.3' fill='none' xmlns='http://www.w3.org/2000/svg'>
<path
fill='#FFFFFF'
d='M144.9,14.3c-8.7,11.6-10.8,15.5-19.2,15.5H0v149.4h49.3c11.1,0,21.9-5.4,28.9-14.3c9-12,10.2-15.5,18.9-15.5 H223V0h-49.6C162.3,0,151.5,5.4,144.9,14.3L144.9,14.3z M183.9,110.9h-52.6c-11.4,0-21.9,4.8-28.9,14c-9,12-10.8,15.5-19.2,15.5 H38.8V68.7h52.6c11.4,0,21.9-5.4,28.9-14.3c9-11.6,11.4-15.2,19.5-15.2h44.2V110.9z'
/>
</svg>
)
}

export function BrightDataIcon(props: SVGProps<SVGSVGElement>) {
return (
<svg
Expand Down
2 changes: 2 additions & 0 deletions apps/docs/components/ui/icon-mapping.ts
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ import {
BoxCompanyIcon,
BrainIcon,
BrandfetchIcon,
BrexIcon,
BrightDataIcon,
BrowserUseIcon,
CalComIcon,
Expand Down Expand Up @@ -243,6 +244,7 @@ export const blockTypeToIconMap: Record<string, IconComponent> = {
azure_devops: AzureIcon,
box: BoxCompanyIcon,
brandfetch: BrandfetchIcon,
brex: BrexIcon,
brightdata: BrightDataIcon,
browser_use: BrowserUseIcon,
calcom: CalComIcon,
Expand Down
895 changes: 895 additions & 0 deletions apps/docs/content/docs/en/integrations/brex.mdx

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions apps/docs/content/docs/en/integrations/meta.json
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@
"azure_devops",
"box",
"brandfetch",
"brex",
"brightdata",
"browser_use",
"calcom",
Expand Down
245 changes: 245 additions & 0 deletions apps/sim/app/api/tools/brex/upload-receipt/route.test.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,245 @@
/**
* @vitest-environment node
*/
import {
createMockRequest,
hybridAuthMockFns,
inputValidationMock,
inputValidationMockFns,
} from '@sim/testing'
import { beforeEach, describe, expect, it, vi } from 'vitest'

const { mockProcessFilesToUserFiles, mockDownloadFileFromStorage, mockAssertToolFileAccess } =
vi.hoisted(() => ({
mockProcessFilesToUserFiles: vi.fn(),
mockDownloadFileFromStorage: vi.fn(),
mockAssertToolFileAccess: vi.fn(),
}))

vi.mock('@/lib/core/security/input-validation.server', () => inputValidationMock)
vi.mock('@/lib/uploads/utils/file-utils', () => ({
processFilesToUserFiles: mockProcessFilesToUserFiles,
}))
vi.mock('@/lib/uploads/utils/file-utils.server', () => ({
downloadFileFromStorage: mockDownloadFileFromStorage,
}))
vi.mock('@/app/api/files/authorization', () => ({
assertToolFileAccess: mockAssertToolFileAccess,
}))

import { POST } from '@/app/api/tools/brex/upload-receipt/route'

const mockFetch = vi.fn()

const PINNED_IP = '52.216.0.1'

const baseBody = {
apiKey: 'bxt_test_token',
expenseId: 'expense_123',
file: { key: 'uploads/receipt.pdf', name: 'receipt.pdf', size: 5, type: 'application/pdf' },
}

function jsonResponse(body: unknown, status = 200) {
return {
ok: status >= 200 && status < 300,
status,
text: async () => JSON.stringify(body),
json: async () => body,
}
}

beforeEach(() => {
vi.clearAllMocks()
vi.stubGlobal('fetch', mockFetch)
hybridAuthMockFns.mockCheckInternalAuth.mockResolvedValue({
success: true,
userId: 'user-1',
authType: 'internal_jwt',
})
inputValidationMockFns.mockValidateUrlWithDNS.mockResolvedValue({
isValid: true,
resolvedIP: PINNED_IP,
})
inputValidationMockFns.mockSecureFetchWithPinnedIP.mockResolvedValue(jsonResponse({}))
mockProcessFilesToUserFiles.mockReturnValue([
{ key: 'uploads/receipt.pdf', name: 'receipt.pdf', size: 5, type: 'application/pdf' },
])
mockAssertToolFileAccess.mockResolvedValue(null)
mockDownloadFileFromStorage.mockResolvedValue(Buffer.from('receipt-bytes'))
})

describe('POST /api/tools/brex/upload-receipt', () => {
it('rejects unauthenticated requests', async () => {
hybridAuthMockFns.mockCheckInternalAuth.mockResolvedValueOnce({
success: false,
error: 'unauthorized',
})

const response = await POST(createMockRequest('POST', baseBody))
expect(response.status).toBe(401)
expect(mockFetch).not.toHaveBeenCalled()
})

it('creates a receipt upload for an expense and PUTs the file to the pre-signed URL', async () => {
mockFetch.mockResolvedValueOnce(
jsonResponse({ id: 'receipt_1', uri: 'https://s3.example.com/presigned' })
)

const response = await POST(createMockRequest('POST', baseBody))
expect(response.status).toBe(200)
const data = await response.json()
expect(data).toEqual({
success: true,
output: { receiptId: 'receipt_1', receiptName: 'receipt.pdf', expenseId: 'expense_123' },
})

expect(mockFetch).toHaveBeenCalledTimes(1)
const [createUrl, createInit] = mockFetch.mock.calls[0]
expect(createUrl).toBe('https://api.brex.com/v1/expenses/card/expense_123/receipt_upload')
expect(createInit.method).toBe('POST')
expect(createInit.headers.Authorization).toBe('Bearer bxt_test_token')
expect(JSON.parse(createInit.body)).toEqual({ receipt_name: 'receipt.pdf' })

expect(inputValidationMockFns.mockValidateUrlWithDNS).toHaveBeenCalledWith(
'https://s3.example.com/presigned',
'uri'
)
const [uploadUrl, pinnedIP, uploadInit] =
inputValidationMockFns.mockSecureFetchWithPinnedIP.mock.calls[0]
expect(uploadUrl).toBe('https://s3.example.com/presigned')
expect(pinnedIP).toBe(PINNED_IP)
expect(uploadInit.method).toBe('PUT')
})

it('rejects a whitespace-only expense ID instead of falling back to receipt match', async () => {
const response = await POST(createMockRequest('POST', { ...baseBody, expenseId: ' ' }))
expect(response.status).toBe(400)
expect(mockFetch).not.toHaveBeenCalled()
})

it('trims a padded expense ID before building the upload URL', async () => {
mockFetch.mockResolvedValueOnce(
jsonResponse({ id: 'receipt_5', uri: 'https://s3.example.com/presigned' })
)

const response = await POST(
createMockRequest('POST', { ...baseBody, expenseId: ' expense_123 ' })
)
expect(response.status).toBe(200)
const [createUrl] = mockFetch.mock.calls[0]
expect(createUrl).toBe('https://api.brex.com/v1/expenses/card/expense_123/receipt_upload')
const data = await response.json()
expect(data.output.expenseId).toBe('expense_123')
})

it('rejects a whitespace-only receipt name', async () => {
const response = await POST(createMockRequest('POST', { ...baseBody, receiptName: ' ' }))
expect(response.status).toBe(400)
expect(mockFetch).not.toHaveBeenCalled()
})

it('rejects an API key containing header-breaking characters', async () => {
const response = await POST(
createMockRequest('POST', { ...baseBody, apiKey: 'bxt_test\r\nX-Injected: 1' })
)
expect(response.status).toBe(400)
expect(mockFetch).not.toHaveBeenCalled()
})

it('uses receipt match when no expense ID is provided', async () => {
mockFetch.mockResolvedValueOnce(
jsonResponse({ id: 'receipt_2', uri: 'https://s3.example.com/presigned' })
)

const response = await POST(
createMockRequest('POST', { apiKey: 'bxt_test_token', file: baseBody.file })
)
expect(response.status).toBe(200)
const data = await response.json()
expect(data.output).toEqual({
receiptId: 'receipt_2',
receiptName: 'receipt.pdf',
expenseId: null,
})

const [createUrl] = mockFetch.mock.calls[0]
expect(createUrl).toBe('https://api.brex.com/v1/expenses/card/receipt_match')
})

it('honors a receipt name override', async () => {
mockFetch.mockResolvedValueOnce(
jsonResponse({ id: 'receipt_3', uri: 'https://s3.example.com/presigned' })
)

const response = await POST(
createMockRequest('POST', { ...baseBody, receiptName: 'march-dinner.pdf' })
)
expect(response.status).toBe(200)
const [, createInit] = mockFetch.mock.calls[0]
expect(JSON.parse(createInit.body)).toEqual({ receipt_name: 'march-dinner.pdf' })
})

it('propagates Brex API errors', async () => {
mockFetch.mockResolvedValueOnce(jsonResponse({ message: 'Expense not found' }, 404))

const response = await POST(createMockRequest('POST', baseBody))
expect(response.status).toBe(404)
const data = await response.json()
expect(data.success).toBe(false)
expect(data.error).toContain('Expense not found')
expect(mockFetch).toHaveBeenCalledTimes(1)
expect(inputValidationMockFns.mockSecureFetchWithPinnedIP).not.toHaveBeenCalled()
})

it('rejects files over the 50 MB limit', async () => {
mockDownloadFileFromStorage.mockResolvedValueOnce(Buffer.alloc(50 * 1024 * 1024 + 1))

const response = await POST(createMockRequest('POST', baseBody))
expect(response.status).toBe(400)
const data = await response.json()
expect(data.error).toContain('50 MB')
expect(mockFetch).not.toHaveBeenCalled()
})

it('blocks pre-signed URLs that fail SSRF validation', async () => {
mockFetch.mockResolvedValueOnce(
jsonResponse({ id: 'receipt_6', uri: 'https://169.254.169.254/latest/meta-data' })
)
inputValidationMockFns.mockValidateUrlWithDNS.mockResolvedValueOnce({
isValid: false,
error: 'uri resolves to a blocked IP address',
})

const response = await POST(createMockRequest('POST', baseBody))
expect(response.status).toBe(502)
const data = await response.json()
expect(data.error).toContain('invalid upload URL')
expect(inputValidationMockFns.mockSecureFetchWithPinnedIP).not.toHaveBeenCalled()
})

it('fails when the pre-signed upload fails', async () => {
mockFetch.mockResolvedValueOnce(
jsonResponse({ id: 'receipt_4', uri: 'https://s3.example.com/presigned' })
)
inputValidationMockFns.mockSecureFetchWithPinnedIP.mockResolvedValueOnce(jsonResponse({}, 403))

const response = await POST(createMockRequest('POST', baseBody))
expect(response.status).toBe(502)
const data = await response.json()
expect(data.success).toBe(false)
})

it('denies access to files the caller cannot read', async () => {
const deniedResponse = new Response(
JSON.stringify({ success: false, error: 'File not found' }),
{
status: 404,
}
)
mockAssertToolFileAccess.mockResolvedValueOnce(deniedResponse)

const response = await POST(createMockRequest('POST', baseBody))
expect(response.status).toBe(404)
expect(mockFetch).not.toHaveBeenCalled()
})
})
Loading
Loading