Closed (duplicate)
Project:
Lightweight Directory Access Protocol
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
9 Nov 2010 at 16:05 UTC
Updated:
13 Apr 2011 at 21:53 UTC
Jump to comment: Most recent file
Comments
Comment #1
johnbarclay commentedI roughed out the user interface for ldap authorization. Attached is a pdf. Looking it over will give a sense of the functionality I'm looking for. If I'm missing anything, please add it as a feature request with the subject LDAP Authorization: ....
Remaining tasks to LDAP Authorization:
- connect admin form to database
- connect module to configuration data
- establish default values for configuration
- connect to LDAP Api functions
- clean up the form instructions with advanced help and more examples
Comment #2
johnbarclay commentedI'm using this thread for general progress updates.
Had a good Saturday on this on the ldap authorization component.
Here's the approach I'm thinking for wrapping LDAP up: http://www.gliffy.com/publish/2315088
Though its pretty flexible.
Here's a sequence diagram for ldap authorization, though its up for debate still.
http://www.gliffy.com/publish/2318063/
I have some placeholder code in place for the api part and am hoping to get an alpha of ldap authorization (no authentcation + placeholder api code) early next week.
At that point, we could use some other coders for ldap authorization for:
- code review
- UI review
- Implement exportables for it
- Accessibility review
- unit tests
Comment #3
jsloan commentedComing back to the party after helping with a similar module for 4.7 #63899: Integration with profile module not working on 4.7.0. Is there a working release of LDAP for D7? I just downloaded ldap-7.x-1.x-dev.tar.gz but get several errors...
thanks
jsloan
Comment #4
geste commentedI think the main developers would say that this new module is pre-alpha -- not working yet. I say "new" because it is not a fork of ldap_integration; rather it is more like "start over" for reasons that were explained previously.
So, very exciting, but very early.
Jim
Comment #5
jsloan commentedSounds good! I'll stay tuned and will help as much as I can.
~jsloan
Comment #6
wernercd commentedAny chance to get a "pre-alpha" update for download? This is eagerly awaited by more than me I'm sure...
Comment #7
johnbarclay commentedAttached is:
Most of the functionality of these two will be in the ldap_api part of the package which retsamedoc is working on.
Comment #8
johnbarclay commentedOh. and I think ldap_authentication is up for grabs as far as development currently. I will get back to it after ldap_authorization, but ideally someone else would jump in and move it forward. retsamedoc is working on the ldap api part.
Comment #9
johnbarclay commentedAttached is the pre alpha of ldap_authorization and a temporary ldap_servers and ldap_api_temp (these last 2 may be in the api part of the module in the end).
Since there is no ldap_authentication yet, its only good for the use case where authentication is not ldap, but authorization is.
I need to flush out the configuration a bit to clarify what should be done when ldap_authorization gives a user a role, but the role is removed manually.
I'm going to check in with others and see how this can fit into cvs.
Comment #10
wernercd commentedI just wanted to say thanks for the pre_alpha build to play with. Haven't had a chance to test the latest and greatest, but it's definitely a very important tool that you are building and your hard work is greatly appreciated. :)
Comment #11
johnbarclay commentedretsamedoc, netw3rker and I had a short Skype last night. I'm following up on the ldap authentication part of the module and retsamedoc is plugging away at the API. netw3rker took on the tricky use case where a user needs to merge multiple accounts generated by different authentication modules.
Attached is a pdf of my first shot at the admin interface for ldap authentication. Looking for feedback on the functionality the form implies as well as wording. Comments on the pdf refer to drupal issues.
The classes behind the ldap authentication configuration are attached also if anyone is interested. After I get this functional, I'll get check it in.
Comment #12
johnbarclay commentedGetting close on ldap_authentication. The new d7 user.module has simplified some things. I made a sequence diagram of sorts at http://www.gliffy.com/publish/2362004/ to get my head around the code. I just need to test and debug then I will check in ldap_authentication and ldap_authorization and the temporary api files I'm using. I've already checked the ldap_authorization in.
I'm working against D7 RC1.
Here's the way I see the overall project workflow:
http://www.gliffy.com/publish/2315088/
Comment #13
johnbarclay commentedI checked into head, d7 rc2 code for ldap_authentication and ldap_authorization. The api and servers part is still temporary code until the api is done. I also threw in an ldap_developer module that I use for breaking down and rebuilding ldap configurations. It may be handy for testers.
I would definately call this alpha code and I've done very little testing. Its far enough along that:
1) patches within the ldap_authorization and ldap_authentication will be helpful.
2) I'm going to start moving more of my @todos into the issue queue
3) anyone interested in working on exportables, unit tests, drush, accessibility, usability, etc can jump right in on ldap_authentication and ldap_authorization.
Here's the flow for ldap_authentication:
http://www.gliffy.com/publish/2362004/
Comment #14
SilviaT commentedThe project seems to be idle, any progress? I've been testing LDAP servers so far and it seems to work fine.
Any news with the others?
Comment #15
johnbarclay commentedYes. Progress is being made. The bugs and unfinished features in unstable 4 are mostly resolved. There were enough of them that it makes no sense to test until I get unstable 5 out.
Comment #16
johnbarclay commentedReleased 7.x-1.0-unstable5.
Main focus was on ldap_authentication. Next will be ldap_authorization. I can really use some help with articulating test for ldap_authorization and cleaning up the UI and creating documentation.
Here's the unstable 5 changes:
- install instructions: disable and uninstall previous ldap modules. make sure tables are removed.
------------------
- main focus of unstable 5 is ldap_authentication including finishing out features, improving usability,
fixing bugs, documentation and setting up tests. ldap_authorization still needs features to be
finished out, documentation and usability improvements, and a set of tests.
- ldap_authentication: reordered authentication validation related to issue #1022362
- ldap_authentication: added test grid for ldap_authentication testing: http://drupal.org/node/1053818
- ldap_authentication: added typical authentication configurations: http://drupal.org/node/1053748
- ldap_help: added "status" and "issue reporting" tabs to improve testing and bug reporting
- ldap_authentication: removed option to allow ldap users to change, have separate drupal email. this feature needs more work if desired at all.
- ldap_authentication: added configurable help link for ldap password resetting.
- ldap_authentication: made more content themeable, especially where localized ldap user help may be a factor
- ldap_authentication: #807416 finished and tested white lists
- ldap_authentication: clarified and tested rules for existing drupal account conflicts. Desired behavior is in
tests (ldap_authentication.tests.txt).
- ldap_authentication: #968574, #256226, #258974 added "Account Creation for LDAP Authenticated Users" section for account creation. Desired behavior is in
tests (ldap_authentication.tests.txt). Pushed some configuration over to user settings page.
- ldap_servers: #1050590 added check for existance of https in $_SERVER array per
- ldap_authorization: #1050944 fixed array work ldap to role mapping.
- ldap_servers: improved wording on required https message and made themeable
- ldap_authentication: #1023510 several tweaks for ldap authentication settings related to location and visibility of password,
password reset links, account creation links, etc. Many of the combinations made no sense and the form was overly complex. The logic is
articulated in the ldap authentication settings form and ldap_authentication.tests.txt
- ldap_authentication: moved request account and password reset link hide/show logic from theming layer to menu/permission layer.
makes more sense and reduced code alot.
- another trip through drupal code module\
- ldap_authorization: #1026138 bug that didn't remove past ldap applied roles #1026138
- ldap_authorization: fixed 1 case insensitivity issue; may be more. target role mappings are case sensitive in php, but not in mysql.
Comment #17
jdelaune commentedThanks for your work on this, it's looking good.
The company I work for are looking to move to ldap for all authentication. In the past we have used ldap_integration only for internal staff.
This module looks the most promising for our upcoming D7 projects but we might need some D6 backporting once the D7 branch is stable enough.
Just wanted to touch base and see if there is anything I can do. I'll be honest and say I know very little about ldap but I'm help to help out with the Drupal side of things. I've been dabbling in some D7 things and I'm very similar with D6. If funding is needed to speed progress along then I'll ask my boss if we can do anything, but we're a non-profit so it won't be massive.
Let me know if there are any low hanging fruit I can pick off.
Thanks
Comment #18
johnbarclay commentedFunding would help to get some outside eyes on the module or perhaps set up test scripts. I would likely use it to hire someone to give the module a once over. I don't have any extra purchaseable time as I'm quite busy so it would not help out with the ldap_authorization part I'm working on. It may be useful in the long run or for some of the other developers, but I don't think they have time for this project at the moment.
What the project really needs is more hands on at this point. Its not a viable project without more active participants. We are particulaly weak in issue queue support, testing, and documentation. There is also much coding to do. Maybe you can get a time commitment from your boss to work on some tasks; you seem to have a good deal of drupal contrib coding experience.
Here is the path I would recommend:
- read up a little on the project in the issue queue
- take a read through the documentation pages and make any obvious improvements
- pick a clear initial role for yourself.
-- If you really need a backport of this to d6, that might be your role. My guess is that if there is an insurmountable problem with ldap_integration for d6, that a patch/fork may be a better solution for you.
-- this task would be of the most use: #1062994: Automated Testing for Developers but its a big one. If you are familiar with this sort of testing, it might be a good task. We aren't looking for a crazy amount of functionality in the modules, besides the existing feature set, our highest priorities are test coverage, good documentation, support for exportables, and other goodies that create a stable and secure authn/authz module.
-- some other issues http://drupal.org/node/806068#comment-4097492
Comment #19
rconstantine commentedSubscribing. Not sure what my bandwidth is, but I'll see if I can't pitch in. I pretty much can't do D7 development where I work unless I have LDAP integration, so it's in my interest to help. I need to get some things off my plate first.
So shall I do a basic D7 install, then run through the issue queue and see what I find?
Comment #20
micahw156This conversation has moved to #1115704: Drupal 7 Status Updates.