revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-26 08:42:41 +01:00
Code Issues Releases Wiki Activity

Fix an issue with setting "Real Name Attributes" in LDAP auth

Browse source 
Summary: We currently don't read/save this value correctly. Fix the issue. Ref T1536.

Test Plan: Set real name attributes to "x, y".

Reviewers: btrahan

Reviewed By: btrahan

CC: aran, colegleason

Maniphest Tasks: T1536

Differential Revision: https://secure.phabricator.com/D6388
This commit is contained in:
epriestley 2013-07-09 16:22:50 -07:00
parent 13e2489739
commit 1888a84b7e
1 changed files with 2 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
src/applications/auth/provider/PhabricatorAuthProviderLDAP.php
View file

@ -185,20 +185,7 @@ final class PhabricatorAuthProviderLDAP
const KEY_ACTIVEDIRECTORY_DOMAIN = 'ldap:activedirectory-domain'; const KEY_ACTIVEDIRECTORY_DOMAIN = 'ldap:activedirectory-domain';
private function getPropertyKeys() { private function getPropertyKeys() {
return array( return array_keys($this->getPropertyLabels());
self::KEY_HOSTNAME,
self::KEY_PORT,
self::KEY_DISTINGUISHED_NAME,
self::KEY_SEARCH_ATTRIBUTE,
self::KEY_USERNAME_ATTRIBUTE,
self::KEY_VERSION,
self::KEY_REFERRALS,
self::KEY_START_TLS,
self::KEY_ANONYMOUS_USERNAME,
self::KEY_ANONYMOUS_PASSWORD,
self::KEY_SEARCH_FIRST,
self::KEY_ACTIVEDIRECTORY_DOMAIN,
);
} }
private function getPropertyLabels() { private function getPropertyLabels() {
@ -239,6 +226,7 @@ final class PhabricatorAuthProviderLDAP
break; break;
} }
} }
return $values; return $values;
} }