#2507: [patch] LDAPBackend in django/contrib/auth/backends.py
--------------------------------+-------------------------------------------
Reporter: [EMAIL PROTECTED] | Owner: adrian
Status: new | Component: Contrib apps
Version: | Resolution:
Keywords: | Stage: Accepted
Has_patch: 1 | Needs_docs: 1
Needs_tests: 0 | Needs_better_patch: 1
--------------------------------+-------------------------------------------
Comment (by Scott Paul Robertson <[EMAIL PROTECTED]>):
Replying to [comment:10 Marc Fargas <[EMAIL PROTECTED]>]:
> Replying to [comment:9 Scott Paul Robertson <[EMAIL PROTECTED]>]:
> > Also, why are the calls to the ldap module always done as self.ldap?
It works either way, and removing the self would be cleaner code in my
opinion.
> import ldap was inside the class when I took the patch, so I left it
there. I think there would be no problem to move the import to the top of
the file and then s/self.ldap/ldap/ ;)
>
Sounds good to me.
> > Group membership isn't the same between AD and OpenLDAP. OpenLDAP has
group objects with have lists of users. We should go for a base
LDAPBackend (which doesn't bother with groups), and subclass it for AD and
OpenLDAP.
> AD has the "members" inside the group objects and users have "memberOf",
in OpenLDAP how do you know which groups a user is member of? with a
search?
>
Yeah, you do a search on a group sub-tree to find what groups a user is a
member of. We have a lot of LDAP traffic in our labs. :)
>
> Maybe we can bring discussion to django-developers about where to place
the docs and the .py itself :)
I'll make a post tonight.
--
Ticket URL: <http://code.djangoproject.com/ticket/2507#comment:11>
Django Code <http://code.djangoproject.com/>
The web framework for perfectionists with deadlines
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---
