Skip to content

fix(security): scope queries to project_id to prevent IDOR - #52

Open
tamish-max wants to merge 1 commit into
Litlyx:mainfrom
tamish-max:fix/idor-project-scope
Open

fix(security): scope queries to project_id to prevent IDOR#52
tamish-max wants to merge 1 commit into
Litlyx:mainfrom
tamish-max:fix/idor-project-scope

Conversation

@tamish-max

Copy link
Copy Markdown

Fixes #46

Security Issue

Two IDOR vulnerabilities allowed authenticated users to access/modify resources from other projects.

Fix

Added project_id filter to:

  1. TeamMemberModel.updateOne in members/edit.post.ts
  2. AiNewChatModel.findOne in AiService.ts

Fixes Litlyx#46 — Two IDOR vulnerabilities:
1. Team member edit allowed cross-project modification
2. AI chat retrieval allowed cross-project disclosure

Both fixed by adding project_id filter to database queries.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

Security: Cross-Project IDOR in Team Member Edit and AI Chat

1 participant