wrong error msg in create.py

[wolfeggers]

When I create a list using create.py (http://hostname/create) and enter an invalid list name (including umlauts for example), MM says:

"Bad owner email address: ...@lists...."
altough the error message should be
"Illegal list name: ...".

This can be fixed in MailList.py
def Create(...

in MM 2.1.10 line 490 by replacing
       try:
            Utils.ValidateEmail(postingaddr)
        except Errors.MMBadEmailError:
by
       try:
            Utils.ValidateEmail(postingaddr)
        except (Errors.MMBadEmailError, Errors.EmailAddressError):

I haven't really looked into Errors.py/Utils.py and why the error thrown by ValidateEmail is "EmailAddressError" and not "MMBadEmailError". But the above change fixed the problem for me. :)

cheers,
Gero
(<email address hidden>)

[msapiro (msapiro-users)]

1) It is only by accident I saw this report. All Mailman bugs, even for 2.1
should be reported at <https://bugs.launchpad.net/mailman>. The sourceforge
tracker is only available so old URLs to bug reports still work.

2) You are correct about the bug, but the fix

 except (Errors.MMBadEmailError, Errors.EmailAddressError):

is redundant as Errors.MMBadEmailError is a sub class of Errors.EmailAddressError so just changing it to

 except Errors.EmailAddressError:

is just as good. The real problem is Utils.ValidateEmail() considers any address which has non-ascii characters to be 'hostile' and raises Errors.MMHostileAddress which is another sub-class of Errors.EmailAddressError.

The fix will be in the next release.