Page MenuHomePhabricator

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

Authored by epriestley on Aug 13 2014, 9:07 PM.
Tags
None
Referenced Files
F18631389: D10260.id24696.diff
Tue, Sep 16, 3:13 PM
F18142780: D10260.id.diff
Aug 14 2025, 2:07 PM
F18120164: D10260.diff
Aug 13 2025, 10:23 AM
F18102637: D10260.id.diff
Aug 9 2025, 10:23 PM
Unknown Object (File)
Jun 6 2025, 9:50 AM
Unknown Object (File)
May 7 2025, 5:15 PM
Unknown Object (File)
May 5 2025, 10:33 PM
Unknown Object (File)
Apr 28 2025, 5:13 PM
Subscribers

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 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.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Aug 13 2014, 9:57 PM
epriestley updated this revision to Diff 24704.

Closed by commit rPef0460c1ffee (authored by @epriestley).