[Mailman-Users] Subscription errors

[Mark Sapiro]

Duncan Drury wrote:
> 
> This seems to be related to the known bug in 2.1.10 and I have tried
> installing the patch that has been discussed in many places - but I am still
> getting the same problem.  In log/error I am seeing the following:
> 
> May 28 23:36:05 2008 (31672) Uncaught runner exception: 'NoneType' object
> has no attribute 'lower'
> May 28 23:36:05 2008 (31672) Traceback (most recent call last):
>   File "/usr/local/mailman/Mailman/Queue/Runner.py", line 120, in _oneloop
>     self._onefile(msg, msgdata)
>   File "/usr/local/mailman/Mailman/Queue/Runner.py", line 191, in _onefile
>     keepqueued = self._dispose(mlist, msg, msgdata)
>   File "/usr/local/mailman/Mailman/Queue/CommandRunner.py", line 241, in
> _dispose
>     res.do_command('join')
>   File "/usr/local/mailman/Mailman/Queue/CommandRunner.py", line 139, in
> do_command
>     return handler.process(self, args)
>   File "/usr/local/mailman/Mailman/Commands/cmd_subscribe.py", line 74, in
> process
>     if digest is None and password.lower() in ('digest', 'nodigest'):
> AttributeError: 'NoneType' object has no attribute 'lower'


You haven't installed the patch correctly, or you patched something 
other than /usr/local/mailman/Mailman/Commands/cmd_subscribe.py.

The properly installed patch changes line 74 of 
/usr/local/mailman/Mailman/Commands/cmd_subscribe.py from

     if digest is None and password.lower() in ('digest', 'nodigest'):

to

     if (digest is None
             and password and password.lower() in ('digest', nodigest')):

but if you are changing it manually, you could just change

     if digest is None and password.lower() ...

to

     if digest is None and password and password.lower() ...

i.e. just insert ' and password' following 'None'.

-- 
Mark Sapiro <mark at msapiro.net>        The highway is for gamblers,
San Francisco Bay Area, California    better use your sense - B. Dylan