Skip to content

Security Review — 2026-07-08 — security-warning#137

Open
jdrumgoole wants to merge 1 commit into
mainfrom
security-review/2026-07-08
Open

Security Review — 2026-07-08 — security-warning#137
jdrumgoole wants to merge 1 commit into
mainfrom
security-review/2026-07-08

Conversation

@jdrumgoole

Copy link
Copy Markdown
Owner

Summary

Daily security review for 2026-07-08. 3 WARNING findings identified by a deeper owner_id data isolation audit.

WARNING findings (3)

  • WARN-1: winebox/routers/export.py:250 — Wine lookup in export_transactions missing owner_id filter
  • WARN-2: winebox/services/cellar_event_view.py:179 — Bulk Wine lookup in list_events_as_transactions missing owner_id filter
  • WARN-3: winebox/services/cellar_event_view.py:219 — Single Wine lookup in get_event_as_transaction missing owner_id filter

All three are secondary lookups where the wine_ids are derived from owner-scoped data. MongoDB ObjectIds are globally unique, so exploitation requires data corruption. However, these violate the project's stated security policy that all user data queries must filter by owner_id.

New INFO findings (2)

  • INFO-38: Pillow 12.2.0 — 3 new CVEs (CVE-2026-55379, CVE-2026-55380, CVE-2026-55798), none exploitable via WineBox
  • INFO-39: Six update_one calls that don't re-assert owner_id in the filter (defence-in-depth)

Carried forward

  • 37 INFO findings from previous review, unchanged
  • No code changes since last review
  • All dependency versions unchanged

Top 3 priorities

  1. Fix WARN-1/2/3 — simple one-line fixes to add owner_id to three query filters
  2. Batch dependency upgrade (cryptography, python-multipart, PyJWT, Pillow) — closes 12 CVEs
  3. Plan coordinated FastAPI + Starlette upgrade (4 accumulated CVEs)

Full report: docs/security-reports/2026-07-08.md


Generated by Claude Code

3 WARNING findings: secondary Wine lookups in export/transaction views
missing owner_id filter (WARN-1/2/3). 2 new INFO findings: Pillow CVEs
(INFO-38) and update_one defence-in-depth (INFO-39). 37 INFO carried
forward.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PTKbdDdHseAP7G7u1c6aRv
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants