[phpldapadmin] [ phpldapadmin-Bugs-2826573 ] re-enable login, string

Bugs item #2826573, was opened at 2009-07-24 09:38
Message generated for change (Comment added) made by stran2005
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=498546&aid=2826573&group_id=61828

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: 1.2.x
Status: Pending
Resolution: None
Priority: 5
Private: No
Submitted By: Aurélien Requiem (requie_a)
Assigned to: Nobody/Anonymous (nobody)
Summary: re-enable login,string

Initial Comment:
Hi,

In config.php, I've defined the following line as shown is config.php.exemple
$servers->setValue('login','string','uid=<username>,ou=addressbook,ou=accounts,dc=foo,dc=bar');

When I refresh the webpage to log in, I have the following message:
Function error called incorrectly [ERROR: Setting a index [login,string] that isnt predefined.]

If the line is commented/removed, all is fine and I have the log in page, but sadly not with the expected functionality.

Cheers

----------------------------------------------------------------------

Comment By: Samuel Tran (stran2005)
Date: 2009-08-19 12:25

Message:
I was able to fix that problem with those two patches:

1)

--- /tmp/phpldapadmin-1.2.0.1/lib/ds_ldap.php   2009-08-13
08:04:16.000000000 -0400
+++ lib/ds_ldap.php     2009-08-19 12:24:14.000000000 -0400
@@ -79,6 +79,11 @@
                        'desc'=>'Limit logins to users who match any of
the following LDAP filters',
                        'default'=>array());

+                $this->default->login['string'] = array(
+                        'desc'=>'Login string if using auth_type=string',
+                        'var'=>'login_string',
+                        'default'=>null);
+
                $this->default->proxy['attr'] = array(
                        'desc'=>'Attribute to use to find the users DN for
proxy based authentication',
                        'default'=>array());

2)
--- /tmp/phpldapadmin-1.2.0.1/lib/ds_ldap_pla.php       2009-08-13
08:04:16.000000000 -0400
+++ lib/ds_ldap_pla.php 2009-08-19 12:14:49.000000000 -0400
@@ -147,6 +147,13 @@
                        return
$this->getValue('appearance','show_create');
        }

+        /**
+         * Return the attribute used for login
+         */
+        function getLoginAttr() {
+                return $this->login_attr;
+        }
+
        /**
         * Fetches whether the login_attr feature is enabled for a
specified server.
         *


----------------------------------------------------------------------

Comment By: markwu05 (markwu05)
Date: 2009-08-14 18:57

Message:
I need to have the string function in 1.2 cause I have a stripped-down
version that sets base dn to "ou=group,dc=xxx,dc=com" so our users to edit
groups only, however, login does not work without string
"uid=username,ou=people,dc=xxx,dc=com".  setting login_attr to  Uid does
not work either under base DN "ou=group,dc=xxx,dc=com".

----------------------------------------------------------------------

Comment By: Deon George (wurley)
Date: 2009-08-12 18:47

Message:
Actually, I'm considering not re-iplementing this function. The same result
can be achieved by using an attribute based login. ie: setting
$servers->setValue('login','attr','uid');.

What does this feature give over and above using login,attr?

----------------------------------------------------------------------

Comment By: Deon George (wurley)
Date: 2009-07-24 09:43

Message:
Ahh, yes, login string hasnt been re-implemented in 1.2 (PLA's object was
completely reworked for 1.2)... I'll leave this bug open to remind me to
re-implement it.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=498546&aid=2826573&group_id=61828

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with 
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
______________________________________
phpLDAPadmin development mailing list.
To unsbuscribe: https://lists.sourceforge.net/lists/listinfo/phpldapadmin-devel
http://phpldapadmin.sourceforge.net/