| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 1.6.4 |
| Request #2229 | Ability to attach fall through containers | ||
|---|---|---|---|
| Submitted: | 2004-08-26 17:49 UTC | Modified: | 2007-01-04 00:27 UTC |
| From: | brandon at osuosl dot org | Assigned: | aashley |
| Status: | Closed | Package: | Auth |
| PHP Version: | Irrelevant | OS: | Irrelevant |
| Roadmaps: | (Not assigned) | ||
[2004-08-26 17:49 UTC] brandon at osuosl dot org
Description:
------------
The ability to add a few different containers for fall trough would be great. For example try LDAP, then MySQL, then RADIUS.
This could be done using a simple method like addContainer().
I would be willing to submit a patch to do this if you guys are interested in adding the feature.
Comments
[2004-09-02 07:54 UTC] quinton at free dot fr
perhaps you should take care of account type and add a field in user table :
table user (
id int,
username text,
password text,
type int, # may be ldap, nis, radius, pop ...
first_name,
last_name
);
when user create his account, he has the ability to choose validation method.
[2006-02-21 01:06 UTC] aashley at php dot net
sounds interesting. would definatly be interested in an addition like this for the next feature addition version
[2006-08-09 14:28 UTC] lsmith
(Lukas Smith)
I know that LiveUser is a huge beast for most people, so it is definately not a replacement for adding this feature to Auth. That being said LiveUser already has the ability to use multiple auth containers in sequential order. It features many different containers (xml, database, session as well as a pear::auth container).
[2006-08-09 23:16 UTC] aashley at php dot net
(Adam Ashley)
The biggest problem with implementing this is the user management functions on the top level Auth object. Do they work on the first container? all of the containers? change the api to specify which container to work on? specify by integer? or add naming of containers?
I don't think this will really work as a patch for the 1.3 series while still maintaining full functionality of auth when using multiple containers.
Possibly a patch that only handles authentication when using multple containers could be accepted but that is likely it before 2.0.
[2007-01-04 00:27 UTC] aashley at php dot net
(Adam Ashley)
This bug has been fixed in CVS.
If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET).
If this was a problem with the pear.php.net website, the change should be live shortly.
Otherwise, the fix will appear in the package's next release.
Thank you for the report and for helping us make PEAR better.
This is implemented in CVS as the new Auth_Container_Multiple, with the insight from Adam Harvey to just implement it as another Auth Container took all of 20mins to implement, will be in Auth 1.5.0