You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@struts.apache.org by "James Holmes (JIRA)" <ji...@apache.org> on 2008/12/05 15:11:36 UTC

[jira] Commented: (WW-2805) Email validator regex is incorrect. Does not allow quotes

    [ https://issues.apache.org/struts/browse/WW-2805?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=45126#action_45126 ] 

James Holmes commented on WW-2805:
----------------------------------

I did a little research on this and apparently the apostrophe character is considered legal.

See here:
http://www.remote.org/jochen/mail/info/chars.html

This will have to be "fixed" in XWork in the following class:

com.opensymphony.xwork2.validator.validators.EmailValidator

> Email validator regex is incorrect.  Does not allow quotes
> ----------------------------------------------------------
>
>                 Key: WW-2805
>                 URL: https://issues.apache.org/struts/browse/WW-2805
>             Project: Struts 2
>          Issue Type: Bug
>          Components: XML Configuration
>    Affects Versions: 2.1.3
>            Reporter: Matthew Payne
>
> The following email address should be valid: 
> D'Souza.Shawn@pennmutual.com
> O'Malley.Dave@xyz.com
> Refer to jquery's validation for proper validation
> http://jquery.bassistance.de/validate/jquery.validate.js
> // http://docs.jquery.com/Plugins/Validation/Methods/email
> 		email: function(value, element) {
> 			// contributed by Scott Gonzalez: http://projects.scottsplayground.com/email_address_validation/
> 			return this.optional(element) || /^((([a-z]|\d|[!#\$%&'\*\+\-\/=\?\^_`{\|}~]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])+(\.([a-z]|\d|[!#\$%&'\*\+\-\/=\?\^_`{\|}~]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])+)*)|((\x22)((((\x20|\x09)*(\x0d\x0a))?(\x20|\x09)+)?(([\x01-\x08\x0b\x0c\x0e-\x1f\x7f]|\x21|[\x23-\x5b]|[\x5d-\x7e]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])|(\\([\x01-\x09\x0b\x0c\x0d-\x7f]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF]))))*(((\x20|\x09)*(\x0d\x0a))?(\x20|\x09)+)?(\x22)))@((([a-z]|\d|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])|(([a-z]|\d|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])([a-z]|\d|-|\.|_|~|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])*([a-z]|\d|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])))\.)+(([a-z]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])|(([a-z]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])([a-z]|\d|-|\.|_|~|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])*([a-z]|[\u00A0-\uD7FF\uF900-\uFDCF\uFDF0-\uFFEF])))\.?$/i.test(element.value);
> 		},

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.