Surface a better warning when the LDAP extension is not installed
ClosedPublic
Actions

Authored by epriestley on Aug 13 2014, 9:07 PM.

Tags

None

Referenced Files

	F19670928: D10260.id24696.diff
	Sat, Feb 7, 11:56 PM

	F19644675: D10260.id24696.diff
	Thu, Feb 5, 10:37 PM

	F19515728: D10260.id24696.diff
	Wed, Jan 14, 3:20 PM

	F19515727: D10260.id24704.diff
	Wed, Jan 14, 3:19 PM

	F19060833: D10260.id.diff
	Nov 29 2025, 10:24 AM

	F19058543: D10260.id24704.diff
	Nov 29 2025, 2:24 AM

	F19038174: D10260.id24696.diff
	Nov 26 2025, 2:53 AM

	F19033375: D10260.diff
	Nov 25 2025, 10:02 AM

View All Files

Subscribers

epriestley

Details

Reviewers

btrahan

Maniphest Tasks

Restricted Maniphest Task

Commits

Restricted Diffusion Commit
rPef0460c1ffee: Surface a better warning when the LDAP extension is not installed

Summary

Fixes T3347. We can't really do this one as a config thing since we don't know if the user wants to use LDAP.

Instead, just give them a better message than they otherwise get when they try to install/configure/use LDAP.

Test Plan

Faked it and got a reasonable message.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 24696.Aug 13 2014, 9:07 PM

epriestley retitled this revision from to Surface a better warning when the LDAP extension is not installed.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

epriestley added a task: Restricted Maniphest Task.

Herald added a subscriber: epriestley. · View Herald TranscriptAug 13 2014, 9:07 PM

Harbormaster completed remote builds in B2216: Diff 24696.Aug 13 2014, 9:08 PM

btrahan accepted this revision.Aug 13 2014, 9:57 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Aug 13 2014, 9:57 PM

Closed by commit rPef0460c1ffee (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

auth/

management/

PhabricatorAuthManagementLDAPWorkflow.php

2 lines

provider/

PhabricatorLDAPAuthProvider.php

14 lines

Diff 24704

View Options

src/applications/auth/management/PhabricatorAuthManagementLDAPWorkflow.php