Page MenuHomePhabricator

Add an "Always Search" flag to the libphutil LDAP adapter
ClosedPublic

Authored by epriestley on Apr 8 2014, 6:13 PM.
Tags
None
Referenced Files
F14073704: D8722.diff
Thu, Nov 21, 1:47 AM
Unknown Object (File)
Fri, Nov 15, 11:54 PM
Unknown Object (File)
Tue, Nov 12, 1:49 AM
Unknown Object (File)
Thu, Nov 7, 9:40 PM
Unknown Object (File)
Wed, Nov 6, 10:24 PM
Unknown Object (File)
Tue, Oct 29, 2:44 AM
Unknown Object (File)
Thu, Oct 24, 10:28 AM
Unknown Object (File)
Oct 16 2024, 5:43 PM
Subscribers

Details

Reviewers
Firehed
Maniphest Tasks
Restricted Maniphest Task
Commits
rPHU3435a259ab0f: Add an "Always Search" flag to the libphutil LDAP adapter
Summary

Ref T3208. This adds a flag which acts like the old "Search First" flag did, more or less, probably. LDAP?

Specifically, the issue at heart here is that some LDAP servers require totally uncredentialed binds, some require "anonymous" binds (not sure if this is a term of art or if whoever contributed the first LDAP patch made it up), and some require full DN binds. This flag lets us distinguish between the "full DN" and "completely uncredentialed" use cases, both of which have no "anonymous" credentials.

Test Plan

It compiles!

Diff Detail

Repository
rPHU libphutil
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Add an "Always Search" flag to the libphutil LDAP adapter.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: Firehed.
epriestley added a task: Restricted Maniphest Task.
Firehed edited edge metadata.

Worked on my install combined with D8723 :D

This revision is now accepted and ready to land.Apr 8 2014, 6:28 PM

Also tested that incorrect credentials fail. Because, you know, security. All looks good here!

epriestley updated this revision to Diff 20685.

Closed by commit rPHU3435a259ab0f (authored by @epriestley).