From eaa8d084e8d58d3c92f38ab6522aac8b71773e6c Mon Sep 17 00:00:00 2001 From: Stefan Sterz Date: Wed, 7 Sep 2022 11:37:40 +0200 Subject: [PATCH] fix: toolkit: make email regex pattern match pve-common `proxmoxMail` used its own regex pattern to validate emails. that meant certain email addresses were rejected by the front-end that were accepted by the backend that uses the functionality from `pve-common`. examples include the following: - "user@host.test-tld" - "user-@host.testtld" - "user@host" reported on the forum: https://forum.proxmox.com/threads/gui-bugulance-using-the-user-add-gui-interface.114743/ Signed-off-by: Stefan Sterz --- src/Toolkit.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/Toolkit.js b/src/Toolkit.js index ad64f89..c730374 100644 --- a/src/Toolkit.js +++ b/src/Toolkit.js @@ -131,9 +131,9 @@ Ext.apply(Ext.form.field.VTypes, { }, DnsNameOrWildcardText: gettext('This is not a valid DNS name'), - // workaround for https://www.sencha.com/forum/showthread.php?302150 + // email regex used by pve-common proxmoxMail: function(v) { - return (/^(\w+)([-+.][\w]+)*@(\w[-\w]*\.){1,5}([A-Za-z]){2,63}$/).test(v); + return (/^[\w+-~]+(\.[\w+-~]+)*@[a-zA-Z0-9-]+(\.[a-zA-Z0-9-]+)*$/).test(v); }, proxmoxMailText: gettext('Example') + ": user@example.com", -- 2.39.2