Skip to content
Snippets Groups Projects

authz: Remove sub-repo SupportedChecker

Merged Administrator requested to merge 503/remove-supported-checker into main

Created by: ryanslade

Sub-repo permissions are additional filters applied after we have already checked our existing repo permissions.

It is therefore not necessary to check whether sub-repo perms are enabled for a repo. If they are, we'll have data in the sub_repo_permissions table, which we plan to cache. If not, we assume that the user has access to the whole repo based on our traditional repo permissions checks.

SubRepoPermsClient was also exported as it is bad practice to have constructors return private structs.

Thanks @bobheadxi for spotting this simplification!

Merge request reports

Merged by avatar (Jul 5, 2025 4:31am UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading