[Mailman-Users] sync_members problem

[Brent Kearney]

Hi,

I found the problem with my data by temporarily adding a  
print_("Checking:" ... line to sync_members at around line 209:

     for name, addr in filemembers:
         try:
             print _('Checking: %(addr)1s')
             Utils.ValidateEmail(addr)
         except Errors.EmailAddressError:
             print _('Invalid: %(addr)1s')
             invalid = 1
     if invalid:
         print _('You must fix the preceding invalid addresses first.')
         sys.exit(1)

There was a comma in one of the email addresses, like this:  
user at domain,.edu.

So I solved my data problem, but the question remains as to why  
add_members didn't fail in the same place.

Cheers,
Brent

On Sep 19, 2008, at 15:08 , Brent Kearney wrote:

> Hello,
>
> I'm having a strange problem with the sync_members script on  
> Maiman.  I have a text file with about 8500 emails, one per line,  
> and I've filtered out whitespace for each email in the file, and  
> checked that they all have "@".  I can add them to a mail list using  
> "add_members -r /tmp/file.txt", and they all get subscribed without  
> any errors.  However, if I run sync_members with the same text file,  
> I get the error:
>
> Invalid :                            .edu
> You must fix the preceding invalid addresses first.
>
> How can it work with add_members but fail with sync_members? About  
> 3700 of the emails on that list are .edu, and they all look  
> legitimate to me.  I could remove all members, then add all members  
> each time I want to sync this list with our dynamic data source, but  
> it would be nice if sync_members were functional.
>
> Any advice is appreciated.
>
> Thanks,
> Brent
>
>