[pve-devel] [PATCH common] fix #2111: regex match email address without tld
Oguz Bektas
o.bektas at proxmox.com
Fri Feb 22 16:05:55 CET 2019
do not apply this (just noticed after some internal discussion with
thomas and dominik that it could possibly cause problems), i'll take a better look on monday.
On Fri, Feb 22, 2019 at 03:10:13PM +0100, Oguz Bektas wrote:
> for matching root at localhost or similar.
>
> expression taken from emailregex.com for html5.
>
> Signed-off-by: Oguz Bektas <o.bektas at proxmox.com>
> ---
> src/PVE/JSONSchema.pm | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/PVE/JSONSchema.pm b/src/PVE/JSONSchema.pm
> index 36fa994..b5f9621 100644
> --- a/src/PVE/JSONSchema.pm
> +++ b/src/PVE/JSONSchema.pm
> @@ -356,8 +356,8 @@ register_format('email', \&pve_verify_email);
> sub pve_verify_email {
> my ($email, $noerr) = @_;
>
> - # we use same regex as in Utils.js
> - if ($email !~ /^(\w+)([\-+.][\w]+)*@(\w[\-\w]*\.){1,5}([A-Za-z]){2,63}$/) {
> + # html5 email regex
> + if ($email !~ /^[a-zA-Z0-9.!#$%&’*+/=?^_`{|}~-]+@[a-zA-Z0-9-]+(?:\.[a-zA-Z0-9-]+)*$/) {
> return undef if $noerr;
> die "value does not look like a valid email address\n";
> }
> --
> 2.11.0
>
>
More information about the pve-devel
mailing list