aboutsummaryrefslogtreecommitdiffstats
path: root/phpBB
diff options
context:
space:
mode:
authorJoseph Warner <hardolaf@hardolaf.com>2013-06-20 16:46:25 -0400
committerJoseph Warner <hardolaf@hardolaf.com>2013-06-20 16:47:55 -0400
commit0633666e2b5e39a7ebf7d2a68dc4c1b4dbbc0db1 (patch)
treefdc1a23606262b4aca6074acf02d2cb118d19d07 /phpBB
parent8214e6e8377b0858092e48aba3ba2a01994be47f (diff)
downloadforums-0633666e2b5e39a7ebf7d2a68dc4c1b4dbbc0db1.tar
forums-0633666e2b5e39a7ebf7d2a68dc4c1b4dbbc0db1.tar.gz
forums-0633666e2b5e39a7ebf7d2a68dc4c1b4dbbc0db1.tar.bz2
forums-0633666e2b5e39a7ebf7d2a68dc4c1b4dbbc0db1.tar.xz
forums-0633666e2b5e39a7ebf7d2a68dc4c1b4dbbc0db1.zip
[feature/auth-refactor] Fix LDAP conversion error
I messed up when converting over auth_ldap this commit fixes that error. I have not been able to extensively test ldap due to not having ldap set up on this computer yet. Apache authentication appears to work. PHPBB3-9734
Diffstat (limited to 'phpBB')
-rw-r--r--phpBB/includes/auth/provider_ldap.php6
1 files changed, 3 insertions, 3 deletions
diff --git a/phpBB/includes/auth/provider_ldap.php b/phpBB/includes/auth/provider_ldap.php
index 8270f50440..ee9b8100ee 100644
--- a/phpBB/includes/auth/provider_ldap.php
+++ b/phpBB/includes/auth/provider_ldap.php
@@ -338,11 +338,11 @@ class phpbb_auth_provider_ldap implements phpbb_auth_provider_interface
*
* @return string A filter string for ldap_search
*/
- public function ldap_user_filter($username)
+ private function ldap_user_filter($username)
{
global $config;
- $filter = '(' . $config['ldap_uid'] . '=' . ldap_escape(htmlspecialchars_decode($username)) . ')';
+ $filter = '(' . $config['ldap_uid'] . '=' . $this->ldap_escape(htmlspecialchars_decode($username)) . ')';
if ($config['ldap_user_filter'])
{
$_filter = ($config['ldap_user_filter'][0] == '(' && substr($config['ldap_user_filter'], -1) == ')') ? $config['ldap_user_filter'] : "({$config['ldap_user_filter']})";
@@ -354,7 +354,7 @@ class phpbb_auth_provider_ldap implements phpbb_auth_provider_interface
/**
* Escapes an LDAP AttributeValue
*/
- public function escape($string)
+ private function ldap_escape($string)
{
return str_replace(array('*', '\\', '(', ')'), array('\\*', '\\\\', '\\(', '\\)'), $string);
}