Skip to content

Show authz provider problems in site alerts

Administrator requested to merge jc/authz-provider-site-alert into master

Created by: unknwon

Fixes https://github.com/sourcegraph/sourcegraph/issues/6061.

Notes:

  • Used ContributeXXX approach (similar to conf.ContributeValidator) for config warnings that should only be validated at runtime to avoid preventing server from starting.

Test plan:

  • Locally tested with a invalid authz provider config.
  • Unit tests for changes to conf.Problem.

Preview: Screen Shot 2019-10-31 at 3 07 34 PM

Merge request reports

Loading