Skip to content

fix(discover): Add org id to project filter#116174

Merged
nsdeschenes merged 2 commits into
masterfrom
nd/BROWSE-519/fix-discover-add-organization-to-filter
May 26, 2026
Merged

fix(discover): Add org id to project filter#116174
nsdeschenes merged 2 commits into
masterfrom
nd/BROWSE-519/fix-discover-add-organization-to-filter

Conversation

@nsdeschenes
Copy link
Copy Markdown
Contributor

Add in org id to projects filter, and add a regression test.

Closes BROWSE-519

…n_id

DiscoverProcessor.get_projects accepted organization_id but never used
it in the query filter, allowing cross-org project lookup if called
without pre-validated project IDs. Add organization_id to the filter
to match ExploreProcessor.get_projects.

Refs VULN-1690
Co-Authored-By: Claude Opus 4 <noreply@anthropic.com>
@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 26, 2026

BROWSE-519

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 26, 2026
The get_projects filter now scopes by organization_id, so the test
project must belong to the same org used in ExportedData.
@nsdeschenes nsdeschenes marked this pull request as ready for review May 26, 2026 13:43
@nsdeschenes nsdeschenes requested a review from a team as a code owner May 26, 2026 13:43
Copy link
Copy Markdown
Contributor

@adrianviquez adrianviquez left a comment

Choose a reason for hiding this comment

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

Lgtm, though why is this needed now?

@nsdeschenes nsdeschenes merged commit 50400eb into master May 26, 2026
62 checks passed
@nsdeschenes nsdeschenes deleted the nd/BROWSE-519/fix-discover-add-organization-to-filter branch May 26, 2026 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants