[Mailman-Developers] lowercase in mailman create function

Barry Warsaw barry at python.org
Fri Oct 5 23:15:57 CEST 2007


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Oct 5, 2007, at 3:51 PM, Mark Sapiro wrote:

> We still need to make sure that we only create lists with lower case
> names in the first place. The API which is the Create() method needs
> to enforce this. So, I'm inclined to lower-case the name provided to
> Create(), but also lower-case the name provided to Utils.list_exists 
> ().

I'm actually not convinced this is a bug, but a problem with the  
client of Create() and list_exists().  The problem is that once you  
lowercase the list name in those two methods, you might have to start  
lowercasing them in an API that might get called via script, web, or  
email.  I really don't think we need viral .lower() calls all over  
the place.

It's tempting to lowercase in Create() but I think it's better to  
lowercase the list names at the edges of the system, in much the same  
way that we should be converting strings to unicode at the edges of  
the system.  This means that Lawren's email and web extensions should  
be doing the lowercasing there.

Another problem of course is that the constraints for internal API  
functions like Create and list_exists() isn't really documented  
anywhere.  I'm trying to improve that situation for Mailman 3.

Cheers,
- -Barry

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.7 (Darwin)

iQCVAwUBRwapjnEjvBPtnXfVAQJZ4QP/RrFuMZ3FcwzCWFdDdq1xdBVvV6ZUcllB
+J/nHS5vqY8Fw36JcbJ+BlmzDbwNcWk6FivcjQehktt2owkYJjEZPfvTXytpjgvr
Um7UjX4Vbc+p9u2e3l2J3rQuwTt8HNZmU8yTgX7x/lzgLnvHOGujuqXbAkSv5ILv
3gpbHGaOqbM=
=A+Ef
-----END PGP SIGNATURE-----


More information about the Mailman-Developers mailing list