Skip to content

fix(ui): exclude service filter from finding group resources endpoint#10652

Merged
Alan-TheGentleman merged 3 commits intomasterfrom
fix/ui-finding-group-resources-service-filter
Apr 10, 2026
Merged

fix(ui): exclude service filter from finding group resources endpoint#10652
Alan-TheGentleman merged 3 commits intomasterfrom
fix/ui-finding-group-resources-service-filter

Conversation

@Alan-TheGentleman
Copy link
Copy Markdown
Contributor

@Alan-TheGentleman Alan-TheGentleman commented Apr 10, 2026

Context

When a service filter (e.g. "account") was active on the finding groups page and a group row was expanded to show its resources, the filter[service__in] parameter was being forwarded to the finding group resources sub-endpoint. This filter is not valid for the resources endpoint, causing it to return empty results ("No resources found").

Description

Strip filter[service__in] from the filters passed to the finding group resources endpoint in normalizeFindingGroupResourceFilters. Uses a declarative FINDING_GROUP_ONLY_FILTERS constant so additional finding-group-only filters can be added easily if needed.

Steps to review

  1. Navigate to findings page with finding groups view
  2. Apply a Services filter (e.g. "account")
  3. Expand a finding group row
  4. Verify resources load correctly instead of showing "No resources found"

Checklist

UI

  • All issue/task requirements work as expected on the UI
  • Screenshots/Video of the functionality flow (if applicable) - Mobile (X < 640px)
  • Screenshots/Video of the functionality flow (if applicable) - Table (640px > X < 1024px)
  • Screenshots/Video of the functionality flow (if applicable) - Desktop (X > 1024px)
  • Ensure new entries are added to CHANGELOG.md, if applicable.

License

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

The service filter from the top-level finding groups page was being
forwarded to the resources sub-endpoint, causing "No resources found"
when a service filter was active.
@Alan-TheGentleman Alan-TheGentleman requested a review from a team as a code owner April 10, 2026 11:48
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 10, 2026

✅ All necessary CHANGELOG.md files have been updated.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 10, 2026

Conflict Markers Resolved

All conflict markers have been successfully resolved in this pull request.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 10, 2026

🔒 Container Security Scan

Image: prowler-ui:6c89c66
Last scan: 2026-04-10 11:54:55 UTC

✅ No Vulnerabilities Detected

The container image passed all security checks. No known CVEs were found.

📋 Resources:

@Alan-TheGentleman Alan-TheGentleman merged commit 1aa91cf into master Apr 10, 2026
37 checks passed
@Alan-TheGentleman Alan-TheGentleman deleted the fix/ui-finding-group-resources-service-filter branch April 10, 2026 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant