D8395.id19945.diff
No OneTemporary
Actions

Size

1 KB

Referenced Files

None

Subscribers

None

D8395.id19945.diff
View Options

	diff --git a/src/applications/config/check/PhabricatorSetupCheckAuth.php b/src/applications/config/check/PhabricatorSetupCheckAuth.php
	--- a/src/applications/config/check/PhabricatorSetupCheckAuth.php
	+++ b/src/applications/config/check/PhabricatorSetupCheckAuth.php
	@@ -3,8 +3,21 @@
	final class PhabricatorSetupCheckAuth extends PhabricatorSetupCheck {

	protected function executeChecks() {
	- $providers = PhabricatorAuthProvider::getAllEnabledProviders();
	- if (!$providers) {
	+ // NOTE: We're not actually building these providers. Building providers
	+ // can require additional configuration to be present (e.g., to build
	+ // redirect and login URIs using `phabricator.base-uri`) and it won't
	+ // necessarily be available when running setup checks.
	+
	+ // Since this check is only meant as a hint to new administrators about
	+ // steps they should take, we don't need to be thorough about checking
	+ // that providers are enabled, available, correctly configured, etc. If
	+ // they've ever touched the Auth app, this warning probably isn't useful.
	+
	+ $configs = id(new PhabricatorAuthProviderConfigQuery())
	+ ->setViewer(PhabricatorUser::getOmnipotentUser())
	+ ->execute();
	+
	+ if (!$configs) {
	$message = pht(
	'You have not configured any authentication providers yet. You '.
	'should add a provider (like username/password, LDAP, or GitHub '.