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

Bugs item #2826573, was opened at 2009-07-24 13:38
Message generated for change (Comment added) made by sf-robot
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: Closed
Resolution: Rejected
Priority: 5
Private: No
Submitted By: Aurlien 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: SourceForge Robot (sf-robot)
Date: 2009-09-10 02:20

Message:
This Tracker item was closed automatically by the system. It was
previously set to a Pending status, and the original submitter
did not respond within 28 days (the time period specified by
the administrator of this Tracker).

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

Comment By: Deon George (wurley)
Date: 2009-08-21 05:23

Message:
Aurelien, Im still not keen on the login,string feature, however I do
recognise that it would be nice to have user's login from a distinct (set
of) base DN. IE: Login from a Base DN, however the tree shows a different
Base DN.

So Ive just implemented login,base as a configuration parameter which will
default to the LDAP servers base DN if not set (which is the default).

Thus, using an attr based login, and configuring login,base, you can have
your users authenticate against a "people" base, however, the tree only
shows a "group" tree.

It will be in 1.2.0.2, which I release probably over the weekend if you
want to test it before hand, you can get it here:
http://dev.phpldapadmin.org/devel/pla-dev-1.2.0.2.tar.bz2

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

Comment By: Samuel Tran (stran2005)
Date: 2009-08-19 16: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 22: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 22: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 13: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/