Skip to content

Move SAML group filtering to GroupsFinder

Abdul Wadood requested to merge 461083-move-filter-member-expired-sso-to-groups-finder into master

What does this MR do and why?

The SAML group filtering logic is not specific to Organizations::GroupsFinder so it is moved to GroupsFinder.

A request spec is also added for organization groups to avoid a regression.

This also aligns with Fix GraphQL query returning less than expected ... (!160592 - merged).

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Query plan

The query hasn't changed here. If you want to see the query plans see Query plans.

How to set up and validate locally

See How to set up and validate locally

Related to #461083

Merge request reports