GHSA-hr66-5mqr-8mpx Budibase: Unauthenticated user information disclosure via public tenant user lookup endpoint
SummaryThe Budibase Worker service exposes a public, unauthenticated API endpoint (`GET /api/global/users/tenant/:id`) that returns sensitive user information including `tenantId`, `userId`, `email`, and `ssoId`. The endpoint is registered in the `PUBLIC_ENDPOINTS` list with a `TODO` comment acknowledging it "should be an internal API." Any unauthenticated party can enumerate user emails or IDs to extract sensitive tenant and user metadata, enabling targeted attacks against multi-tenant deployments.Details**Public endpoint registration** at `packages/worker/src/api/index.ts` lines 56-59:This endpoint is listed in `PUBLIC_ENDPOINTS`, which is passed to `auth.buildAuthMiddleware(PUBLIC_ENDPOINTS)` at line 154. When a request matches a public endpoint pattern, the authentication middleware sets `ctx.publicEndpoint = true` and calls `next()` without performing any authentication (verified at `packages/backend-core/src/middleware/authenticated.ts` lines 124-126, 249-251).All subsequent middleware also skips for public endpoints:`buildTenancyMiddleware` — passes through`activeTenant` — passes through`buildCsrfMiddleware` — skipped for GET methods (line 48 of csrf.ts)The `budibaseAccess` gate at lines 160-168 explicitly returns `next()` when `ctx.publicEndpoint` is true**Route registration** at `packages/worker/src/api/routes/global/users.ts` line 139:`loggedInRoutes` has no auth middleware group — it is created with `endpointGroupList.group()` (no middleware).**Handler implementation** at `packages/worker/src/api/controllers/global/users.ts` lines 548-562:The `id` parameter accepts either an email address (detected by `@` presence) or a user ID. The response returns the **full** `PlatformUser` object from `packages/types/src/documents/platform/users.ts`:The lookup function (`packages/backend-core/src/users/lookup.ts:48-53`) queries the `PLATFORM_USERS_LOWERCASE` CouchDB view with `include_docs: true`, returning the complete platform user document including CouchDB `_id` and `_rev`.**Affected files:**`packages/worker/src/api/index.ts:56-59` — Public endpoint registration`packages/worker/src/api/routes/global/users.ts:139` — Route on unauthenticated group`packages/worker/src/api/controllers/global/users.ts:548-562` — Handler returning full user object`packages/backend-core/src/users/lookup.ts:48-53` — Platform user lookup with `include_docs: true``packages/types/src/documents/platform/users.ts:6-36` — PlatformUser typesPoC**Static verification:**Observe `packages/worker/src/api/index.ts:56-59`: endpoint in `PUBLIC_ENDPOINTS` with `// TODO: This should be an internal api`Trace handler at `packages/worker/src/api/controllers/global/users.ts:548-562`: no auth checks, returns `ctx.body = user` (full object)Trace middleware chain: all middleware passes through for `ctx.publicEndpoint === true`Confirm no field filtering, sanitization, or authorization between request and response**Dynamic verification (requires running Budibase instance with at least one user):****Negative case:** Requesting a non-existent user returns HTTP 400 with `"No tenant user found."`, while an existing user returns HTTP 200 with full data. The different status codes confirm user existence, enabling enumeration.ImpactThis is a **CWE-200: Exposure of Sensitive Information to an Unauthorized Actor** vulnerability.**Who is impacted:** All Budibase deployments — both self-hosted and cloud. The impact is highest for multi-tenant (cloud) deployments where tenant IDs are security boundaries and user enumeration across tenants enables targeted attacks.An unauthenticated attacker can:**Enumerate all user accounts** by testing known or guessed email addresses against the endpoint**Extract tenant IDs** for any known user, enabling targeted cross-tenant attacks**Extract user IDs** (`userId`) for use in other API calls or attacks**Extract SSO identifiers** (`ssoId`) which may link to external identity providers (Google, OIDC)**Confirm user existence** through different HTTP responses (200 vs 400)**Harvest CouchDB revision tokens** (`_rev`) which could assist in CouchDB-level attacksThe returned tenant IDs are particularly dangerous in multi-tenant deployments because they identify the security boundary between organizations. Combined with the hardcoded session keys (separate finding), an attacker could use enumerated tenant IDs to craft targeted session fixation attacks.Suggested remediation**Remove the endpoint from `PUBLIC_ENDPOINTS`** and move it to internal-only routes, as the TODO comment at line 56 already suggests**Add authentication and authorization** if the endpoint must remain accessible — require at least `builderOrAdmin` role**Limit returned fields** to only what the consumer actually needs (strip `_rev`, `ssoId`, and other sensitive fields)**Return generic 404** for both "not found" and "access denied" to prevent user enumeration**Add rate limiting** to prevent automated mass enumeration**Regression test:** Add a test verifying `GET /api/global/users/tenant/:id` returns 403 without authentication
CVSS v3.1 score: 7.5