17488 Posts in 4473 Topics by 1978 members
| Go to End | Next > | |
| Author | Topic: | 13008 Views |
-
Re: LDAP support for Silverstripe
10 October 2007 at 6:16am
Hi Lancer,
I looked at the code (without actually trying it).. good work!
The only thing I would change I would suggest is to remove the global array $LDAPSERVER. You could use static members on the LDAPAuthenticator class instead.
-
Re: LDAP support for Silverstripe
10 October 2007 at 6:36am
> The only thing I would change I would suggest is to remove the global array $LDAPSERVER.
yeah, that would've been my suggestion as well - planned to write that as a trac-ticket-comment. we should minimize the use of global namespaces, and adhere to silverstripe-conventions.i've extended the wiki-page on this topic a bit the other day:
http://doc.silverstripe.com/doku.php?id=configurationskim-read through the code - great stuff lancer!
-
Re: LDAP support for Silverstripe
10 October 2007 at 8:31am Last edited: 11 October 2007 3:34am
Ah, old habits die hard. This was the conventions for configs at the Horde project. I updated the patch in trac (the change wasn't hard, just a lot of work ;-) )
[edit]
Oops, I forgot to convert the $this->_ds into self::$ds. Will do that this evening
[/edit][edit2]
Done. Updated the ticket again. Now it should be compliant with the conventions
[/edit2] -
Re: LDAP support for Silverstripe
14 October 2007 at 11:25pm Last edited: 14 October 2007 11:27pm
Based on the LDAP code I can very easily write an authentication module that uses IMAP/POP3/SMTP (with and without SSL/TLS). That's probably safer to use if your server is in the DMZ than exposing your internal LDAP server to the DMZ (most companies have these protocols running on the outside interfaces nowadays).
Would there be any interest in that?
[note]
Wouldn't it be an idea to remove the obsolete tag from this forum? Code and direct comments on the code should be put in trac, but discussing code could be done here.
[/note] -
Re: LDAP support for Silverstripe
15 October 2007 at 1:23pm
Re "authentication module that uses IMAP/POP3/SMTP" that sounds useful to me
Keep up the great work
-
Re: LDAP support for Silverstripe
15 October 2007 at 6:17pm
Hi!!!
i'm new to silverstripe and i saw there is a beta for ldap auth???
i saw it, but i have no idea how to test it. Lancer, may you drop us a line to know how to implement it for test?
Thanks a lot. greetings!!!
-
Re: LDAP support for Silverstripe
15 October 2007 at 6:26pm
More like an alpha, since the code is not yet in svn.
Do a checkout of the Silverstripe trunk. Then download the files from trac ticket 1477. Unpack the tar in the silverstripe root and patch the _config.php in the sapphire directory. Then edit that _config.php to suit your LDAP.
| 13008 Views | ||
| Go to Top | Next > |
