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
F15370843: D10260.id24696.diff
Wed, Mar 12, 10:17 AM
F15338811: D10260.diff
Sun, Mar 9, 1:03 PM
Unknown Object (File)
Sun, Feb 23, 11:14 PM
Unknown Object (File)
Sun, Feb 23, 8:28 PM
Unknown Object (File)
Sun, Feb 23, 3:16 PM
Unknown Object (File)
Feb 2 2025, 3:50 PM
Unknown Object (File)
Jan 31 2025, 4:47 PM
Unknown Object (File)
Jan 27 2025, 3:17 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).