Skip to content

authz/github: enforce allowGroupsPermissionsSync if groupsCacheTTL is enabled

Warren Gifford requested to merge github/validate-allowGroupsPermissionsSync into main

Created by: bobheadxi

allowGroupsPermissionsSync is actually a prerequisite to enabling groupsCacheTTL. allowGroupsPermissionsSync was introduced in https://github.com/sourcegraph/sourcegraph/pull/24328. This PR checks if allowGroupsPermissionsSync is enabled, and if not, forcibly sets groupsCacheTTL to nil and reports a warning.

This is a breaking change, but it is currently flagged as an experimental feature and opt-in only, so will stick to a changelog item.

Merge request reports