Add a checkbox to the LDAP auth configuration UI to "Always Search"
ClosedPublic
Actions

Authored by epriestley on Apr 8 2014, 6:15 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Apr 20, 6:45 PM

	Unknown Object (File)
	Tue, Apr 16, 1:57 PM

	Unknown Object (File)
	Tue, Apr 16, 1:57 PM

	Unknown Object (File)
	Tue, Apr 16, 1:51 PM

	Unknown Object (File)
	Tue, Apr 16, 1:37 PM

	Unknown Object (File)
	Tue, Apr 16, 1:37 PM

	Unknown Object (File)
	Tue, Apr 16, 8:17 AM

	Unknown Object (File)
	Sun, Apr 14, 1:43 AM

View All Files

Subscribers

epriestley

Tokens

"Baby Tequila" token, awarded by Firehed.

Details

Reviewers

Firehed

Maniphest Tasks

Restricted Maniphest Task

Commits

Restricted Diffusion Commit
rP582ec5446532: Add a checkbox to the LDAP auth configuration UI to "Always Search"

Summary

Fixes T3208. This forces us to bind+search even if there are no anonymous credentials.

Test Plan

Checked the box, saved the form. Unchecked the box, saved the form. LDAP??

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 20684.Apr 8 2014, 6:15 PM

epriestley retitled this revision from to Add a checkbox to the LDAP auth configuration UI to "Always Search".

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.

Herald added a subscriber: epriestley.Apr 8 2014, 6:15 PM

Firehed accepted this revision.Apr 8 2014, 6:28 PM

Firehed edited edge metadata.

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

Closed by commit rP582ec5446532 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

resources/

sql/

patches/

20130619.authconf.php

6 lines

src/

applications/

auth/

provider/

PhabricatorAuthProviderLDAP.php

29 lines

Diff 20686

View Options

resources/sql/patches/20130619.authconf.php