Hi Dave, Yes, please file an issue in our issue tracker:
http://code.google.com/p/googleappengine/issues/list On Wed, Apr 7, 2010 at 10:18 AM, Dave <[email protected]> wrote: > Hi- > > I was writing a request handler that sent an email address and was > very surprised when my unit tests failed due to is_email_valid > returning unexpected false positives. Upon further investigation of > the sdk source[1], turns out that invalid_email_reason is doing a very > very basic string validation and not actually checking any valid email > address guidelines. > > I created a list of very basic email address problems that I had > expected to get caught[2], and it looks like the live version of gae > is doing the same thing. > > I have seen many email validation regexes was really hoping that this > function did what I had expected. Should I file a bug w/a patch? Or > are there other workarounds that anyone else has used? Just sending > and catching an Error (not InvalidEmailError for the listed reasons > though)? > > links: > [1] invalid_email_reason source in google code > > http://code.google.com/p/googleappengine/source/browse/trunk/python/google/appengine/api/mail.py#115 > [2] test handler and response source > http://gist.github.com/359138 > > -- > You received this message because you are subscribed to the Google Groups > "Google App Engine" group. > To post to this group, send email to [email protected]. > To unsubscribe from this group, send email to > [email protected]<google-appengine%[email protected]> > . > For more options, visit this group at > http://groups.google.com/group/google-appengine?hl=en. > > -- Ikai Lan Developer Programs Engineer, Google App Engine http://googleappengine.blogspot.com | http://twitter.com/app_engine -- You received this message because you are subscribed to the Google Groups "Google App Engine" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/google-appengine?hl=en.
