[Mailman-Users] Problems with Mailman LDAPMemberships Adapter

[Chris Nulk]

Hi Mark,

I don't know about others but I still use the LDAP adapter here. And I 
still appreciate the assistance you gave to get it working.

Thanks again,
Chris

On 6/5/2013 5:47 PM, Mark Sapiro wrote:
> On 06/05/2013 08:29 AM, Lennart Barfod wrote:
>> Hi Mark,
>>
>> I'm using the LDAPMembership.py Version 0.63.
>>
>> Yes I restartet mailman some times after adding the extend.py.
>>
>> Ok, so I should reinstall mailman and Python-ldap? Or where seems to be the problem?
>
> There seems to be a serious coding error in LDAPMemberships.py, version
> 0.63. Namely, at line 274-275, the code
>
>      def __ldap_get_members(self):
>          return self.__ldap_get_regularmembers() +
> self.__ldap_get_digestmembers()
>
> (watch out for wrapped line)
>
> should be
>
>      def __ldap_get_members(self):
>          return self.__ldap_get_regular_members() +
> self.__ldap_get_digest_members()
>
>
> I wonder how many people are actually using this?
>
> If you fix that and restart Mailman, I think you'll be OK.
>