molecule-ai/molecule-core
53
0
Fork 2
Code Issues 188 Pull Requests 41 Actions 6 Packages Projects Releases Wiki Activity

[MEDIUM] Information Disclosure — raw DB errors returned to HTTP clients in DELETE /workspaces/:id #174

New Issue
Closed
opened 2026-05-09 21:32:13 +00:00 by core-offsec · 0 comments
core-offsec commented 2026-05-09 21:32:13 +00:00
Member
Copy Link
Copy Source

Finding

File: workspace-server/internal/handlers/workspace_crud.go, line 335
Severity: MEDIUM

Description

When CascadeDelete() returns an error, the HTTP handler passes err.Error() directly into the JSON response, leaking raw lib/pq / Postgres errors.

if err != nil {
    log.Printf("Delete: CascadeDelete(%s) failed: %v", id, err)
    c.JSON(http.StatusInternalServerError, gin.H{"error": err.Error()})
    return
}

CascadeDelete wraps raw lib/pq / database errors verbatim. Exposing these leaks internal DB schema, table names, and Postgres error formatting.

Repro steps

  1. Trigger CascadeDelete to return a DB error
  2. Observe HTTP 500 response body contains raw Postgres error string

Fix (already merged in PR #168)

c.JSON(http.StatusInternalServerError, gin.H{"error": "internal error processing delete request"})

Raw error already logged server-side — no observability lost.

References

  • Handler: workspace-server/internal/handlers/workspace_crud.go:335
  • CascadeDelete error paths: workspace-server/internal/handlers/workspace_crud.go:418-500
  • Secondary: org.go:610 leaks body.Dir in 404 response — also fixed in PR #168

Filed by: Core-OffSec

## Finding **File:** `workspace-server/internal/handlers/workspace_crud.go`, line 335 **Severity:** MEDIUM ### Description When `CascadeDelete()` returns an error, the HTTP handler passes `err.Error()` directly into the JSON response, leaking raw lib/pq / Postgres errors. ```go if err != nil { log.Printf("Delete: CascadeDelete(%s) failed: %v", id, err) c.JSON(http.StatusInternalServerError, gin.H{"error": err.Error()}) return } ``` `CascadeDelete` wraps raw `lib/pq` / database errors verbatim. Exposing these leaks internal DB schema, table names, and Postgres error formatting. ### Repro steps 1. Trigger `CascadeDelete` to return a DB error 2. Observe HTTP 500 response body contains raw Postgres error string ### Fix (already merged in PR #168) ```go c.JSON(http.StatusInternalServerError, gin.H{"error": "internal error processing delete request"}) ``` Raw error already logged server-side — no observability lost. ### References - Handler: `workspace-server/internal/handlers/workspace_crud.go:335` - `CascadeDelete` error paths: `workspace-server/internal/handlers/workspace_crud.go:418-500` - Secondary: `org.go:610` leaks `body.Dir` in 404 response — also fixed in PR #168 --- *Filed by: Core-OffSec*
claude-ceo-assistant added the tier:medium label 2026-05-09 21:38:55 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
area/ci
CI/CD pipeline issues
 do-not-auto-merge
Opt out of autonomous merge-queue merging
 kind/infrastructure
Infrastructure-related issues
 merge-queue
Ready for serialized Gitea merge queue
 merge-queue-hold
Temporarily hold PR in merge queue
 platform/go
Go platform test issues
 release-blocker
Blocks the staging→main promotion / a release
 release-test
security
test-label-sre
tier:high
High risk per dev-sop §SOP-6 — ceo only, 24h cooldown
 tier:low
Low risk per dev-sop §SOP-6 — engineers/managers/ceo can approve
 tier:medium
Medium risk per dev-sop §SOP-6 — managers/ceo can approve
 triage-test
test
 wip
Work in progress — do not auto-merge
No Label tier:medium
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
agent-dev-a agent-dev-b agent-pm agent-researcher agent-reviewer agent-reviewer-1 agent-reviewer-cr2 app-fe (Molecule AI · app-fe) app-lead (Molecule AI · app-lead) app-qa (Molecule AI · app-qa) claude-ceo-assistant claude-ci-reader claude-status-reaper core-be (Molecule AI · core-be) core-devops (Molecule AI · core-devops) core-fe (Molecule AI · core-fe) core-lead (Molecule AI · core-lead) core-offsec (Molecule AI · core-offsec) core-qa (Molecule AI · core-qa) core-security (Molecule AI · core-security) core-uiux (Molecule AI · core-uiux) cp-be (Molecule AI · cp-be) cp-lead (Molecule AI · cp-lead) cp-qa (Molecule AI · cp-qa) cp-security (Molecule AI · cp-security) cui (Zhanlin Cui) dev-lead (Molecule AI · dev-lead) devops-engineer documentation-specialist (Molecule AI · documentation-specialist) fullstack-engineer (Molecule AI · fullstack-engineer) godwin hongming hongming-ceo-delegated hongming-codex-laptop hongming-kimi-laptop hongming-pc2 hongming-personal infra-lead (Molecule AI · infra-lead) infra-runtime-be (Molecule AI · infra-runtime-be) infra-sre (Molecule AI · infra-sre) integration-tester (Molecule AI · integration-tester) mc-drift-bot molecule-code-reviewer plugin-dev (Molecule AI · plugin-dev) pm publish-runtime-bot pypi-publisher (Molecule AI PyPI Publisher (RFC#596)) release-manager (Molecule AI · release-manager) sdk-dev (Molecule AI · sdk-dev) sdk-lead (Molecule AI · sdk-lead) sop-drift-bot sop-tier-bot (SOP Tier-Check Bot) technical-writer (Molecule AI · technical-writer) triage-operator (Molecule AI · triage-operator)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: molecule-ai/molecule-core#174
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?