[PATCH manager 1/2] fix #7649: ui: active directory realm: expose user attribute name option

[Michal Fox <me@dualfroz.com>]

AD realms support the 'user_attr' option, which sets the attribute used
as username when syncing users, but it can only be set via the CLI. It
is needed for logging in with the user principal name, if that does not
end with the AD domain: with 'userPrincipalName' as attribute, the users
are synced with their full user principal name, which is then used as
is to bind to the server on login.

Add a field for it, like the LDAP realm has, with the 'sAMAccountName'
default as empty text.

Signed-off-by: Michal Fox <me@dualfroz.com>
---
Tested in the browser with the realm edit window and mocked API calls:
on create, the new field is only sent if set; on edit, a 'user_attr'
set via the CLI is shown, kept when saving, and deleted when the field
is cleared. Without the patch, there is no field, and the value can
only be changed via the CLI.

That the attribute is only used by the sync and the login binds with
the given name as is, if it contains an '@', is from reading
PVE::Auth::AD; the reporter confirmed that UPN logins work with it.

 www/manager6/dc/AuthEditAD.js | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/www/manager6/dc/AuthEditAD.js b/www/manager6/dc/AuthEditAD.js
index ec5e7808..530b1a67 100644
--- a/www/manager6/dc/AuthEditAD.js
+++ b/www/manager6/dc/AuthEditAD.js
@@ -17,6 +17,13 @@ Ext.define('PVE.panel.ADInputPanel', {
                 emptyText: 'company.net',
                 allowBlank: false,
             },
+            {
+                xtype: 'proxmoxtextfield',
+                name: 'user_attr',
+                fieldLabel: gettext('User Attribute Name'),
+                emptyText: Proxmox.Utils.defaultText + ' (sAMAccountName)',
+                deleteEmpty: !me.isCreate,
+            },
             {
                 xtype: 'proxmoxcheckbox',
                 fieldLabel: gettext('Case-Sensitive'),
-- 
2.43.0