Skip to content

[VULN-118] Stop forwarding MongoDB TLS file paths on Cloud#19244

Merged
jvcalderon merged 4 commits into
masterfrom
fix/vuln-118-read-oracle
Jul 22, 2026
Merged

[VULN-118] Stop forwarding MongoDB TLS file paths on Cloud#19244
jvcalderon merged 4 commits into
masterfrom
fix/vuln-118-read-oracle

Conversation

@jvcalderon

@jvcalderon jvcalderon commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Description

MongoIntegration forwarded tlsCertificateKeyFile/tlsCAFile from the datasource config directly into the MongoDB driver as filesystem paths, regardless of environment. The builder UI only exposes these fields when SELF_HOSTED is true, but that gating was UI-only — the backend still accepted and used them on Cloud if present in the request payload, which is unsafe on a shared, multi-tenant server.

buildMongoClientOptions() now only forwards these fields into the driver when environment.SELF_HOSTED is true, enforcing server-side what the UI already implied. Self-hosted behavior is unchanged — that deployment model's server filesystem belongs to the customer, so the same concern doesn't apply there.

Addresses

Launchcontrol

Fixed a security issue affecting Budibase Cloud related to MongoDB datasource TLS configuration. Self-hosted behavior is unaffected.

MongoIntegration forwarded tlsCertificateKeyFile/tlsCAFile from the
datasource config directly into the MongoDB driver as filesystem
paths, regardless of environment. The builder UI only exposes these
fields when SELF_HOSTED is true, but that gating was UI-only — the
backend still accepted and used them on Cloud if present in the
request payload, which is unsafe on a shared, multi-tenant server.

buildMongoClientOptions() now only forwards these fields into the
driver when environment.SELF_HOSTED is true, enforcing server-side
what the UI already implied. Self-hosted behavior is unchanged.
@jvcalderon
jvcalderon force-pushed the fix/vuln-118-read-oracle branch from d1ec665 to e58aa31 Compare July 21, 2026 15:04
@jvcalderon jvcalderon self-assigned this Jul 21, 2026
@jvcalderon
jvcalderon marked this pull request as ready for review July 21, 2026 15:06

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

Comment thread packages/server/src/integrations/mongodb.ts Outdated
Comment thread packages/server/src/integrations/tests/mongodb.spec.ts
@jvcalderon
jvcalderon enabled auto-merge July 22, 2026 07:50
@jvcalderon
jvcalderon merged commit 5e19b93 into master Jul 22, 2026
36 checks passed
@jvcalderon
jvcalderon deleted the fix/vuln-118-read-oracle branch July 22, 2026 08:04
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 22, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants