[prev in list] [next in list] [prev in thread] [next in thread] 

List:       squirrelmail-devel
Subject:    Re: [SM-DEVEL] Options->Personal Information: No validation
From:       "Thijs Kinkhorst" <kink () squirrelmail ! org>
Date:       2006-05-12 10:09:20
Message-ID: 50678.145.99.151.133.1147428560.squirrel () wm ! kinkhorst ! com
[Download RAW message or body]

On Thu, May 11, 2006 21:12, Bryan Loniewski wrote:
> 
> The Email Address and Reply To fields (among others) on the Personal
> Information page are optional, however, if one does fill them in,
> there is no validation done. Some of our users typo the Email
> Address field (ie, user@foo.rutgers,edu) and mail does not get
> delivered for various reasons (ie, user@foo.rutgers,edu will
> translate to just user@foo.rutgers and postfix complains "Sender
> address rejected: Domain not found").
> 
> Any plans to provide validation code for this page and possibly any
> others that may need it. I was looking through the code and could not
> find any validation functions, did I overlook something, if so could
> you give me a pointer to where it is, and I may try to apply a patch
> to my above issue(s).

I agree that simple validation for those fields would be a good idea. I'll
prepare a patch for that for devel. There is validation code present in
SquirrelMail, although there's not many places where we actually have a
need to check the syntax of an email address.


Thijs



-------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd--
squirrelmail-devel mailing list
Posting Guidelines: http://www.squirrelmail.org/wiki/MailingListPostingGuidelines
List Address: squirrelmail-devel@lists.sourceforge.net
List Archives: http://news.gmane.org/thread.php?groupList Archives: \
http://sourceforge.net/mailarchive/forum.php?forum_idq39 List Info: \
https://lists.sourceforge.net/lists/listinfo/squirrelmail-devel


[prev in list] [next in list] [prev in thread] [next in thread]