Skip to content
Snippets Groups Projects

internal/authz: remove potentially expensive conf check in SubRepoPermsClient

Closed Warren Gifford requested to merge remove-conf-check into main

Created by: bobheadxi

Callers should do the check themselves - having it inside SubRepoPermsClient means potentially fetching conf very frequently in tight loops, e.g. when iterating over a set of files for permissions

Merge request reports

Loading
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