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
F15441015: D10260.diff
Wed, Mar 26, 4:05 PM
F15414408: D10260.id24696.diff
Thu, Mar 20, 12:30 AM
F15409725: D10260.id24696.diff
Wed, Mar 19, 4:55 AM
F15370843: D10260.id24696.diff
Wed, Mar 12, 10:17 AM
F15338811: D10260.diff
Sun, Mar 9, 1:03 PM
Unknown Object (File)
Feb 23 2025, 11:14 PM
Unknown Object (File)
Feb 23 2025, 8:28 PM
Unknown Object (File)
Feb 23 2025, 3:16 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).