Re: ldap.open() SSL patch

2010-04-19 Thread Adam Tauno Williams
On Mon, 2010-04-19 at 12:12 -0300, ygormu...@dcc.ufba.br wrote: Michael Ströder mich...@stroeder.com: Just raising DeprecationWarning exception breaks existing code. I have to reject your patch. ;-) I understand. I've checked in a solution which only outputs a warning (just like md5.py

Re: ldap.open() SSL patch

2010-04-18 Thread Michael Ströder
ygormu...@dcc.ufba.br wrote: Michael Ströder mich...@stroeder.com: Yes, you're right. I would accept such a patch. ;-) At least it's documented: http://www.python-ldap.org/doc/html/ldap.html#ldap.open I was using ldap.open because there are many code samples where it's still used, in

ldap.open() SSL patch

2010-04-17 Thread ygormutti
Hi, I'm a member of GrudeSL (Free Software Development Group) at UFBA (Federal University of Bahia). We were trying to integrate our new Django site authentication with the existing LDAP server, so we ran for python-ldap and I can say it does its job very well (thank you very much! :D).

Re: ldap.open() SSL patch

2010-04-17 Thread ygormutti
Michael Ströder mich...@stroeder.com: Using ldap.open() is deprecated for quite a while now. python-ldap applications still using this function should be fixed. In this case, I think ldap.open() must raise a DeprecationWarning. Don't you? -- Ygor Mutti (http://icaju.com - @Mamutti)