[pve-devel] [PATCH v3 common] fix #2111: regex match for email addresses
Oguz Bektas
o.bektas at proxmox.com
Thu Feb 28 13:40:26 CET 2019
now allows:
* addresses without tld (such as user at localhost per bug)
* change the min char limit from 2 to 1 in the host part
* allow more than 5 subdomains
* allow addresses ending with +, -
also used the backslash escape character in the character sets, in
hopes of improving readability.
Signed-off-by: Oguz Bektas <o.bektas at proxmox.com>
---
src/PVE/JSONSchema.pm | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/PVE/JSONSchema.pm b/src/PVE/JSONSchema.pm
index 36fa994..1f93a6b 100644
--- a/src/PVE/JSONSchema.pm
+++ b/src/PVE/JSONSchema.pm
@@ -356,8 +356,7 @@ 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}$/) {
+ if ($email !~ /^\w+([\+\-\.]\w*)*@([a-zA-Z0-9\-]+\.)*([a-zA-Z0-9\-]){1,63}$/) {
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