{"id":"CVE-2026-54351","title":"Budibase: Mass Assignment in Webhook Trigger Allows Cross-Workspace Automation Execution via appId Override","summary":"Budibase: Mass Assignment in Webhook Trigger Allows Cross-Workspace Automation Execution via appId Override","severity":"high","cvss":8.2,"cwe":["CWE-915"],"vendor":"budibase","product":"@budibase/server","ecosystem":"npm","affected":["@budibase/server < 3.39.9"],"patched":["@budibase/server 3.39.9"],"published":"2026-06-22","updated":"2026-06-22","source":"GHSA","sourceUrl":"https://github.com/advisories/GHSA-rgvg-3wpc-h44p","references":[{"url":"https://github.com/Budibase/budibase/security/advisories/GHSA-rgvg-3wpc-h44p"},{"url":"https://github.com/advisories/GHSA-rgvg-3wpc-h44p"}],"tags":["ghsa","npm"],"epss":0.00461,"epssPercentile":0.3912,"ingestedAt":"2026-06-29T13:24:35.487Z","slug":"CVE-2026-54351","body":"## Overview\n\n## Summary\n\nThe webhook trigger endpoint in Budibase is publicly accessible and passes the full HTTP request body into automation execution parameters. A mass assignment vulnerability in `externalTrigger()` allows an attacker to overwrite the internal `appId` property by including it in the webhook POST body. When the automation is processed asynchronously (the default path for webhooks without a collect step), the worker executes the attacker-defined automation in the context of the victim's workspace, granting full read/write access to the victim's database.\n\n## Details\n\nThe webhook trigger route is registered as a public endpoint with no authentication:\n\n```typescript\n// packages/server/src/api/routes/webhook.ts:12\npublicRoutes.post(\"/api/webhooks/trigger/:instance/:id\", controller.trigger)\n```\n\nThe controller passes the raw request body as `fields` alongside the server-derived `appId`:\n\n```typescript\n// packages/server/src/api/controllers/webhook.ts:142-148\nawait triggers.externalTrigger(target, {\n  fields: {\n    ...ctx.request.body,  // attacker-controlled\n    body: ctx.request.body,\n  },\n  appId: prodAppId,       // server-controlled\n})\n```\n\nIn `externalTrigger()`, for webhook-triggered automations, `params.fields` is spread back into `params`:\n\n```typescript\n// packages/server/src/automations/triggers.ts:237-241\nparams = {\n  ...params,          // appId: prodAppId (server-controlled)\n  ...params.fields,   // appId: VICTIM_ID (attacker-controlled, overwrites above)\n  fields: {},\n}\n```\n\nBecause `params.fields` is spread **after** `params`, any key in the attacker's body overwrites the corresponding property in `params`. An attacker including `\"appId\": \"app_VICTIM_WORKSPACE_ID\"` in the POST body overwrites the legitimate, server-derived `appId`.\n\nThe contaminated params become `data.event` and are queued asynchronously:\n\n```typescript\n// packages/server/src/automations/triggers.ts:244,271\nconst data: AutomationData = { automation, event: params }\n// ...\nreturn quotas.addAction(() => automationQueue.add(data, JOB_OPTS))\n```\n\nThe async worker uses `job.data.event.appId` to set the workspace context:\n\n```typescript\n// packages/server/src/threads/automation.ts:917,929-930\nconst workspaceId = job.data.event.appId  // attacker-controlled\n// ...\nreturn await context.doInAutomationContext({\n  workspaceId,  // victim's workspace\n  automationId,\n  task: async () => { /* automation steps run here */ }\n})\n```\n\nThe synchronous path (for webhooks with a collect step) correctly overwrites `appId` at `triggers.ts:264`:\n```typescript\ndata.event = {\n  ...data.event,\n  appId: context.getWorkspaceId(),  // server-controlled fix\n  automation,\n}\n```\n\nThis proves the developers intended `appId` to be server-controlled but missed applying the same fix to the async path, which is the default for all webhooks without a collect step.\n\n## PoC\n\n**Prerequisites:** Attacker has builder access to their own Budibase workspace and knows a victim workspace ID (format: `app_<uuid>`).\n\n**Step 1:** Attacker creates an automation in their own workspace with a webhook trigger and data-exfiltration steps (e.g., Query Rows → Execute Script to send data externally).\n\n**Step 2:** Attacker creates a webhook for that automation and notes the webhook URL:\n```\nPOST /api/webhooks/trigger/<ATTACKER_INSTANCE>/<WEBHOOK_ID>\n```\n\n**Step 3:** Attacker triggers the webhook with the victim's workspace ID injected into the body:\n\n```bash\ncurl -X POST https://budibase.example.com/api/webhooks/trigger/app_ATTACKER_ID/wh_WEBHOOK_ID \\\n  -H 'Content-Type: application/json' \\\n  -d '{\"appId\": \"app_VICTIM_WORKSPACE_ID\", \"normalData\": \"test\"}'\n```\n\n**Expected result:** The automation defined in the attacker's workspace executes in the context of the victim's workspace. All database operations (Query Rows, Create Row, Delete Row, Execute Script, etc.) operate on the victim's data.\n\n**Additional overridable fields via the same mechanism:**\n- `timeout` (`automation.ts:443-444`): override automation execution timeout\n- `user` (`automation.ts:413,435`): set user context for automation steps\n- `metadata.automationChainCount` (`automation.ts:293`): bypass chain depth limits\n\n## Impact\n\nAn attacker with builder access to their own Budibase workspace can execute arbitrary automations (of their own design) in the context of any other workspace on the same Budibase instance, provided they know the victim's workspace ID. This enables:\n\n- **Full data exfiltration**: Query Rows steps read all tables in the victim's workspace\n- **Data manipulation**: Create Row, Update Row, Delete Row steps modify victim data\n- **Arbitrary code execution in victim context**: Execute Script steps run JavaScript with access to victim's environment variables and database\n- **Cross-tenant boundary violation**: In multi-tenant deployments (Budibase Cloud), the tenant ID is derived from the workspace ID, so the attack crosses tenant boundaries\n\nThe attack requires no authentication (the webhook endpoint is public) and leaves minimal audit trail since the automation execution is attributed to the attacker's automation definition but runs in the victim's context.\n\n## Recommended Fix\n\nIn `packages/server/src/automations/triggers.ts`, apply the same `appId` fix that exists in the synchronous path to the async path as well. The fix should ensure `appId` is always server-controlled before queuing:\n\n```typescript\n// packages/server/src/automations/triggers.ts:244-272\nconst data: AutomationData = { automation, event: params }\n\n// ... trigger filter check ...\n\n+ // Ensure appId is always server-controlled, not user-supplied\n+ data.event.appId = context.getWorkspaceId()\n\nif (getResponses) {\n  data.event = {\n    ...data.event,\n    appId: context.getWorkspaceId(),\n    automation,\n  }\n  return quotas.addAction(() =>\n    executeInThread({ data } as AutomationJob, { onProgress })\n  )\n} else {\n  return quotas.addAction(() => automationQueue.add(data, JOB_OPTS))\n}\n```\n\nAlternatively, use an allowlist approach for the webhook field spread to prevent any internal property from being overwritten:\n\n```typescript\n// packages/server/src/automations/triggers.ts:237-241\nconst { appId, timeout, user, metadata, ...safeFields } = params.fields\nparams = {\n  ...params,\n  ...safeFields,\n  fields: {},\n}\n```\n\n## Affected packages\n\n- `@budibase/server < 3.39.9`\n\n## Remediation\n\nUpgrade to a patched release:\n\n- `@budibase/server 3.39.9`","depth":"twilight","depthScore":45,"depthScoreParts":{"impact":45.1,"likelihood":0.1,"exploitation":0,"ransomware":0},"changes":[]}